Fail on unknown config file options #15021

1. 
   MarcoFalke commented at 7:16 pm on December 21, 2018: member

   A solution would be to either make all binaries be aware of each other’s options, or to permit config file options that only apply to specific binaries (bitcoind, bitcoin-qt, bitcoin-cli) or the wallet. (https://github.com/bitcoin/bitcoin/pull/13799#issue-204717028)
2. MarcoFalke added the label Utils/log/libs on Dec 21, 2018
3. MarcoFalke referenced this in commit ea45967e85 on Jun 20, 2019
4. sidhujag referenced this in commit 63f310d35e on Jun 20, 2019
5. 
   luke-jr commented at 5:33 pm on December 3, 2020: member

   Concept NACK, this creates unnecessary problems switching between versions for no reason
6. 
   harding commented at 5:36 pm on December 3, 2020: contributor

   Concept ACK. A good reason for a policy of failing on unknown settings would be eliminating the annoyance of having to figure out that your configuration is broken and then having to restart the node—which is something that can take a long time (e.g. if you have a high db cache or are using an underpowered computer) and which inexperienced users might not know how to do (or might do wrong, e.g. sending SIGKILL rather than SIGTERM, which could result in a need to reindex).

   Based on my own misadventures, wrongly configuring a node is a much more common problem than wanting to use a future/spinoff configuration file with an old/baseline node, so I think we should optimize for helping users fix misconfigurations as fast and painlessly as possible.

7. 
   promag commented at 5:40 pm on December 3, 2020: member

   I still think that a -strict option (on by default on future releases) makes everyone happy.
8. 
   promag commented at 5:42 pm on December 3, 2020: member

   Concept NACK, this creates unnecessary problems switching between versions for no reason @luke-jr do you agree that very few do this?

9. 
   MarcoFalke commented at 5:44 pm on December 3, 2020: member

   Switching versions happens on literally every upgrade/downgrade. Ignoring unknown config values is dangerous and may lead to loss of funds.
10. 
    luke-jr commented at 6:11 pm on December 3, 2020: member

    @promag As @MarcoFalke points out, pretty much everyone does it…
11. linuxsh2 referenced this in commit 0bcf02739b on Aug 11, 2021

Contributors
MarcoFalke luke-jr harding promag

Labels
Utils/log/libs