Added features: saslauthd-ldap support, postfix-ldap support, postfix…#335
Added features: saslauthd-ldap support, postfix-ldap support, postfix…#335alinmear wants to merge 5 commits intodocker-mailserver:masterfrom alinmear:master
Conversation
I am currently working on a kopano docker integration and needed ldap support, for authentication and also for user validation. I also updated the README.
|
Integration Test failed because of failure retrieving gpg key. I tested it and it worked: https://travis-ci.org/alinmear/docker-mailserver |
|
Maybe restarting the test could help ... I try |
|
Would fix #187 |
| ##### LDAP | ||
| - **empty** => LDAP support disabled | ||
| - 1 => set virtual_mailbox_maps = ldap:/etc/postfix/ldap-users.cf and virtual_alias_maps = ldap:/etc//postfix/ldap-aliases.cf | ||
| - $SMTP_ONLY must be set to 1 |
There was a problem hiding this comment.
Why SMTP_ONLYmust be set? Dovecot also supports LDAP authentication.
There was a problem hiding this comment.
i implemented this because i needed those features for a kopano (former zarafa) integration. i didn't implement ldap for dovecot, but postfix-ldap and saslauthd-ldap. Maybe we could write this to the LDAP Section of the README
edit:
and take this as starting point of the dovecot ldap implementation.
|
@alinmear any way to have integration tests as highlighted by @dennis95stumm ? |
|
Does this mean you removed dovecot globably from the project? Why should we use kopano? |
|
Does not seem to be the case when looking at the changes, hopefully. |
|
I am not sure, what you mean in particular @kopax. I am using this docker image in different use cases. So i implemented the kopano, sslauthd, ldap as options, so one can chose between them, depending on the particular needs. Do i miss something here because i can't see, where i touched the dovecot implementation. I contributed this here, because @tomav said in his Project Description, that he wanna provide a full featured mailserver based on postfix. This would indicate that we should flexibel and not bind to dovecot as mda. |
|
@understood. I will probably have a try with your official ldap implementation then. Thanks! |
|
@alinmear let us know for the integration test. |
|
@tomav just commented on #352. So you could close this PR. Integration tests are implemented by dennis with #352, where he picked up the commits from this PR and extended them. After merging i will rebase and will make a new PR for having the option to choose between kopano_dagent and dovecot on the virtual_transport parameter and between cyrus sasl and dovecot sasl. With the PR i will cover the integration tests for cyrus sasl and changing virtual_transport. |
5 participants
I am worked on a kopano docker integration and needed
I also updated the README.