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

DLPX-91396 All emails generated by specific cron job, are redirected to the user SMTP. #485

Merged

Conversation

prakashsurya
Copy link
Contributor

@prakashsurya prakashsurya commented Jun 17, 2024

Problem

Emails generated by "cron" are being sent externally via SMTP.

Solution

Disable "nullmailer" service, so it doesn't send email's externally.

@prakashsurya prakashsurya force-pushed the dlpx/pr/prakashsurya/62b1dc37-e86b-4c98-8272-b117c2b897ba branch from 19c50aa to ea49176 Compare June 17, 2024 20:34
@prakashsurya prakashsurya marked this pull request as ready for review June 17, 2024 20:36
@prakashsurya prakashsurya force-pushed the dlpx/pr/prakashsurya/62b1dc37-e86b-4c98-8272-b117c2b897ba branch from ea49176 to caa8875 Compare June 17, 2024 20:45
@prakashsurya
Copy link
Contributor Author

Got this idea from: bruceg/nullmailer#64

@prakashsurya
Copy link
Contributor Author

Actually, looks like I might be able to just disable the service, which seems like a cleaner solution to me; updated the PR to reflect that.

@sebroy
Copy link
Contributor

sebroy commented Jun 17, 2024

Sounds good. It might also be worth looking into the cron configuration. Since we know sending email should never happen, cron may also have a configuration to suppress email. I think CRONDARGS can be set so that cron sends output to syslog instead of email?

@prakashsurya prakashsurya enabled auto-merge (squash) June 18, 2024 16:03
@prakashsurya prakashsurya disabled auto-merge June 18, 2024 16:04
@prakashsurya prakashsurya changed the base branch from develop to release June 18, 2024 16:04
@prakashsurya prakashsurya merged commit 79a4920 into release Jun 18, 2024
16 checks passed
@prakashsurya prakashsurya deleted the dlpx/pr/prakashsurya/62b1dc37-e86b-4c98-8272-b117c2b897ba branch June 18, 2024 17:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants