HomePhabricator

rpc: Remove chain-specific RequireRPCPassword
85ee55b5c37bUnpublished

Unpublished Commit ยท Learn More

Repository Importing: This repository is still importing.

Description

rpc: Remove chain-specific RequireRPCPassword

I've never liked the chain-specific exception to having to set a
password. It gives issues with #6388 which makes it valid to
set no password in every case (as it enables random cookie authentication).

This pull removes the flag, so that all chains are regarded the same.

It also removes the username==password test, which doesn't provide any
substantial extra security.

Details

Provenance
Wladimir J. van der Laan <laanwj@gmail.com>Authored on Jul 8 2015, 12:47
deadalnixPushed on May 14 2017, 22:04
Parents
rABC11576a57d2ee: Merge pull request #6387
Branches
Unknown
Tags
Unknown

Event Timeline

Wladimir J. van der Laan <laanwj@gmail.com> committed rABC85ee55b5c37b: rpc: Remove chain-specific RequireRPCPassword (authored by Wladimir J. van der Laan <laanwj@gmail.com>).Jul 10 2015, 13:01