Remove Log4j v1 support completely, even with log4j-1.2-api bridge

See #11925 for background.
netty · Dec 15, 2021 · 776c5ba · 776c5ba
1 parent c83027c
commit 776c5ba
Show file tree

Hide file tree

Showing 6 changed files with 0 additions and 715 deletions.
diff --git a/common/pom.xml b/common/pom.xml
@@ -63,11 +63,6 @@
       <artifactId>commons-logging</artifactId>
       <optional>true</optional>
     </dependency>
-    <dependency>
-      <groupId>org.apache.logging.log4j</groupId>
-      <artifactId>log4j-1.2-api</artifactId>
-      <optional>true</optional>
-    </dependency>
     <dependency>
       <groupId>org.apache.logging.log4j</groupId>
       <artifactId>log4j-api</artifactId>

diff --git a/common/src/main/java/io/netty/util/internal/logging/InternalLoggerFactory.java b/common/src/main/java/io/netty/util/internal/logging/InternalLoggerFactory.java
@@ -49,11 +49,6 @@ private static InternalLoggerFactory newDefaultFactory(String name) {
             return f;
         }
 
-        f = useLog4JLoggerFactory(name);
-        if (f != null) {
-            return f;
-        }
-
         return useJdkLoggerFactory(name);
     }
 
@@ -83,19 +78,6 @@ private static InternalLoggerFactory useLog4J2LoggerFactory(String name) {
         }
     }
 
-    private static InternalLoggerFactory useLog4JLoggerFactory(String name) {
-        try {
-            InternalLoggerFactory f = Log4JLoggerFactory.INSTANCE;
-            f.newInstance(name).debug("Using Log4J as the default logging framework");
-            return f;
-        } catch (LinkageError ignore) {
-            return null;
-        } catch (Exception ignore) {
-            // We catch Exception and not ReflectiveOperationException as we still support java 6
-            return null;
-        }
-    }
-
     private static InternalLoggerFactory useJdkLoggerFactory(String name) {
         InternalLoggerFactory f = JdkLoggerFactory.INSTANCE;
         f.newInstance(name).debug("Using java.util.logging as the default logging framework");