Fix log replay of the Log4j 2 appender (#10243)

open-telemetry · Jan 17, 2024 · 9ec6401 · 9ec6401
1 parent dbb8ab1
commit 9ec6401
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 3 deletions.
diff --git a/...ain/java/io/opentelemetry/instrumentation/log4j/appender/v2_17/OpenTelemetryAppender.java b/...ain/java/io/opentelemetry/instrumentation/log4j/appender/v2_17/OpenTelemetryAppender.java
@@ -157,9 +157,6 @@ public B setOpenTelemetry(OpenTelemetry openTelemetry) {
     @Override
     public OpenTelemetryAppender build() {
       OpenTelemetry openTelemetry = this.openTelemetry;
-      if (openTelemetry == null) {
-        openTelemetry = OpenTelemetry.noop();
-      }
       return new OpenTelemetryAppender(
           getName(),
           getLayout(),

diff --git a/...pentelemetry/instrumentation/log4j/appender/v2_17/LogReplayOpenTelemetryAppenderTest.java b/...pentelemetry/instrumentation/log4j/appender/v2_17/LogReplayOpenTelemetryAppenderTest.java
@@ -14,6 +14,7 @@
 import java.util.List;
 import org.apache.logging.log4j.message.StringMapMessage;
 import org.apache.logging.log4j.message.StructuredDataMessage;
+import org.junit.jupiter.api.AfterEach;
 import org.junit.jupiter.api.BeforeEach;
 import org.junit.jupiter.api.Test;
 
@@ -22,6 +23,10 @@ class LogReplayOpenTelemetryAppenderTest extends AbstractOpenTelemetryAppenderTe
   @BeforeEach
   void setup() {
     generalBeforeEachSetup();
+  }
+
+  @AfterEach
+  void resetOpenTelemetry() {
     OpenTelemetryAppender.install(null);
   }