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

fix(android): Parse java exception name when message missing #159

Merged
merged 1 commit into from
Jul 3, 2019

Conversation

kattrali
Copy link
Contributor

@kattrali kattrali commented Jul 1, 2019

Not all Java exceptions have messages, so the default should be to set the class and java exception parsing style if the Android error class is detected.

Depends on bugsnag/bugsnag-android#511

@kattrali kattrali force-pushed the kattrali/read-android-java-class-without-message branch from f82a87d to 3b080d0 Compare July 2, 2019 13:53
@kattrali kattrali force-pushed the kattrali/read-android-java-class-without-message branch from 3b080d0 to a1578f0 Compare July 3, 2019 09:10
Not all Java exceptions have messages, so the default should be to set
the class and java exception parsing style if the error class is
detected.
@kattrali kattrali force-pushed the kattrali/read-android-java-class-without-message branch from a1578f0 to 518e2e9 Compare July 3, 2019 09:31
@kattrali
Copy link
Contributor Author

kattrali commented Jul 3, 2019

Updated with some additional comments around how Android Java exceptions are passed through the Unity log handler.

Copy link
Contributor

@tomlongridge tomlongridge left a comment

Choose a reason for hiding this comment

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

Thanks @kattrali

@kattrali kattrali merged commit 1f12c54 into master Jul 3, 2019
@kattrali kattrali deleted the kattrali/read-android-java-class-without-message branch July 3, 2019 10:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants