Do not use Sentry logging API if logs are disabled

adinauer · adinauer · commit e71d692573f8 · 2025-07-29T13:15:26.000+02:00
diff --git a/sentry-log4j2/src/main/java/io/sentry/log4j2/SentryAppender.java b/sentry-log4j2/src/main/java/io/sentry/log4j2/SentryAppender.java
@@ -191,7 +191,8 @@ public void start() {
 
   @Override
   public void append(final @NotNull LogEvent eventObject) {
-    if (eventObject.getLevel().isMoreSpecificThan(minimumLevel)) {
+    if (scopes.getOptions().getLogs().isEnabled()
+      && eventObject.getLevel().isMoreSpecificThan(minimumLevel)) {
       captureLog(eventObject);
     }
     if (eventObject.getLevel().isMoreSpecificThan(minimumEventLevel)) {
diff --git a/sentry-samples/sentry-samples-log4j2/src/main/resources/sentry.properties b/sentry-samples/sentry-samples-log4j2/src/main/resources/sentry.properties
@@ -1,2 +1,3 @@
 in-app-includes="io.sentry.samples"
 logs.enabled=true
+debug=true

Original file line number	Diff line number	Diff line change
`@@ -191,7 +191,8 @@ public void start() {`
`191`	`191`
`192`	`192`	`@Override`
`193`	`193`	`public void append(final @NotNull LogEvent eventObject) {`
`194`		`- if (eventObject.getLevel().isMoreSpecificThan(minimumLevel)) {`
	`194`	`+ if (scopes.getOptions().getLogs().isEnabled()`
	`195`	`+ && eventObject.getLevel().isMoreSpecificThan(minimumLevel)) {`
`195`	`196`	`captureLog(eventObject);`
`196`	`197`	`}`
`197`	`198`	`if (eventObject.getLevel().isMoreSpecificThan(minimumEventLevel)) {`
Original file line number	Diff line number	Diff line change
`@@ -1,2 +1,3 @@`
`1`	`1`	`in-app-includes="io.sentry.samples"`
`2`	`2`	`logs.enabled=true`
	`3`	`+debug=true`