-
Notifications
You must be signed in to change notification settings - Fork 47
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
Allow to read bind_password from a file #172
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks really useful, thanks!
Could we update the README to mention the new config option?
Also do you mind adding a sign off to confirm that you're happy for your contribution to fall under the project license, etc?
https://matrix-org.github.io/synapse/latest/development/contributing_guide.html#sign-off
27a9e74
to
ec3062c
Compare
Updated the README accordingly and added a |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good! Thanks for updating the README.
One very minor wording suggestion:
ec3062c
to
ae16854
Compare
The linter check in CI seems unhappy. Could you give |
That way you don't have to leak your bind password into your config. Useful for e.g. NixOS where config is stored in a world-readable location. Signed-off-by: Maximilian Bosch <maximilian@mbosch.me>
ae16854
to
cd5b6d9
Compare
Applied the diff from CI for now, let's see if that's sufficient. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. Thank you for your contribution!
AFAICS the mypy error also occurs on master, so it's not actionable here, correct? |
Yes, seems like it. |
That way you don't have to leak your bind password into your config.
Useful for e.g. NixOS where config is stored in a world-readable
location.