Cannot read property '__sentry_instrumentation_handlers__' of undefined

[dyatko]

• Review the documentation: https://docs.sentry.io/
• Search for existing issues: https://github.com/getsentry/sentry-javascript/issues
• Use the latest release: https://github.com/getsentry/sentry-javascript/releases
• Provide a link to the affected event from your Sentry account

Package + Version

• @sentry/browser
• @sentry/node
• raven-js
• raven-node (raven for node)
• other:

Version:

6.0.3

Description

@sentry/browser 6.0.3 does not work with web-vitals

It breaks here:

sentry-javascript/packages/utils/src/instrument.ts

Line 535 in 61563f8

const handlers = (el.__sentry_instrumentation_handlers__ = el.__sentry_instrumentation_handlers__ || {});

Due to:

https://github.com/GoogleChrome/web-vitals/blob/43ce5464e3068c796830c5e98950fd94e480f4dd/src/getLCP.ts#L64

[dyatko]

@kamilogorek I believe, this intended to be window but since it's not called as window.addEventListener, the context is missing.

[kamilogorek]

Thanks for letting us know. Patch has been merged and 6.0.4 is coming in few moments.

[dyatko]

Nice, thank you!

[MattFanto]

Hi I'm facing the same issue with

    "@sentry/browser": "6.16.1",
    "@sentry/react": "6.16.1",
    "@sentry/tracing": "6.16.1",
    "@sentry/integrations": "6.16.1"

I have also "web-vitals": "2.1.3" which I'm using to push vitals to gtag as

// gatsby-browser.js

const sendToGTMNamespaced = ({name, id, value, delta}) => {
    const dlObj = {
        event: `webVitalsMeasurement.${name}`,
        webVitalsMeasurement: {}
    }
    dlObj.webVitalsMeasurement[name] = {
        name: name,
        id: id,
        value: Math.round(name === 'CLS' ? value * 1000 : value),
        delta: Math.round(name === 'CLS' ? delta * 1000 : delta),
    }
    window.dataLayer.push(dlObj)
}

exports.onClientEntry = () => {
    webVitals.getCLS(sendToGTMNamespaced)
    webVitals.getFID(sendToGTMNamespaced)
    webVitals.getLCP(sendToGTMNamespaced)
    webVitals.getFCP(sendToGTMNamespaced)
    webVitals.getTTFB(sendToGTMNamespaced)
}

[MattFanto]

Sorry that was a cache issue