-
-
Notifications
You must be signed in to change notification settings - Fork 830
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.
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
Postsrsd to avoid SPF rejects when Mailu used as MTA #1042
Conversation
Thanks for submitting this pull request. bors try Note: if this build fails, read this. |
tryBuild failed |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Some suggested changes, but I'll make a more general comment tonight when I find the time.
@@ -112,4 +112,7 @@ milter_default_action = tempfail | |||
############### | |||
# Extra Settings | |||
############### | |||
|
|||
sender_canonical_maps = tcp:localhost:10001 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I would put these under a previous section specific to SRS instead of just extra settings.
@@ -50,7 +51,9 @@ def start_podop(): | |||
|
|||
convert("/conf/rsyslog.conf", "/etc/rsyslog.conf") | |||
|
|||
# Run Podop and Postfix | |||
# Generate SRS Secret file, start SRS, run Podop and Postfix |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I feel bad about this. Maybe we could use native python (os.urandom and files)?
This PR should resolve #328
It implements Postsrsd with Mailu Postfix so that emails from alias are not rejected under SPF protocol.