attachScreenshot: true still takes screenshots even when enabled: false

[liamjones]

Platform:

• iOS
• Android

SDK:

• @sentry/react-native (>= 1.0.0)

SDK version: 5.9.2

react-native version: 0.71.13

Are you using Expo?

• No

Are you using sentry.io or on-premise?

• sentry.io (SaaS)

If you are using sentry.io, please post a link to your issue so we can take a look:

N/A

Configuration:

(@sentry/react-native)

Sentry.init({
  dsn: 'https://[email protected]/4505951781191680',
  enabled: false,
  attachScreenshot: true,
});

---

I have following issue:

[Description]

Steps to reproduce:

• Configure Sentry with attachScreenshot: true and enabled: false.
• Connect debugger and set breakpoints in 'take screenshot' functions of Android/iOS native modules
• Call Sentry.captureException()

Actual result:

Breakpoints hit as native modules try to capture a screenshot

Expected result:

Screenshots not taken as no reporting is going to happen

[krystofwoldrich]

Hi, thank you for the information,
I agree, I would also expect the screenshot not to be taken when the SDK is disabled.

[krystofwoldrich]

This is caused by the Screenshot integration being hooked to captureException and not using the event processor.

sentry-react-native/src/js/client.ts

Line 55 in 5af0e04

return Screenshot.attachScreenshotToEventHint(hint, this._options).then(hintWithScreenshot =>