Couldn't removeEventListener on version 5.19.1 in some cases

[scorpio13love]

• 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/integrations

Version:

5.19.1

Description

Some "removeEventListener methods" didn't work when i upgrade packages to version 5.19.1.

But things work out all right on previous version, such as version 5.15.1.

And i found that this config below will cause "callback.sentry" to be "undefined" which committed at 9a68a35

Sentry.init({
  // ...
  integrations: [new Sentry.Integrations.Breadcrumbs({ dom: false })]
});

Is this the reason that make event listeners which registered after sentry inited couldn't be remove ?

How to fix it ？

[kamilogorek]

Are you able to provide some sort of repro-case? If there's indeed an issue, I'd like to get it fixed right away.

[scorpio13love]

@kamilogorek
I created a repo like this ：https://github.com/scorpio13love/sentry-dom-false.

1. Input sentry "dsn".
2. Click button to change router between "page home" and "page detail" several times.
3. You can count the "page detail click" msg in DevTools Console Or look at the Document's Binding EventListeners.

[kamilogorek]

@scorpio13love thanks, I'll look into it. From my first pass, it appears that disabling of dom breadcrumbs is causing it. Very odd though. Will get back to you once I find the issue.