question-mark
Stuck on an issue?

Lightrun Answers was designed to reduce the constant googling that comes with debugging 3rd party libraries. It collects links to all the places you might be looking at while hunting down a tough bug.

And, if you’re still stuck at the end, we’re happy to hop on a call to see how we can help out.

Add option to set dist on sourcemaps

See original GitHub issue

I’m using the Sentry Release action to create a release of my React Native app and part of that is uploading sourcemaps. Here’s how I’m doing it:

- name: Create Sentry release
  uses: getsentry/action-release@v1
  env:
    SENTRY_AUTH_TOKEN: ${{ secrets.SENTRY_AUTH_TOKEN }}
    SENTRY_ORG: ${{ secrets.SENTRY_ORG }}
    SENTRY_PROJECT:  ${{ secrets.SENTRY_PROJECT }}
    VERSION_CODE: ${{ steps.get_version.outputs.VERSION_CODE }}
    VERSION_NAME: ${{ steps.get_version.outputs.VERSION_NAME }}
  with:
    environment: production
    version: com.a494studios.koreanconjugator@${{ env.VERSION_NAME }}+${{ env.VERSION_CODE }}
    url_prefix: app:///
    sourcemaps: android/app/build/generated/assets/react/release/index.android.bundle android/app/build/generated/sourcemaps/react/release/index.android.bundle.map

The problem is that the sourcemaps aren’t working correctly because they’re not being assigned a dist value. With the CLI I would use the --dist flag, but I don’t see an option for that in the docs or in the source code. Is there any interest in adding a dist option to this action? I could probably make a PR to add that change in if it’s desired.

Issue Analytics

  • State:open
  • Created 2 years ago
  • Reactions:4
  • Comments:5

github_iconTop GitHub Comments

1reaction
beondecommented, Nov 19, 2021

@Ninjaman494 Thats a good work around. Thanks for the suggestion! I think I’ll implement that for a fix

0reactions
anonimitorafcommented, Dec 5, 2022

If dist (in events) isn’t a must-have, assigning dist = '' in Sentry.init seems strip it (as opposed to dist = undefined or dist = null).

Read more comments on GitHub >

github_iconTop Results From Across the Web

Allow adding --dist option in uploading source map #703
Hi @HazAT, currently I'm facing a problem of Source code was not found in Url app:///index.android.bundle. I'm using Sentry together with ...
Read more >
source-map-loader - webpack
source-map -loader allows webpack to maintain source map data continuity across libraries so ease of debugging is preserved. The source-map-loader will extract ...
Read more >
gulp-sourcemaps - npm
Start using gulp-sourcemaps in your project by running `npm i ... To load existing source maps, pass the option loadMaps: true to ...
Read more >
Create and upload source maps (RUM) | APM User Guide [8.5]
sourcemap - The location of the source map. If you have multiple source maps, you'll need to upload each individually. Upload via curledit....
Read more >
Troubleshooting Source Maps for JavaScript
To add a dist value to your uploaded source maps, use the --dist flag with sentry-cli or the dist option in our Sentry...
Read more >

github_iconTop Related Medium Post

No results found

github_iconTop Related StackOverflow Question

No results found

github_iconTroubleshoot Live Code

Lightrun enables developers to add logs, metrics and snapshots to live code - no restarts or redeploys required.
Start Free

github_iconTop Related Reddit Thread

No results found

github_iconTop Related Hackernoon Post

No results found

github_iconTop Related Tweet

No results found

github_iconTop Related Dev.to Post

No results found

github_iconTop Related Hashnode Post

No results found