Merge pull request #62 from jamezp/use-immutable-map

Ensure an immutable map is returned for the getImmutableMapOrNull().
jboss-logging · Nov 9, 2023 · 7fbecf8 · 7fbecf8
2 parents 51d11ea + a8c2ab9
commit 7fbecf8
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/src/main/java/org/jboss/logmanager/log4j/ThreadContextMDCMap.java b/src/main/java/org/jboss/logmanager/log4j/ThreadContextMDCMap.java
@@ -19,6 +19,7 @@
 
 package org.jboss.logmanager.log4j;
 
+import java.util.Collections;
 import java.util.Map;
 
 import org.apache.logging.log4j.spi.ThreadContextMap;
@@ -53,7 +54,7 @@ public Map<String, String> getCopy() {
     @Override
     public Map<String, String> getImmutableMapOrNull() {
         final Map<String, String> copy = MDC.copy();
-        return copy.isEmpty() ? null : copy;
+        return copy.isEmpty() ? null : Collections.unmodifiableMap(copy);
     }
 
     @Override