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

Allow to specify specify ldap_tls_cacert_file or/and ldap_tls_cacert_dir #1704

Closed
williamdes opened this issue Dec 6, 2020 · 2 comments · Fixed by #1708
Closed

Allow to specify specify ldap_tls_cacert_file or/and ldap_tls_cacert_dir #1704

williamdes opened this issue Dec 6, 2020 · 2 comments · Fixed by #1708

Comments

@williamdes
Copy link
Member

https://github.com/tomav/docker-mailserver/blob/3780783145d3fe118f446b850b6cdb22d5146de0/target/start-mailserver.sh#L935

Source: https://blog.sys4.de/cyrus-sasl-saslauthdconf-man-page-en.html
And https://github.com/winlibs/cyrus-sasl/blob/master/saslauthd/LDAP_SASLAUTHD#L215

ldap_tls_check_peer (default: no)
Require and verify server certificate. If this option is yes, you must specify ldap_tls_cacert_file or ldap_tls_cacert_dir.

Seems to come from #980

IMO, the options should be provided to the user

@erik-wramner
Copy link
Contributor

Feel free to submit a PR. Alternatively (if you just need to fix this for your server) you can use user-patches.sh to set the options at startup.

@williamdes
Copy link
Member Author

Hi !
Done in #1708 :)

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

Successfully merging a pull request may close this issue.

2 participants