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

Add ldap auth for postfix and dovecot #352

Merged
merged 6 commits into from
Oct 30, 2016
Merged

Add ldap auth for postfix and dovecot #352

merged 6 commits into from
Oct 30, 2016

Conversation

dsst95
Copy link
Contributor

@dsst95 dsst95 commented Oct 12, 2016

No description provided.

@tomav
Copy link
Contributor

tomav commented Oct 13, 2016

Hi @dennis95stumm, what do we do with #335 ?

@dsst95
Copy link
Contributor Author

dsst95 commented Oct 13, 2016

My opinion is that there are tests missing. @alinmear could rebase on develop after merging this pull request and add a pull request for kopano support. But lets ask also @alinmear.

@alinmear
Copy link
Contributor

Hey @dennis95stumm sounds good. i will rebase and make a new pull request as proposed!

@dsst95 dsst95 closed this Oct 26, 2016
@dsst95 dsst95 reopened this Oct 26, 2016
@tomav
Copy link
Contributor

tomav commented Oct 30, 2016

Waiting for @alinmear to respond on #335. Otherwise this PR is ready to merge.

@alinmear
Copy link
Contributor

alinmear commented Oct 30, 2016

@tomav @dennis95stumm picked up my changes and extended them, also with several integration tests. I will rebase on this and make a pull request for kopano support (options for cyrus sasl instead of dovecot sasl and changing the virtual_transport parameter)

After merging this pull request i will rebase and make a new pull request. So you could just close #335.

@tomav
Copy link
Contributor

tomav commented Oct 30, 2016

Roger That.

RichardFevrier pushed a commit to RichardFevrier/docker-mailserver that referenced this pull request Aug 26, 2019
* Add ldap support for postfix and dovecot

* Add SASLAUTHD

* Update README.md

* Add necessary packages to dockerfile

* Add config files for ldap

* Add tests for ldap auth
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 this pull request may close these issues.

None yet

3 participants