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(update-ngxblocker): use correct unix mail return address flag #602

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

bfahrenfort
Copy link

@bfahrenfort bfahrenfort commented Dec 16, 2024

update-ngxblocker uses -f to specify the return address. GNU mail.mailutils uses -r to specify the return address.

previously, update-ngxblocker would report:

Emailing report to: <snip>

mail: conflicting options

The second line is gone with this change, and /var/log/mail.log reports (250 Ok).

You still specify the right return address with the same command, update-ngxblocker -e <notification email address> -f <send address> [other mailgun args, if using mailgun].

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.

1 participant