search

getsentry / sentry-javascript

Official Sentry SDKs for JavaScript
https://sentry.io
MIT License
7.94k stars 1.56k forks source link

Chromium update: use_dynamic_url property set to false causes errors to not be reported with stack trace #14003

Open ReneGreen27 opened 3 days ago

ReneGreen27 commented 3 days ago

Environment

SaaS (https://sentry.io/)

Steps to Reproduce

Chrome 130 is where the customer sees this change occur.

When use_dynamic_url property is set to false this causes errors to not be reported to Sentry properly with a stack trace/ associated sourcemaps. Flipped back to true errors and associated stack traces are correctly reported. Customer is concerned this will not be a viable solution for long: see this Chromium issue thread: https://issues.chromium.org/issues/363027634

They were able to replicate this in both React and Browser sdk

Expected Result

To be able to set use_dynamic_url property to false and still have errors and associated sourcemaps reported properly

Actual Result

When set to true, they see reported errors without any stack trace and reported errors without any reference to source maps.

Product Area

Unknown

Link

No response

DSN

No response

Version

No response

getsantry[bot] commented 3 days ago

Auto-routing to @getsentry/product-owners-issues for triage ⏲️

mrduncan commented 3 days ago

@ReneGreen27 maybe I'm misunderstanding, but this seems like an SDK problem, is there a reason it was sent to issues?

ReneGreen27 commented 3 days ago

@mrduncan this was automatically routed to Issues, I will manually reroute to javascript SDK repo

chargome commented 3 days ago

@ReneGreen27 could you get more context on this issue? Is the SDK in this case running in a custom browser extension?

andreiborza commented 3 days ago

@ReneGreen27 are they uploading sourcemaps to Sentry? It sounds to me like they rely on the sourcemap url in sourcemaps. A more robust solution would be to use the Sentry CLI or one of our bundler plugins to inject debug ids into their sourcemaps and upload them on build to Sentry which should not be affected by the use_dynamic_url flag in Chrome.