Skip to content

Commit b85f24e

Browse files
romtsnclaude
andcommitted
fix(screenshot): Log missing replay module warning once in constructor instead of per event
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
1 parent b85eb89 commit b85f24e

File tree

1 file changed

+6
-10
lines changed

1 file changed

+6
-10
lines changed

sentry-android-core/src/main/java/io/sentry/android/core/ScreenshotEventProcessor.java

Lines changed: 6 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -62,20 +62,16 @@ public ScreenshotEventProcessor(
6262

6363
if (options.isAttachScreenshot()) {
6464
addIntegrationToSdkVersion("Screenshot");
65+
if (!isReplayAvailable && !options.getScreenshotOptions().getMaskViewClasses().isEmpty()) {
66+
options
67+
.getLogger()
68+
.log(SentryLevel.WARNING, "Screenshot masking requires sentry-android-replay module");
69+
}
6570
}
6671
}
6772

6873
private boolean isMaskingEnabled() {
69-
if (options.getScreenshotOptions().getMaskViewClasses().isEmpty()) {
70-
return false;
71-
}
72-
if (!isReplayAvailable) {
73-
options
74-
.getLogger()
75-
.log(SentryLevel.WARNING, "Screenshot masking requires sentry-android-replay module");
76-
return false;
77-
}
78-
return true;
74+
return !options.getScreenshotOptions().getMaskViewClasses().isEmpty() && isReplayAvailable;
7975
}
8076

8177
@Override

0 commit comments

Comments
 (0)