Validation of Configuration Parameters #10091

issue jlopp opened this issue on March 26, 2017
  1. jlopp commented at 10:02 PM on March 26, 2017: contributor

    The current way that incorrectly named configuration parameters are handled (they are silently ignored) seems to be a poor user experience for novice node operators. Because config values are only fetched when needed at runtime, if a config parameter is passed in with a typo, it is assumed not to exist and the default value is used.

    Wouldn't it be an improvement to add logic during node initialization that iterates through all user-supplied configuration parameters and checks that they are supported parameter names? It seems like this would require the creation of a single data structure that can act as a directory for all available parameters.

  2. jonasschnelli commented at 6:30 AM on March 27, 2017: contributor

    Yes. This would be a great improvement. Ideally we create a structure that contains all available configuration options with help-information, rules how they depend/interact with other options. A check for invalid options during startup would be good. The only downside might be that an older version with a newer configuration (or vice versa) could break the startup (in case we stop the startup process).

  3. jonasschnelli added the label Feature on Mar 27, 2017
  4. laanwj commented at 7:14 AM on March 27, 2017: member

    Agreed. Duplicate of #1044, though.

    The only downside might be that an older version with a newer configuration (or vice versa) could break the startup (in case we stop the startup process).

    Running an older version with a newer configuration is not supported. The other way around should usually work, but if we do remove or rename command line arguments it's better if the user is made aware of that. And some arguments could be explicitly ignored in some cases.

  5. jlopp commented at 12:19 PM on March 27, 2017: contributor

    Aha, I figured this had probably been discussed before but didn't find 1044 - I'll close this issue.

  6. jlopp closed this on Mar 27, 2017

  7. MarcoFalke locked this on Sep 8, 2021
Labels

github-metadata-mirror

This is a metadata mirror of the GitHub repository bitcoin/bitcoin. This site is not affiliated with GitHub. Content is generated from a GitHub metadata backup.
generated: 2026-05-01 18:15 UTC

This site is hosted by @0xB10C
More mirrored repositories can be found on mirror.b10c.me