Skip to content

Fix #526: fatal: no SASL authentication mechanisms#556

Merged
tomav merged 4 commits intodocker-mailserver:masterfrom
alinmear:#526
Apr 26, 2017
Merged

Fix #526: fatal: no SASL authentication mechanisms#556
tomav merged 4 commits intodocker-mailserver:masterfrom
alinmear:#526

Conversation

@alinmear
Copy link
Copy Markdown
Contributor

@alinmear alinmear commented Mar 19, 2017

When using the container with SMTP_ONLY = 1, the postfix process within the container fails
on ehlo because there is no valid sasl authentication mechanism available. This happens because sasl has been enabled within postfix/main.cf per default but sasl is not configured.

To fix this _setup_postfix_sasl does not depend anymore on ENABLE_SASLAUTHD and will check in it's logic, whether to enable or disable sasl explicit within postfix/main.cf.

@tomav
Copy link
Copy Markdown
Contributor

tomav commented Mar 25, 2017

I think you should add an integration test for that.

Copy link
Copy Markdown
Contributor

@tomav tomav left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Missing integration test

When using the container with SMTP_ONLY = 1, then the container fails
on ehlo because there is no valid sasl authentication mechanism
available. This happens because sasl has been enabled within
postfix/main.cf per default but sasl is not configured.

To fix this _setup_postfix_sasl does not depend anymore on
ENABLE_SASLAUTHD and will check in it's logic, whether to enable sasl
or not within postfix/main.cf.
When using the container with SMTP_ONLY = 1, then the container fails
on ehlo because there is no valid sasl authentication mechanism
available. This happens because sasl has been enabled within
postfix/main.cf per default but sasl is not configured.

To fix this _setup_postfix_sasl does not depend anymore on
ENABLE_SASLAUTHD and will check in it's logic, whether to enable sasl
or not within postfix/main.cf.
@alinmear
Copy link
Copy Markdown
Contributor Author

Added integration test

@alinmear
Copy link
Copy Markdown
Contributor Author

Just to be accurate on this issue, i just coverd the testing for this particular problem, checking whether sending an email is working or not. But i think we should cover more test cases for the mailserver state SMTP_ONLY use-cases.

Here some examples:

  • Check Mail send is working
  • Check SASL authentication is working
  • Check Relay is working
  • Check Relay with sasl authentication is working

Maybe we could create a section for this.

@tomav tomav merged commit 50ac2bd into docker-mailserver:master Apr 26, 2017
RichardFevrier pushed a commit to RichardFevrier/docker-mailserver that referenced this pull request Aug 26, 2019
…ocker-mailserver#556)

* Fix docker-mailserver#526: fatal: no SASL authentication mechanisms

When using the container with SMTP_ONLY = 1, then the container fails
on ehlo because there is no valid sasl authentication mechanism
available. This happens because sasl has been enabled within
postfix/main.cf per default but sasl is not configured.

To fix this _setup_postfix_sasl does not depend anymore on
ENABLE_SASLAUTHD and will check in it's logic, whether to enable sasl
or not within postfix/main.cf.

* Fix docker-mailserver#526: fatal: no SASL authentication mechanisms

When using the container with SMTP_ONLY = 1, then the container fails
on ehlo because there is no valid sasl authentication mechanism
available. This happens because sasl has been enabled within
postfix/main.cf per default but sasl is not configured.

To fix this _setup_postfix_sasl does not depend anymore on
ENABLE_SASLAUTHD and will check in it's logic, whether to enable sasl
or not within postfix/main.cf.

* Fix test
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.

2 participants