Skip to content

fix: Android - make sure logcat.log is attached to unhandled event (SIGSEGV Segfault)#3694

Merged
jamescrosswell merged 46 commits intomainfrom
fix/#3461
Dec 18, 2024
Merged

fix: Android - make sure logcat.log is attached to unhandled event (SIGSEGV Segfault)#3694
jamescrosswell merged 46 commits intomainfrom
fix/#3461

Conversation

@bricefriha
Copy link
Contributor

@bricefriha bricefriha commented Oct 18, 2024

From sentry-native#1027, we had to add the handler strategy to the .NET SDK, from the sentry-java (7.15.0-alpha1).

This is also meant to fix #3461. I tested this, and it seems to be fixed now.

More context at:

@bricefriha bricefriha changed the title Android - add handler strategy Android - implement handler strategy Oct 18, 2024
@bricefriha
Copy link
Contributor Author

We'd like to wait for their release though

The release is out now!
https://github.com/getsentry/sentry-native/releases/tag/0.7.13

We just need an update from sentry-java

@jamescrosswell
Copy link
Collaborator

The release is out now! https://github.com/getsentry/sentry-native/releases/tag/0.7.13

We just need an update from sentry-java

@markushi any timeframe on getting the fix we tested in 7.15.0-alpha.1 into an official release? I'd love to roll this out in the .NET SDK but we've got a bit of a dependency chain.

Looks like we'll have to do something similar for iOS as well, and it would be nice to reference this solution when discussing with the cocoa team.

@markushi
Copy link
Member

markushi commented Dec 5, 2024

The release is out now! getsentry/sentry-native@0.7.13 (release)
We just need an update from sentry-java

@markushi any timeframe on getting the fix we tested in 7.15.0-alpha.1 into an official release? I'd love to roll this out in the .NET SDK but we've got a bit of a dependency chain.

Looks like we'll have to do something similar for iOS as well, and it would be nice to reference this solution when discussing with the cocoa team.

If it's tested and we know the change works, we can merge and get this released at any time. Let me get the PR up-to-date first.

@jamescrosswell
Copy link
Collaborator

Looks like the fix was included with the 7.19.0 release of the Java SDK. We just need to bump the version in this PR and we should be good to go!

@jamescrosswell jamescrosswell self-requested a review December 17, 2024 21:04
@jamescrosswell jamescrosswell merged commit 1ecab82 into main Dec 18, 2024
20 checks passed
@jamescrosswell jamescrosswell deleted the fix/#3461 branch December 18, 2024 09:18
jpnurmi added a commit that referenced this pull request Oct 21, 2025
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.

logcat.log is not attached to unhandled event (SIGSEGV Segfault)

5 participants