-
Notifications
You must be signed in to change notification settings - Fork 87
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
Enable PostSRSd to avoid being marked as spam by failed SPF checks. #103
Merged
Conversation
This file contains 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
Use alpine:latest as base image
* Update base image to Alpine 3.8 * Install s6 process manager directly * Upgraded BATS to 1.1.0 * Install syslog-ng for postfix logging to stdout
These spam the console too much, so disable them.
Update Base to Alpine 3.8
couse => course
Fix typo in README
* Fix nickname typo * Add support for EC keys
Add custom timezone support
…nfiguration (huan#98) * Add an option to override postfix's default logging configuration * Add tests * Update README.md * push to build * fix overwriting variable * update with postfix-configuration * delete drone for pull request Co-authored-by: Tamaro Skaljic <[email protected]>
…an#97) * Add an option to override postfix's default logging configuration * Add tests * Update README.md
* fix default postfix logging configuration test * Change logfile path in custom postfix logging configuration test * Make postfix logging configuration tests restore the preconditions
…warder into PostSRSd Changed wrong Email :/
huan
approved these changes
Jun 18, 2021
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.
Thank you very much for your contribution!
It seems that this PR is good to me, however, could you please try to invite more contributors to approve this PR so that we can merge it with enough confidence?
dimitrovs
approved these changes
Jul 5, 2021
@dimitrovs Thank you very much for the approval! |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Forwarding to a gmail account never worked for me. The problem was that the SPF check always failed. However, the problem was not my SPF entry, but that of the sender.
After some research I found the solution: SRS (Sender Rewriting Scheme). The sender is replaced by a temporary sender from my domain and the SPF check passes.
The required daemon was already installed in the container, but unfortunately not activated. With a few lines of code the daemon can be activated, a security key generated and Postfix configured so that now everything works.
My mails now no longer end up as spam \o/.
I ask you to consider the changes and optionally enable SRS in the original branch.
Thank you for your time
SRS