sentry-react-native icon indicating copy to clipboard operation
sentry-react-native copied to clipboard

Request to have a specific flag to disable upload debug symbol build phase

Open Lucienzera opened this issue 1 year ago • 3 comments

Specs

SDK Version: @sentry/react-native: 5.28.0 RN Version: 0.74.5 expo: 51.0.26

OS:

  • [ ] Windows
  • [x] MacOs
  • [ ] Linux

Platform:

  • [x] iOS
  • [ ] Android

Are you using Expo?

  • [x] Yes
  • [ ] No

Are you using sentry.io or on-premise?

  • [x] sentry.io (SaaS)
  • [ ] on-premise

Context

Our expo managed react native application consists of hardening process that is done after an .ipa is generated. We use the expo plugin to connect the application with Sentry. This also adds Upload Debug Symbol for Sentry step into the build phases.

The hardening process will make some changes to the Debug Information Files, so we are required to use upload this new one into Sentry Dashboard instead of the one uploaded during the build phase.


Current workaround

We are required to patch the package and add a Release check under sentry-dexcode-debug-files.sh to disable the build phase step.


Request

To have a specific flag to disable this step entirely, something like SENTRY_DISABLE_DIF_UPLOAD. Instead of using the general SENTRY_DISABLE_AUTO_UPLOAD.

Was the reasoning to coupled both sourcemap and DIF upload to a single flag, under the assumption that if a dev makes some changes towards it they would be changing both js and native bundles?

Lucienzera avatar Aug 29 '24 07:08 Lucienzera

Hi @Lucienzera, thank you for the request we will discuss this internally.

The SENTRY_DISABLE_AUTO_UPLOAD was added as a simple way for dev to build release versions locally without Sentry.

krystofwoldrich avatar Aug 30 '24 11:08 krystofwoldrich

@krystofwoldrich understandable, because we have another layer for obfuscation and hardening to our ipa. We require to only upload the files after the hardening process is done instead of during the build process.

Its not that we dont want Sentry disabled build release version local, we just want to move the upload process after everything is done instead of during.

Lucienzera avatar Sep 03 '24 07:09 Lucienzera

Thank you for the details @Lucienzera

We decided to add a separate env to only disable native debug files upload.

krystofwoldrich avatar Sep 03 '24 08:09 krystofwoldrich