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

Expose "launch reason" or OOM state #866

Open
bolsinga opened this issue Oct 27, 2020 · 2 comments
Open

Expose "launch reason" or OOM state #866

bolsinga opened this issue Oct 27, 2020 · 2 comments
Labels
feature request Request for a new feature needs discussion Requires internal analysis/discussion

Comments

@bolsinga
Copy link

Description

It would be nice to know "why is the app launching?" Basically we'd like to know if we're re-launching due to a crash, or an OOM, etc. This will allow us to monitor what is going wrong more closely.

Describe the solution you'd like
Right now there is +[Bugsnag appDidCrashLastLaunch]. How about an enumeration about why the application launched? Something along the lines of https://eng.uber.com/startup-reason-reporter/

Describe alternatives you've considered
Right now we use private interfaces in Bugsnag to get -[BSGOutOfMemoryWatchdog didOOMLastLaunch] in v5, and -[BugsnagClient shouldReportOOM] in v6

Related requests with #570.

Additional context
We basically just want to monitor with our own services if Bugsnag believes an OOM has occurred.

@bolsinga bolsinga changed the title Expose "launch reason" Expose "launch reason" or OOM state Oct 27, 2020
@xljones xljones added feature request Request for a new feature needs discussion Requires internal analysis/discussion labels Oct 29, 2020
@xljones
Copy link
Contributor

xljones commented Oct 29, 2020

Hey @bolsinga, thanks for the suggestion! Would you expect this to look like a more nuanced version of appDidCrashLastLaunch()? Such that it would return something like: none, unhandledError, appUpdated, osUpdatedoom rather than just a BOOL?

@bolsinga
Copy link
Author

Yes, that is what I was thinking. The -appDidCrashLastLaunch method could remain for backwards compatibility. The enum could be returned for apps that may want to dig into more of the why. Thanks for your consideration.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature request Request for a new feature needs discussion Requires internal analysis/discussion
Projects
None yet
Development

No branches or pull requests

2 participants