Repository navigation
@sentry/gatsby: TypeError: Cannot read properties of undefined (reading 'href') after upgrading to latest version #10578
Description
Is there an existing issue for this?
- I have checked for existing issues https://github.com/getsentry/sentry-javascript/issues
- I have reviewed the documentation https://docs.sentry.io/
- I am using the latest SDK release https://github.com/getsentry/sentry-javascript/releases
How do you use Sentry?
Sentry Saas (sentry.io)
Which SDK are you using?
@sentry/gatsby
SDK Version
7.100.1
Framework Version
5.12.11
Link to Sentry event
No response
SDK Setup
Sentry.init({
dsn: {my_dsn},
enabled: process.env.NODE_ENV === 'development',
environment: process.env.NODE_ENV,
tracesSampleRate: 1.0
})
Steps to Reproduce
- upgrade to latest
@sentry/gatsbyversion to reduce bundle size - start a staging build in gatsby cloud
- error logged
Expected Result
works as it did in 7.42.0
Actual Result
TypeError: Cannot read properties of undefined (reading 'href')
- browserTracingIntegration.js:185 Object.afterAllSetup
[...]/[@sentry-internal]/tracing/esm/browser/browserTracingInte gration.js:185:31
- integration.js:98 afterSetupIntegrations
[...]/[@sentry]/core/esm/integration.js:98:1
- baseclient.js:423 BrowserClient._setupIntegrations
[...]/[@sentry]/core/esm/baseclient.js:423:5
- baseclient.js:270 BrowserClient.init
[...]/[@sentry]/core/esm/baseclient.js:270:1
- sdk.js:56 initializeClient
[...]/[@sentry]/core/esm/sdk.js:56:1
Activity
elanderos318 commentedon Feb 8, 2024
update, error no longer thrown after downgrading to
7.99.0Lms24 commentedon Feb 9, 2024
Hi @elanderos318 any chance that the
browserTracingIntegrationis registered on the server side? Haven't looked much into Gatsby yet but this smells like browser code being executed on the server.mydea commentedon Feb 14, 2024
I think we can add some extra guards here, but generally it should be avoided to call this in a non-browser environment!
fix(tracing): Guard against missing `window.location` (#10659)
Skeyelab commentedon Feb 20, 2024
I am still having this issue. Even with 7.102
mydea commentedon Feb 21, 2024
Hey, I forgot to backport this to v7, will include it in the next release!
fix(tracing): Guard against missing `window.location` (#10659)
Skeyelab commentedon Feb 21, 2024
@mydea thanks. Any idea when the next release will go out?
mydea commentedon Feb 21, 2024
We'll probably cut another release this week, we'll keep you posted in this issue!