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

Obfuscated built Flutter app obfuscates Sentry message #1663

Closed
b4tchkn opened this issue Sep 28, 2023 · 6 comments
Closed

Obfuscated built Flutter app obfuscates Sentry message #1663

b4tchkn opened this issue Sep 28, 2023 · 6 comments
Assignees

Comments

@b4tchkn
Copy link

b4tchkn commented Sep 28, 2023

Environment

SaaS (https://sentry.io/)

Steps to Reproduce

  1. Obfuscate Flutter apps with using --obfuscate --split-debug-info=./ options.
  2. Upload symbol files to Sentry using sentry-cli debug-files upload -o <org> -p <project> ./ on CI.
  3. Send class runtimeType or StackTrace.current() to SentryEvent message in Flutter.
  4. Check Issues on Sentry.

Expected Result

To be confirmed messages that have not been obfuscated.

Actual Result

The message arrives on Sentry obfuscated.

So, If I don't set stackTrace when call Sentry.captureEvent(), StackTrace on Sentry will be unknown. before Flutter obfuscation, I could see a normal StackTrace.
If I set stackTrace to StackTrace.current, etc. when call Sentry.captureEvent(), I can confirm a normal StackTrace.

Is it a specification that obfuscation occurs when source code objects are included in the message and that StackTrace is obfuscated when StackTrace is not specified?

Product Area

Issues

Link

No response

DSN

No response

Version

No response

@getsantry
Copy link

getsantry bot commented Sep 28, 2023

Assigning to @getsentry/support for routing ⏲️

@getsantry
Copy link

getsantry bot commented Sep 28, 2023

Routing to @getsentry/product-owners-issues for triage ⏲️

@kahest kahest transferred this issue from getsentry/sentry Oct 2, 2023
@github-project-automation github-project-automation bot moved this to Needs Discussion in Mobile & Cross Platform SDK Oct 2, 2023
@getsantry getsantry bot moved this to Waiting for: Product Owner in GitHub Issues with 👀 Oct 2, 2023
@stefanosiano
Copy link
Member

hey @b4tchkn Could you please provide us a link to the event?
If you don't want to provide a link on this github issue, you can send it via email to stefano.siano@sentry.io
Also, are you experiencing it on a specific platform, like web?

@b4tchkn
Copy link
Author

b4tchkn commented Oct 5, 2023

@stefanosiano
This is occurring on iOS and Android.
And sent an email with a link to the Sentry event. Please check it out.

@stefanosiano
Copy link
Member

hey @b4tchkn
I had a look at the event, and it seems like it's related to #1565 and getsentry/sentry#48334
I'll have a closer look and come later. As pointed in getsentry/sentry#48334 (comment), though, it's quite a big scope

@getsantry getsantry bot removed the status in GitHub Issues with 👀 Oct 5, 2023
@stefanosiano stefanosiano moved this from Needs Discussion to Needs Investigation in Mobile & Cross Platform SDK Oct 12, 2023
@stefanosiano stefanosiano self-assigned this Oct 12, 2023
@stefanosiano
Copy link
Member

hey @b4tchkn
after merging #1756, the stacktraces of the events captured with captureMessage or captureEvent will be deobfuscated. Note that you either have to put your own stacktrace or use the await keyword when calling Sentry APIs to get the calling function in the stacktrace, due to technical limitations.
Regarding deobsfuscation of messages, that's a bigger topic that we are discussing internally.

I'm closing this issue for now. Feel free to add a comment in case the fix doesn't work for your case

@github-project-automation github-project-automation bot moved this from Needs Investigation to Done in Mobile & Cross Platform SDK Nov 24, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Archived in project
Archived in project
Development

No branches or pull requests

3 participants