Ensure that the account contains a @#924
Merged
johansmitsnl merged 3 commits intodocker-mailserver:masterfrom Apr 4, 2018
piwai:master
Merged
Ensure that the account contains a @#924johansmitsnl merged 3 commits intodocker-mailserver:masterfrom piwai:master
johansmitsnl merged 3 commits intodocker-mailserver:masterfrom
piwai:master
Conversation
Contributor
|
Could you add a test for this change? |
johansmitsnl
suggested changes
Apr 2, 2018
Contributor
johansmitsnl
left a comment
johansmitsnl
left a comment
There was a problem hiding this comment.
Could you add a test
Contributor
Author
|
@johansmitsnl Done :-) |
Contributor
Author
|
Hmm, there are failures but apparently unrelated to the test I added :/ |
johansmitsnl
approved these changes
Apr 3, 2018
Contributor
|
restarted the test build |
Contributor
|
@piwai Same test 130 failed. This might be related to the setup change you made. Could you verify this test? |
Contributor
johansmitsnl
added a commit
that referenced
this pull request
Apr 23, 2018
Release 6.0.0 * Implementation of multi-domain relay hosts (#922, #926) AWS_SES_HOST and AWS_SES_PORT are deprecated now. RELAY_HOST and RELAY_PORT are introduced to replace them. * Password creation fix (#908, #914) * Fixes 'duplicate log entry for /var/log/mail/mail.log' (#925, #927) * fixed cleanup (mail_with_relays didn't get cleaned up) (#930) * fix line breaks in postfix-summary mail error case (#936) * Set default virus delete time (#932, #935) This defaults to 7 days * Ensure that the account contains a @ (#923, #924) * Introducing global filters. (#934) * add missing env vars to docker-compose.yml (#937) * set postmaster address to a sensible default (#938, #939, #940)
piwai
added a commit
to piwai/docker-mailserver
that referenced
this pull request
May 18, 2024
…ailserver#924) * Ensure that the provided username actually contains a domain * Update README.md to be consistent with addmailuser script * Add a test to check if the username includes the domain
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Fixes #923