Closed Bug 1551996 Opened 6 years ago Closed 5 years ago

Let's use raven instead of mail_admins()

Categories

(developer.mozilla.org Graveyard :: General, enhancement, P3)

Product:
developer.mozilla.org Graveyard
Component:
General
Type:
enhancement

Tracking

(Not tracked)

Status:
RESOLVED WONTFIX

People

(Reporter: peterbe, Unassigned)

References

Details

(Keywords: in-triage, Whiteboard: [specification][type:bug][points=2])

For example; https://github.com/mozilla/kuma/blob/f35cf3d8923a70dfd86aae638b2a954379dff822/kuma/wiki/tasks.py#L50-L52

I suspect it's too late to remove the except Exception as e: because it feels so ingrained in how kuma works.
But we shouldn't rely on emailing admins when exceptions happen. Instead we should use raven's captureException. After all, Sentry will email the relevant people but only once instead of an error on every single failing render.

Depends on: 1551997
Keywords: in-triage
Priority: -- → P3
Whiteboard: [specification][type:bug][points=2]
MDN Web Docs' bug reporting has now moved to GitHub. From now on, please file content bugs at https://github.com/mdn/sprints/issues/ and platform bugs at https://github.com/mdn/kuma/issues/.
Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → WONTFIX
Product: developer.mozilla.org → developer.mozilla.org Graveyard
You need to log in before you can comment on or make changes to this bug.