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

RAM bundles symbolication

Open marandaneto opened this issue 3 years ago • 2 comments

Problem: Some events don't get every frame symbolicated, but only a subset of it, not for every event. Repro: Not using Hermes Using RAM bundles with default parameter only bundleCommand: "ram-bundle" Works fine if not using RAM bundles Not using indexed RAM bundle Using this cmd to output the sourcemaps:

extraPackagerArgs: [
    "--sourcemap-output", file("$buildDir/sourcemap.js")
  ]

Using bundleConfig:"packager/config.js", with an unknown configuration.

https://docs.sentry.io/platforms/react-native/manual-setup/ram-bundles/ https://blog.sentry.io/2019/05/21/building-a-sentry-ram-bundles https://github.com/getsentry/rust-sourcemap/blob/9140bdfea1a304992b3733c772c16039bbd1327d/src/ram_bundle.rs

Frames that were symbolicated are coming from .js files, and those that are broken are coming from app:///index.android.bundle. I'd guess that this is the culprit somehow

It feels like there’s a case that’s not properly covered by our RAM bundle processing code

sentry-cli unpacks the sourcemap.js into multiple sourcemaps files https://github.com/getsentry/sentry-cli/blob/07f5b381efcc6bb66bca809d8fb749e54f4bac33/src/commands/sourcemaps/upload.rs#L204-L211

marandaneto avatar Jun 22 '22 11:06 marandaneto

Compare result with https://github.com/facebook/metro/tree/main/packages/metro-symbolicate

marandaneto avatar Jun 22 '22 12:06 marandaneto

We need the original index.android.bundle.map file before uploading to Sentry, so we can debug the whole process, from breaking up the single maps into multiple maps and the symbolication on the server.

marandaneto avatar Jun 22 '22 12:06 marandaneto

Closing the issue as a part of large repository cleanup, due to it being inactive and/or outdated. Please do not hesitate to ping me if it is still relevant, and I will happily reopen and work on it. Cheers!

marandaneto avatar Sep 27 '22 09:09 marandaneto