sentry does not consider sourcemaps

getsentry / sentry-javascript-bundler-plugins

JavaScript Bundler Plugins for Sentry

https://sentry.io

BSD 3-Clause "New" or "Revised" License

141 stars 36 forks source link

sentry does not consider sourcemaps #553

Closed norflin321 closed 4 months ago

norflin321 commented 5 months ago

After updating @sentry/browser to version 8.10.0 and @sentry/vite-plugin to version 2.19.0, code in issues on sentry website is shown minified. Sourcemaps are uploaded correctly, sourceMappingURL is correct, debugId is correct.
Literally the same code and the same vite config, but with @sentry/browser version 8.9.2 and @sentry/vite-plugin version 2.18.0, works correct!

is this expected behavior? what steps need to be taken to upgrade to new versions?

lforst commented 5 months ago

Hi, can you share more information? The website, a Sentry event, your setup, debug logs when setting debug: true, ideally a reproduction example? Thanks!

Fox32 commented 4 months ago

Is this fixed by #556?

lforst commented 4 months ago

@Fox32 Yeah that might very likely be it. @norflin321 @keithgg-carebit @dhenneke (I saw you guys gave this a thumb). Would you mind trying out version 2.20.1 or higher? Thanks!

keithgg-carebit commented 4 months ago

@lforst looks like that fixed it. I can confirm that the correct number of artifacts are uploaded (with the broken version the number was half), but I haven't checked if the source maps are actually linked properly.

I'll ping you if that's not the case.

lforst commented 4 months ago

Awesome! I'm gonna close this issue until then. Feel free to ping us here any time.