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

Load config files before applying cmdargs options. Fixes #6014 #6043

wants to merge 2 commits into
base: main
Choose a base branch


Copy link

@wesolowski-gh wesolowski-gh commented Apr 2, 2023


It's an attempt to fix issue 6014. I've moved config files loading before applying options from command line arguments. I'm not sure it's a correct solution thou because it still separately notifies about changes of options from config files and command line arguments. I think it would be better to merge options from config files and command line (where the first ones are overwritten by the second ones) and only then once notify about changed option values.

Resolves #6014

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

Successfully merging this pull request may close these issues.

cert_passphrase has no effect when declared in config.yaml
1 participant