Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

witness_node uses two incompatible parsers for config.ini #521

Closed
pmconrad opened this issue Jan 14, 2016 · 0 comments
Closed

witness_node uses two incompatible parsers for config.ini #521

pmconrad opened this issue Jan 14, 2016 · 0 comments
Milestone

Comments

@pmconrad
Copy link

The config.ini file used by the witness_node is parsed twice, with different parsers:

  1. By bpo::parse_config_file to get the basic options
  2. By boost::property_tree::ini_parser::read_ini for the logging configuration

These two parsers are not entirely compatible, which sometimes leads to the second parse failing. This results in the default logging config being used instead of that in the config file.

Specifically, the ini_parser does not support repeating options. This conflicts with the documented (in the default config file) behaviour wrt the track-account option "Account ID to track history for (may specify multiple times)".

The problem is difficult to diagnose, because the reason for the parse failure is not logged.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants