`beforeSend` is being ignored when `disablePerformance` is `false` (default)

• 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/ember
• @sentry/node
• raven-js
• raven-node (raven for node)
• other:

Version:

6.4.0

Description

We wanted to turn on performance metrics for our project in Sentry as it looked like a pretty neat feature, but… Project is written in Ember, we’re using @sentry/ember and up until now everything worked correctly. After turning on performance metrics (previously we were using disablePerformance: true; option) our hooks stopped working. We’re using beforeSend pretty extensively, like this:

  InitSentryForEmber({
    beforeSend(event, hint) {
      ...
    },
  });

I noticed that if performance metrics are on (or rather disablePerformance is not set), beforeSend hook never fires. I checked it a few times by disabling and enabling metrics, but always with the same result. Looks like enabling performance metrics somehow overrides our settings?

[kiwi-josh]

@netes FYI you may have been experiencing this issue: https://github.com/getsentry/sentry-javascript/pull/3791

[netes]

@netes FYI you may have been experiencing this issue: #3791

Thanks! That was the problem indeed!