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

fix: Send mail notifications via msmtp #84

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

gschafra
Copy link

- since Alpine's mail no longer supports `-S`
- fixes lobaro#82
@gschafra
Copy link
Author

Successfully tested it using a custom image build using this changes in one of our prod environments in my company

@sylarevan
Copy link

sylarevan commented Jan 20, 2024

Hi there. I also had the issue with emails not being sent after a backup. I can confirm the modifications in backup.sh, check.sh and entry.sh proposed in this PR are working when used with a correct msmprtc configuration file.
But the Dockerfile must obviously be modified accordingly to install the msmtp package and to "change" sendmail binary which is using busybox initially.
So, modifing the Dockerfile with :

RUN apk add --update --no-cache curl mailx msmtp

and adding

RUN ["ln", "-sf", "/usr/bin/msmtp", "/usr/sbin/sendmail"]

should also be part of this PR.

With this PR and the modifications above in the Dockerfile, emails are sent correctly.

@gschafra
Copy link
Author

@sylarevan Oh, thanks. Should be fixed now.

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.

notification mails not being sent
2 participants