diff options
| author | Wladimir J. van der Laan <laanwj@gmail.com> | 2015-07-08 14:47:39 +0200 |
|---|---|---|
| committer | Wladimir J. van der Laan <laanwj@gmail.com> | 2015-07-10 15:01:55 +0200 |
| commit | 85ee55b5c37bedca98ec472439e1710277e8e4b2 (patch) | |
| tree | 23b253176a0570875c274982ec1172545e40d699 /src/chainparams.cpp | |
| parent | 11576a57d2ee37aa421e0fce69e70751b5af9a61 (diff) | |
| download | bitcoin-85ee55b5c37bedca98ec472439e1710277e8e4b2.tar.xz | |
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.
Diffstat (limited to 'src/chainparams.cpp')
| -rw-r--r-- | src/chainparams.cpp | 3 |
1 files changed, 0 insertions, 3 deletions
diff --git a/src/chainparams.cpp b/src/chainparams.cpp index 7785417518..3d172765cb 100644 --- a/src/chainparams.cpp +++ b/src/chainparams.cpp @@ -99,7 +99,6 @@ public: vFixedSeeds = std::vector<SeedSpec6>(pnSeed6_main, pnSeed6_main + ARRAYLEN(pnSeed6_main)); - fRequireRPCPassword = true; fMiningRequiresPeers = true; fDefaultConsistencyChecks = false; fRequireStandard = true; @@ -171,7 +170,6 @@ public: vFixedSeeds = std::vector<SeedSpec6>(pnSeed6_test, pnSeed6_test + ARRAYLEN(pnSeed6_test)); - fRequireRPCPassword = true; fMiningRequiresPeers = true; fDefaultConsistencyChecks = false; fRequireStandard = false; @@ -218,7 +216,6 @@ public: vFixedSeeds.clear(); //! Regtest mode doesn't have any fixed seeds. vSeeds.clear(); //! Regtest mode doesn't have any DNS seeds. - fRequireRPCPassword = false; fMiningRequiresPeers = false; fDefaultConsistencyChecks = true; fRequireStandard = false; |