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

Parse config file at startup and produce errors if some important sections are missing #31

Open
lelutin opened this issue Nov 2, 2022 · 0 comments

Comments

@lelutin
Copy link

lelutin commented Nov 2, 2022

  • onionrouter version: 0.6.2
  • Python version: 3.9.2
  • Operating System: debian bullseye

Description

Once a configuration file exists and the daemon is able to start, it might end up responding to requests with errors about some configuration sections or options missing when postfix sends a request to it.

For example:

$ nc localhost 23000
[email protected]
500 No section: 'IGNORED'

or with only the IGNORED section delimiter present without anything in it:

$ nc localhost 23000
[email protected]
500 No option 'domains' in section: 'IGNORED'

Since the daemon will effectively be useless (and will indefinitely stall queries by postfix) it would make more sense for the daemon to find those configuration problems and noisily exit upon starting up. This would make it easier to debug the problem and make the service functional.

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

1 participant