Skip to content

Add ldap auth for postfix and dovecot#352

Merged
tomav merged 6 commits intodocker-mailserver:masterfrom
dsst95:master
Oct 30, 2016
Merged

Add ldap auth for postfix and dovecot#352
tomav merged 6 commits intodocker-mailserver:masterfrom
dsst95:master

Conversation

@dsst95
Copy link
Copy Markdown
Contributor

@dsst95 dsst95 commented Oct 12, 2016

No description provided.

@tomav
Copy link
Copy Markdown
Contributor

tomav commented Oct 13, 2016

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

@dsst95
Copy link
Copy Markdown
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
Copy Markdown
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
Copy Markdown
Contributor

tomav commented Oct 30, 2016

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

@alinmear
Copy link
Copy Markdown
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
Copy Markdown
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.

3 participants