init: Throw error when network specific config is ignored #15629

1. 
   MarcoFalke commented at 8:31 pm on March 20, 2019: member

   This should have no effect on mainnet users, but simplifies testing, where config settings are currently ignored with only a warning. Fix this by making it an error.

   Issues:

   • bitcoin client 0.17.0 ignores wallet’s name (file) #14523
   • Can’t set custom rpcport on testnet #13777
   • …
2. init: Throw error when network specific config is ignored dddd6f0f58
3. MarcoFalke added the label Tests on Mar 20, 2019
4. MarcoFalke added the label Utils/log/libs on Mar 20, 2019
5. 
   promag commented at 2:59 pm on March 21, 2019: member

   Tested ACK dddd6f0.

   0$ cat bitcoin.conf
   1rpcport=54321
   2
   3bitcoind -testnet
   42019-03-21T14:58:35Z Bitcoin Core version v0.18.99.0-dddd6f0f5 (release build)
   52019-03-21T14:58:35Z Error: Config setting for -rpcport only applied on test network when in [test] section.
   6Error: Config setting for -rpcport only applied on test network when in [test] section.
   

   Deserves a small release note, just to point to if someone reports this behavior change?

6. 
   practicalswift commented at 3:06 pm on March 21, 2019: contributor

   Concept ACK

   Nice usability improvement!

7. 
   promag commented at 7:40 pm on March 21, 2019: member

   @MarcoFalke here’s a test for your consideration 53469f780a8e2d88337f4efa6eb9ca52c9ce34f8.
8. MarcoFalke force-pushed on Mar 21, 2019
9. qa: Add test for missing testnet section in conf file fa4a922d78
10. doc: Add release notes for 15629 fa9058f0ed
11. MarcoFalke force-pushed on Mar 21, 2019
12. doc: Fix all typos reported by codespell fae38c3dc6
13. in doc/release-notes.md:80 in fa9058f0ed outdated

    72@@ -73,9 +73,14 @@ Example item
    73 Low-level changes
    74 =================
    75 
    76-Example item
    77+Configuration
    78 ------------
    79 
    80+* An error is issued where previously a warning was issued when a setting in
    81+  the config file was specified in the default section, but not overriden for
    

    ---

    ---

    

    practicalswift commented at 8:11 am on March 22, 2019:

    Should be “overridden” :-)
14. 
    DrahtBot commented at 11:30 pm on April 6, 2019: member

    The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.

    Conflicts

    No conflicts as of last run.

15. MarcoFalke referenced this in commit 5392aee64f on Apr 10, 2019
16. MarcoFalke merged this on Apr 10, 2019
17. MarcoFalke closed this on Apr 10, 2019

    ---

18. MarcoFalke deleted the branch on Apr 10, 2019
19. 
    promag commented at 1:27 am on April 10, 2019: member

    utACK fae38c3.
20. vijaydasmp referenced this in commit 4082cef036 on Oct 15, 2021
21. vijaydasmp referenced this in commit 0f74ebae16 on Oct 16, 2021
22. vijaydasmp referenced this in commit 96f2c7d1d3 on Oct 16, 2021
23. vijaydasmp referenced this in commit 01ec4b3fb6 on Oct 16, 2021
24. vijaydasmp referenced this in commit 6d8217ed63 on Oct 16, 2021
25. vijaydasmp referenced this in commit 95e5a21d1e on Oct 16, 2021
26. vijaydasmp referenced this in commit ca0c5df7f0 on Oct 16, 2021
27. vijaydasmp referenced this in commit 5df7adac26 on Oct 17, 2021
28. vijaydasmp referenced this in commit 68de6f2973 on Oct 19, 2021
29. PastaPastaPasta referenced this in commit 6e32a464f5 on Oct 21, 2021
30. pravblockc referenced this in commit 9876447066 on Nov 18, 2021
31. ftrader referenced this in commit 9b129ffa47 on Nov 21, 2021
32. ftrader referenced this in commit e553c9c36d on Nov 21, 2021
33. DrahtBot locked this on Dec 16, 2021

Contributors
MarcoFalke promag practicalswift DrahtBot

Labels
Tests Utils/log/libs