Closed Bug 1851676 Opened 3 months ago Closed 2 months ago

Add ability to initialize sentry on startup


(Fenix :: Crash Reporting, task, P1)



(firefox117 wontfix, firefox118 fixed, firefox119 fixed)

119 Branch
Tracking Status
firefox117 --- wontfix
firefox118 --- fixed
firefox119 --- fixed


(Reporter: boek, Assigned: boek)




(2 files)

Currently we're initializing Sentry before we send a crash report. This is causing sentry to think that every session is a crashed session which is causing issues with reporting. We want to be able to initialize on start so that Sentry will be able to send their session start ping to accurately measure crash free sessions.

Severity: -- → S2
Priority: -- → P1
Assignee: nobody → jboek

For added background: we are seeing only Release show up with a spike of "crash free sessions", but this seems to be case because we only send fatal crashes to Sentry when the Sentry SDK is initialized which starts a session at that point. So therefore almost all sessions turn into crashed sessions because we don't initialize the SDK if nothing is wrong.

See Also: → 1853059

Added bug 1835107 for traceability so we know what lead us down this path to decide to do this change.

Regressions: 1835107
Closed: 2 months ago
Flags: qe-verify+
Resolution: --- → FIXED
Target Milestone: --- → 119 Branch

Hello Jeff,

Is QA verification needed for this issue? If so, could you please provide us with some steps to reproduce (STR) or testing scenarios?

Thank you!

Flags: needinfo?(jboek)

Hey Adina,

No QA verification needed. We will monitor Sentry for success

Flags: needinfo?(jboek)

Jeff, should we uplift this Sentry fix to next week's 118 dot release?

Flags: qe-verify-
Flags: qe-verify+
Flags: needinfo?(jboek)

Comment on attachment 9356421 [details] [review]
[mozilla-mobile/firefox-android] Bug 1851676 - Adds ability to initialize Sentry on startup (backport #3502) (#3914)

Beta/Release Uplift Approval Request

  • User impact if declined: Sentry will continue to report every session as a crashed session for release.
  • Is this code covered by automated tests?: Yes
  • Has the fix been verified in Nightly?: Yes
  • Needs manual test from QE?: No
  • If yes, steps to reproduce:
  • List of other uplifts needed: None
  • Risk to taking this patch: Low
  • Why is the change risky/not risky? (and alternatives if risky): We delay the initialization of Sentry till after first paint. Patch has lived in Nightly and Beta for a couple of weeks.
  • String changes made/needed:
  • Is Android affected?: Yes
Flags: needinfo?(jboek)
Attachment #9356421 - Flags: approval-mozilla-release?
Comment on attachment 9356421 [details] [review]
[mozilla-mobile/firefox-android] Bug 1851676 - Adds ability to initialize Sentry on startup (backport #3502) (#3914)

Approved for our next 118 dot release, thanks.
Attachment #9356421 - Flags: approval-mozilla-release? → approval-mozilla-release+
You need to log in before you can comment on or make changes to this bug.