Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

fix opendkim permissions in docker-compose setup #2868

Merged
merged 2 commits into from
Jun 12, 2020
Merged

fix opendkim permissions in docker-compose setup #2868

merged 2 commits into from
Jun 12, 2020

Conversation

kimsible
Copy link
Contributor

@kimsible kimsible commented Jun 11, 2020 •
edited by rigelk
Loading

@rigelk rigelk linked an issue Jun 11, 2020 that may be closed by this pull request
Mail send fails with docker postfix #2867
Closed
@rigelk rigelk added the Component: Docker 🐳 Deals with containerisation, a hellish nightmare for Chocobos label Jun 11, 2020
@Chocobozzz
Copy link
Owner

Thanks. Could you add a comment with a link to the postfix-relay PR? So we understand the purpose of this setting

@kimsible
Copy link
Contributor Author

Thanks. Could you add a comment with a link to the postfix-relay PR? So we understand the purpose of this setting

The link to the postfix-relay is in the description

@Chocobozzz
Copy link
Owner

Chocobozzz commented Jun 12, 2020 •
edited
Loading

The link to the postfix-relay is in the description

I meant a comment in the code, below the OPENDKIM_RequireSafeKeys=no line

@rigelk rigelk changed the title Fix opendkim permissions docker-compose setup fix opendkim permissions in docker-compose setup Jun 12, 2020
@rigelk rigelk merged commit 0283fe9 into Chocobozzz:develop Jun 12, 2020
@rigelk
Copy link
Collaborator

rigelk commented Jun 12, 2020

Thanks @kimsible ; I added the comment, so nothing else to do 🙂

@kimsible
Copy link
Contributor Author

Thanks. Could you add a comment with a link to the postfix-relay PR? So we understand the purpose of this setting

Ok, sorry @Chocobozzz and thanks @rigelk =)

@wader
Copy link

wader commented Jun 14, 2020

Hello, wader/postfix-relay#18 has been merged now. Let me know if there are issues!

@Chocobozzz
Copy link
Owner

Thanks @wader!

@kimsible
Copy link
Contributor Author

The previous issue seems to be fixed with the default docker image =).
Thanks @wader !

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@rigelk rigelk rigelk left review comments

Assignees
No one assigned
Labels
Component: Docker 🐳 Deals with containerisation, a hellish nightmare for Chocobos
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Mail send fails with docker postfix
4 participants
@kimsible @Chocobozzz @rigelk @wader