Skip to content

Make the required changes for relay support#83

Merged
tomav merged 4 commits intodocker-mailserver:masterfrom
squired:master
Mar 18, 2016
Merged

Make the required changes for relay support#83
tomav merged 4 commits intodocker-mailserver:masterfrom
squired:master

Conversation

@robertdxyz
Copy link
Copy Markdown
Contributor

Now you can configure AWS SES as relay for postfix.

Comment thread start-mailserver.sh

if [ -f /tmp/postfix/main.cf ]; then
while read line; do
postconf -e "$line"
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Could you add an integration test on this, like overriding a default value.
Add a test/main.cf with one override. Do not forget to update make clean task in the Makefile.

@tomav
Copy link
Copy Markdown
Contributor

tomav commented Feb 20, 2016

Thank for this @robertdolca! 👍
Let me know if I can help regarding comments left above.

@tomav
Copy link
Copy Markdown
Contributor

tomav commented Feb 24, 2016

@robertdolca how are you? Did you have time to work on comments above?
Let me know if I can help.
Thank you.

@robertdxyz
Copy link
Copy Markdown
Contributor Author

Your comment seem justified. I didn't have time to implement the changes yet.
Thanks @tomav

@tomav tomav added the wip label Feb 25, 2016
@tomav
Copy link
Copy Markdown
Contributor

tomav commented Mar 4, 2016

@robertdolca still alive? Could you let us know what's going on. Thank you.

@robertdxyz
Copy link
Copy Markdown
Contributor Author

Just a very busy schedule. If you have time and you are wolling to you can make the proposed changes and I will take a look. If not al will make them asap.

@tomav
Copy link
Copy Markdown
Contributor

tomav commented Mar 4, 2016

If I have time this week end, I'll let you know here before to work on it.

@tomav tomav merged commit d36ecaa into docker-mailserver:master Mar 18, 2016
@youmoula youmoula mentioned this pull request Oct 28, 2017
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