Skip to content

Avoid fixing permissions; add regexp alias file, add AWS SES outgoing email support#194

Merged
tomav merged 15 commits intodocker-mailserver:masterfrom
tve:master
May 24, 2016
Merged

Avoid fixing permissions; add regexp alias file, add AWS SES outgoing email support#194
tomav merged 15 commits intodocker-mailserver:masterfrom
tve:master

Conversation

@tve
Copy link
Copy Markdown
Contributor

@tve tve commented May 23, 2016

This PR is not intended to be merged as-is, please indicate whether you want all/some/none of it and I can cherry-pick accordingly. It has:

I added docs to the README, but I would move that to the wiki.

Comment thread README.md Outdated
This is a fork of https://github.com/tomav/docker-mailserver with some additional features
described below.

### Sending outbound mail via Amazon SES
Copy link
Copy Markdown
Contributor

@tomav tomav May 23, 2016

Choose a reason for hiding this comment

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

Could you move this doc to the Wiki? Simpler to maintain.
1 page for AWS, and add regexp aliases to the existing page.

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.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

That was the plan :-)


echo "Fixing permissions"
chown -R 5000:5000 /var/mail
# Fix permissions, but skip this if 3 levels deep the user id is already set
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.

There's a pending PR from @Josef-Friedrich #193
You should discuss the implementation to only keep one.
Thanks.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Started...

@tomav
Copy link
Copy Markdown
Contributor

tomav commented May 23, 2016

Thanks for the work @tve
Let's check these little things and I'll merge.

@tve
Copy link
Copy Markdown
Contributor Author

tve commented May 24, 2016

This PR is ready from my point of view. Please let me know if there's anything else you would like changed.

@tomav tomav added can be merged and removed wip labels May 24, 2016
@tomav tomav merged commit e2409ed into docker-mailserver:master May 24, 2016
@tomav
Copy link
Copy Markdown
Contributor

tomav commented May 24, 2016

Thank you @tve, really nice work and wiki documentation.

RichardFevrier pushed a commit to RichardFevrier/docker-mailserver that referenced this pull request Aug 26, 2019
Avoid fixing permissions; add regexp alias file, add AWS SES outgoing email support
polarathene pushed a commit to polarathene/docker-mailserver that referenced this pull request Feb 12, 2021
polarathene pushed a commit to polarathene/docker-mailserver that referenced this pull request Feb 12, 2021
polarathene pushed a commit to polarathene/docker-mailserver that referenced this pull request Feb 22, 2021
polarathene pushed a commit to polarathene/docker-mailserver that referenced this pull request Feb 22, 2021
polarathene pushed a commit to polarathene/docker-mailserver that referenced this pull request Feb 26, 2021
polarathene pushed a commit to polarathene/docker-mailserver that referenced this pull request Feb 26, 2021
@polarathene polarathene mentioned this pull request Jan 21, 2024
4 tasks
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