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
Collect crashes with stacktraces #259
Conversation
Remove RollbarPLCrashReporter library.
Simplify reporting
Remove faulty Out of Memory detection
Collect and report all types of crashes together with full stack trace information
Provide useful error descriptions on all use cases that trigger crash reporting.
Validate crash reports before diagnosis, propagating errors. Simplified validation and error propagation. Unit tests.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nothing looked out of the ordinary to me!
# s.license = "MIT (example)" | ||
s.resource = "rollbar-logo.png" | ||
s.authors = { "Andrey Kornich (Wide Spectrum Computing LLC)" => "akornich@gmail.com", | ||
"Rollbar" => "support@rollbar.com" } |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Gotta give yourself some credit :P
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Meh. My commits have my name.
This should be changed to just Rollbar, though methinks.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yea I'd agree with that sentiment
"\nApplication Specific Information:" | ||
"*** Terminating app due to uncaught exception '" *? exception?.name *? "' , reason: '" *? exception?.reason *? "'" | ||
|
||
if report.exceptionType == .zombie { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🧟
// "rdi", "rsi", "rbp", "rsp", | ||
// "r8", "r9", "r10", "r11", "r12", "r13", "r14", "r15", | ||
// "rip", "rfl", "cr2" | ||
//] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🤔
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I wanted to keep these regs for now.
Description of the change
This is a collection of all previously approved PRs, not intended for review.
Each commit has a link to its corresponding approved PR.
Type of change