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.

Sign up for GitHub

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

Jump to bottom

provide SMTP email backend with Sentry capture #6480

Merged
merged 1 commit into from
Feb 3, 2025
Merged

provide SMTP email backend with Sentry capture #6480

merged 1 commit into from
Feb 3, 2025
+36 −0

Conversation

escattone
Copy link
Contributor

@escattone escattone commented Jan 30, 2025 •
edited
Loading

Helps with mozilla/sumo#1993

This is intended for use in prod only -- since dev and stage use the django-email-bandit backend -- by changing the EMAIL_LOGGING_REAL_BACKEND setting from "django.core.mail.backends.smtp.EmailBackend" to "kitsune.lib.email.SMTPEmailBackendWithSentryCapture" in K8s.

See companions PR: https://github.com/mozilla-it/webservices-infra/pull/3900

smithellis
smithellis approved these changes Jan 31, 2025
View reviewed changes
Copy link
Contributor

@smithellis smithellis left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This will definitely help us monitor and manage email issues. I don't think this can be tested great locally, but the logic here seems solid.

@escattone escattone merged commit 3be4dbc into mozilla:main Feb 3, 2025
2 checks passed
@escattone escattone deleted the log-failed-email-sends branch February 3, 2025 16:20
@escattone escattone mentioned this pull request Feb 3, 2025
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@akatsoulas akatsoulas akatsoulas approved these changes

@smithellis smithellis smithellis approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@escattone @akatsoulas @smithellis