-
-
Notifications
You must be signed in to change notification settings - Fork 323
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
android/ios sourcemap uploads not working with RN 0.74 #3817
Comments
Hi @jpaas, On Android the On iOS the For Android do you change any React Native settings in For iOS could you share with us your Have the source map upload work for you before upgrade to the latest RN version? |
Hi @krystofwoldrich It basically just has me upgrading the targets and tool versions, adding the kotlin-gradle-plugin, removing flipper. On iOS I think my I changed it to this and it now works. BTW I think the sentry docs at https://docs.sentry.io/platforms/react-native/manual-setup/manual-setup/#using-node-with-nvm-or-volta are wrong because if I don't define
On android is there some step I'm missing to compose the source maps? The bundling and uploading of sourcemaps all happens inside the gradle bundleRelease task, so I'm not sure how to insert it. I assumed the sentry plugin would handle it. I was just trying to follow the sentry docs at https://docs.sentry.io/platforms/react-native/manual-setup/manual-setup One thing I did just discover is new that I was missing is the custom metro serializer, so I added that, but it didn't change anything. |
Hi @jpaas, For Android, the compose source maps are handled by RN, https://github.com/facebook/react-native/blob/31f9d45074f069151c155836335e0edff87f70bd/packages/react-native-gradle-plugin/src/main/kotlin/com/facebook/react/tasks/BundleHermesCTask.kt#L102 Do you change any React Native settings in app/build.gradle, for example Hermes Compiler flags? You can check if the Hermes compiler source map is generated the path should be
|
re: AndroidI'm not setting any hermes compile flags. I don't see a
|
Although I just noticed in my iOS logs it has the same complaint about the debugID, but it continues to upload the sourcemap. |
Arg. The perils of a monorepo setup. So it turns out that every other react native command who's location can be configured - e.g. reactNativeDir, codegenDir, cliFile, are all relative to the Before I configured it it told me that it couldn't find the command. Once I configured it relative to the wrong dir I got a different error Sorry for the bother. |
OS:
Platform:
SDK:
@sentry/react-native
(>= 1.0.0)react-native-sentry
(<= 0.43.2)SDK version: 5.2.22
react-native
version: 0.74.1Are you using Expo?
Are you using sentry.io or on-premise?
If you are using sentry.io, please post a link to your issue so we can take a look:
N/A
Configuration:
N/A
I have the following issue:
When I try to build an android or ios release build, the sentry integration attempts to upload the source maps, but has difficulty finding them.
Steps to reproduce:
Actual result:
android: Sentry is looking for the sourcemap at
build/generated/sourcemaps/react/release/index.android.bundle.map
when it was generated atbuild/intermediates/sourcemaps/react/release/index.android.bundle.packager.map
ios: Sentry is looking for main.jsbundle.map in the wrong place. It does not exist in
Release-iphoneos
. It does exist inWorkerNativeApp.app
.Expected result:
It finds the sourcemaps and uploads them.
The text was updated successfully, but these errors were encountered: