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.

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

Dockerfile: preinstall msmtp #10

Merged
merged 6 commits into from
Mar 12, 2024
Merged

Conversation

schildbach
Copy link
Contributor

This implements /usr/sbin/sendmail which is needed by FailureReports true.

Also adds a new section about sending reports to the README.md.

Note that "aggregate reports" are outside the scope of my testing and this PR. FailureReports true generates reports immediately for individual emails.

I sadly didn't find a way around specifying the from address a second time in /etc/msmtprc, even though it's already known via FailureReportsSentBy OpenDMARC configuration option.

Resolves #9

@schildbach
Copy link
Contributor Author

Ah, and I should add I've based most of my testing on the Debian image. However, since msmtp versions only differ minimally (Alpine: 1.8.25 vs Debian: 1.8.23) I expect the same outcome.

alpine/Dockerfile Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
This implements `/usr/sbin/sendmail` which is needed by `FailureReports true`.

Also adds a test and a new section about sending reports to the README.md.
@tyranron tyranron merged commit fe6183c into instrumentisto:main Mar 12, 2024
6 checks passed
@tyranron
Copy link
Member

@schildbach released as 1.4.2-r36.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

preinstall msmtp
2 participants