Sentry-javascript: How to disable Sentry again at runtime?

https://github.com/getsentry/sentry-javascript/issues/2027#issuecomment-484475157

SentryBrowser.getCurrentHub().getClient().getOptions().enabled

That gets me the current value, but how do I _change_ it?

Sentry.getCurrentHub().getClient().getOptions().enabled = false; 😅

And just to be sure, that will make Sentry stop sending anything to Sentry servers?

Correct - https://github.com/getsentry/sentry-javascript/blob/41f01bc4cc138d430e0c58a80ec27e877655b01d/packages/core/src/baseclient.ts#L327-L329

Could you consider this a feature request for something more explicit, or at least a request for official documentation?

@felixfbecker Sure, can you just elaborate why you need this?
Is it some kind of "Ask the user for permission to send crash reports?"?

As described in the issue description, we need to allow users to opt in and out of error reporting for privacy reasons. Anything that is unclear? Happy to provide more info but not sure what you are looking for

@felixfbecker Sorry, still not fully awake 😅 should have read the initial issue.
We will add something like Sentry.setEnabled(boolean) it makes perfect sense to have something like this. We also got some reports for other SDKs already.

Just for the record, there are other ways of handling this (e.g. beforeSend) besides what Kamil posted but they are all not really convenient.

Why is this issue closed if this feature request is in the works ?

Any updates on providing the Sentry.setEnabled(boolean) to toggle error reports on/off?

In Raven I think you had shouldSendCallback to toggle reporting

In Raven I think you had shouldSendCallback to toggle reporting

Sentry.init({
  beforeSend(event) {
    if (shouldSendEvent) return event;
    return null;
  }
});

Any updates on providing the Sentry.setEnabled(boolean) to toggle error reports on/off?

What's your use-case specifically? It's already possible to conditionally drop and/or enable/disable sdk.

Thanks, I couldn't find on the docs if actually sending null would prevent the sdk from reporting errors. this solves my use case

@diegolanda https://docs.sentry.io/error-reporting/configuration/?platform=go#before-send but I agree nothing is not specific enough for javascript 😅