sentry-java icon indicating copy to clipboard operation
sentry-java copied to clipboard

Drop Error captured by sentry.native.android, as default behavior

Open thinkocapo opened this issue 11 months ago • 9 comments

Problem Statement

Steps to Repro

  1. make an app with https://github.com/Unity-Technologies/uaal-example/blob/master/docs/android.md. https://docs.unity3d.com/Manual/UnityasaLibrary.html.
  2. sentry demo app made via... https://github.com/bitsandfoxes/uaal_wip
  3. error when user leaves the Unity game
override fun onDestroy(owner: LifecycleOwner) {
  unityPlayer.quit()
  super.onDestroy(owner)
}

Solution Brainstorm

sdk no capture this as an error

thinkocapo avatar Dec 16 '24 19:12 thinkocapo

@bitsandfoxes @romtsn is this something you guys could add to that UaaL repro you built?

bruno-garcia avatar Dec 16 '24 19:12 bruno-garcia

is this something you guys could add to that UaaL repro you built?

Sure! I'll update here.

bitsandfoxes avatar Dec 16 '24 19:12 bitsandfoxes

Do I read that right, the issue is about dropping all captured native events during shutdown?

bitsandfoxes avatar Dec 16 '24 19:12 bitsandfoxes

Do I read that right, the issue is about dropping all captured native events during shutdown?

My understanding was that when the customer closed the Unity app, the native (Android) SDK captured an event that was not actionable. They indicated as if exiting Unity raised a signal that our SDK picked up. But they see that as "business as usual"

bruno-garcia avatar Dec 16 '24 22:12 bruno-garcia

is this something you guys could add to that UaaL repro you built?

Sure! I'll update here.

Any luck with repro'ing this so we can decide next steps?

bruno-garcia avatar Feb 02 '25 02:02 bruno-garcia

@bitsandfoxes is this something you want to take a look at?

markushi avatar Feb 05 '25 14:02 markushi

Do I read that right, the issue is about dropping all captured native events during shutdown?

@thinkocapo could you please confirm the behavior intended?

@bitsandfoxes can we reproduce this on the UaaL example we have? If so, what's the complexity of filtering out this event?

Internal customer case: https://www.notion.so/sentry/15e8b10e4b5d80a8bac6ff132144b8f8

bruno-garcia avatar Apr 09 '25 20:04 bruno-garcia

@bruno-garcia yes that's correct, they want to drop all captured native events that come from the shutdown code they provided. I updated the customer case with a screenshot of a particular one they ran into recently, as well as the issue link to it.

thinkocapo avatar Apr 09 '25 20:04 thinkocapo

We're going to pick this back up on the way of supporting UaaL properly as part of https://github.com/getsentry/sentry-unity/issues/1727

bitsandfoxes avatar Jul 04 '25 08:07 bitsandfoxes