GH-218 - Fix potential ConcurrentModificationException in PluginLogger.

Switching to a ConcurrentSkipListMap to allow safe concurrent executions but retain the ordering by keys.
xmolecules · Jan 17, 2024 · c086e97 · c086e97
1 parent 27e94a5
commit c086e97
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/jmolecules-bytebuddy/src/main/java/org/jmolecules/bytebuddy/PluginLogger.java b/jmolecules-bytebuddy/src/main/java/org/jmolecules/bytebuddy/PluginLogger.java
@@ -24,6 +24,7 @@
 import java.util.Set;
 import java.util.TreeMap;
 import java.util.TreeSet;
+import java.util.concurrent.ConcurrentSkipListMap;
 
 /**
  * Logger to aggregate information about transformations applied to individual {@link TypeDescription}s.
@@ -35,7 +36,7 @@ enum PluginLogger {
 
 	INSTANCE;
 
-	private Map<String, Set<LogEntry>> logs = new TreeMap<>();
+	private Map<String, Set<LogEntry>> logs = new ConcurrentSkipListMap<>();
 
 	/**
 	 * Obtains the {@link Log} for the given {@link TypeDescription} and module name.