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

iOS source map only works with force-foreground flag #3456

Closed
5 of 11 tasks
maykonmichel opened this issue Dec 8, 2023 · 4 comments
Closed
5 of 11 tasks

iOS source map only works with force-foreground flag #3456

maykonmichel opened this issue Dec 8, 2023 · 4 comments

Comments

@maykonmichel
Copy link

OS:

  • Windows
  • MacOS
  • Linux

Platform:

  • iOS
  • Android

SDK:

  • @sentry/react-native (>= 1.0.0)
  • react-native-sentry (<= 0.43.2)

SDK version: 5.15.1

react-native version: 0.73.0

Are you using Expo?

  • Yes
  • No

Are you using sentry.io or on-premise?

  • sentry.io (SaaS)
  • on-premise

If you are using sentry.io, please post a link to your issue so we can take a look:

Unminified code issue
Issue with workaround

Configuration:

(@sentry/react-native)

Sentry.init({
  dsn: 'https://241a4d607de84f2b3aff4b18f06c88a6@o4506359951392768.ingest.sentry.io/4506359953162240'
});

I have the following issue:

When a crash happens on iOS, Sentry doesn't capture sourcemap unless I set the force foreground flag adding export SENTRY_CLI_EXTRA_ARGS="--force-foreground" at .xcode.env

Steps to reproduce:

  • Initialize a new project
  • Run sentry wizard and setup app
  • Build a release app and test the crash

Reproducer repo

Actual result:

Sentry issues dashboard shows the crash, but with unminified code

Expected result:

Sentry issues dashboard shows the crash with unminified code

@krystofwoldrich
Copy link
Member

Hi,
thank you for the message and the reproducible example. I'm going to clone the project and try it. I'll add a comment here with the result.

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

I reproduced the issue, thank you for the repository, we are still investigation why is it happening.

@krystofwoldrich krystofwoldrich moved this from Needs Investigation to Needs Discussion in Mobile & Cross Platform SDK Jan 12, 2024
@krystofwoldrich krystofwoldrich moved this from Needs Discussion to Todo in Mobile & Cross Platform SDK Jan 12, 2024
@krystofwoldrich
Copy link
Member

Based on our investigation the background upload is working in https://github.com/getsentry/sentry-cli/releases/tag/2.21.3. You can manually downgrade to that version of Sentry CLI until we fix the bug.

@krystofwoldrich
Copy link
Member

Please follow the sentry-cli issue for more details.

@krystofwoldrich krystofwoldrich closed this as not planned Won't fix, can't repro, duplicate, stale Jan 16, 2024
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

2 participants