This is an automated email from the ASF dual-hosted git repository.

lhotari pushed a commit to branch branch-2.7
in repository https://gitbox.apache.org/repos/asf/pulsar.git


The following commit(s) were added to refs/heads/branch-2.7 by this push:
     new 6fee20f  [logging] Use Log4j2's BOM to ensure that Log4j2 version is 
consistent. Replace legacy log4j with log4j-1.2-api
6fee20f is described below

commit 6fee20fddec47bfc98924fdea6a4d28ff3894c1e
Author: Lari Hotari <[email protected]>
AuthorDate: Fri Dec 10 08:53:24 2021 +0200

    [logging] Use Log4j2's BOM to ensure that Log4j2 version is consistent. 
Replace legacy log4j with log4j-1.2-api
    
    backports PR #8880 to branch-2.7
    [logging] Upgrade Log4j2 version to 2.14.0, replace legacy log4j dependency 
with log4j-1.2-api (#8880)
    
    - import Log4j2's BOM into Pulsar's dependency management instead of
      defining versions separately for each Log4j2 library
    
    - distribution/server/pom.xml included log4j:log4j dependency.
      - replace with org.apache.logging.log4j:log4j-1.2-api to
        keep supporting the legacy logging api in the case something
        depends on it
    
    (cherry picked from commit 2e1fb0511bbcf76e08e83d3f3d9a68a275c56973)
---
 buildtools/pom.xml                               |  6 ++--
 distribution/server/pom.xml                      |  4 +--
 distribution/server/src/assemble/LICENSE.bin.txt |  2 +-
 managed-ledger/pom.xml                           |  4 +--
 pom.xml                                          | 36 ++----------------------
 5 files changed, 11 insertions(+), 41 deletions(-)

diff --git a/buildtools/pom.xml b/buildtools/pom.xml
index 9390ba2..46d478e 100644
--- a/buildtools/pom.xml
+++ b/buildtools/pom.xml
@@ -43,17 +43,17 @@
     <dependency>
       <groupId>org.apache.logging.log4j</groupId>
       <artifactId>log4j-api</artifactId>
-      <version>2.10.0</version>
+      <version>2.15.0</version>
     </dependency>
     <dependency>
       <groupId>org.apache.logging.log4j</groupId>
       <artifactId>log4j-core</artifactId>
-      <version>2.10.0</version>
+      <version>2.15.0</version>
     </dependency>
     <dependency>
       <groupId>org.apache.logging.log4j</groupId>
       <artifactId>log4j-slf4j-impl</artifactId>
-      <version>2.10.0</version>
+      <version>2.15.0</version>
     </dependency>
   </dependencies>
 
diff --git a/distribution/server/pom.xml b/distribution/server/pom.xml
index 619d9da..074b519 100644
--- a/distribution/server/pom.xml
+++ b/distribution/server/pom.xml
@@ -83,8 +83,8 @@
     </dependency>
 
     <dependency>
-      <artifactId>log4j</artifactId>
-      <groupId>log4j</groupId>
+      <groupId>org.apache.logging.log4j</groupId>
+      <artifactId>log4j-1.2-api</artifactId>
     </dependency>
 
     <dependency>
diff --git a/distribution/server/src/assemble/LICENSE.bin.txt 
b/distribution/server/src/assemble/LICENSE.bin.txt
index f69a7a4..92762ca 100644
--- a/distribution/server/src/assemble/LICENSE.bin.txt
+++ b/distribution/server/src/assemble/LICENSE.bin.txt
@@ -388,11 +388,11 @@ The Apache Software License, Version 2.0
     - jakarta.validation-jakarta.validation-api-2.0.2.jar
     - javax.validation-validation-api-1.1.0.Final.jar
  * Log4J
-    - log4j-log4j-1.2.17.jar
     - org.apache.logging.log4j-log4j-api-2.15.0.jar
     - org.apache.logging.log4j-log4j-core-2.15.0.jar
     - org.apache.logging.log4j-log4j-slf4j-impl-2.15.0.jar
     - org.apache.logging.log4j-log4j-web-2.15.0.jar
+    - org.apache.logging.log4j-log4j-1.2-api-2.15.0.jar
  * Java Native Access JNA -- net.java.dev.jna-jna-4.2.0.jar
  * BookKeeper
     - org.apache.bookkeeper-bookkeeper-common-4.12.0.jar
diff --git a/managed-ledger/pom.xml b/managed-ledger/pom.xml
index 691b74a..86cb5ed 100644
--- a/managed-ledger/pom.xml
+++ b/managed-ledger/pom.xml
@@ -103,8 +103,8 @@
     </dependency>
 
     <dependency>
-      <groupId>log4j</groupId>
-      <artifactId>log4j</artifactId>
+      <groupId>org.apache.logging.log4j</groupId>
+      <artifactId>log4j-1.2-api</artifactId>
       <scope>test</scope>
     </dependency>
 
diff --git a/pom.xml b/pom.xml
index d55a49a..b66da71 100644
--- a/pom.xml
+++ b/pom.xml
@@ -529,40 +529,10 @@ flexible messaging model and an intuitive client 
API.</description>
 
       <dependency>
         <groupId>org.apache.logging.log4j</groupId>
-        <artifactId>log4j-api</artifactId>
-        <version>${log4j2.version}</version>
-      </dependency>
-
-      <dependency>
-        <groupId>org.apache.logging.log4j</groupId>
-        <artifactId>log4j-core</artifactId>
-        <version>${log4j2.version}</version>
-      </dependency>
-
-      <dependency>
-        <groupId>org.apache.logging.log4j</groupId>
-        <artifactId>log4j-slf4j-impl</artifactId>
-        <version>${log4j2.version}</version>
-      </dependency>
-
-      <dependency>
-        <groupId>org.apache.logging.log4j</groupId>
-        <artifactId>log4j-web</artifactId>
-        <version>${log4j2.version}</version>
-      </dependency>
-
-      <dependency>
-        <groupId>org.apache.logging.log4j</groupId>
-        <artifactId>log4j-api</artifactId>
-        <type>test-jar</type>
-        <version>${log4j2.version}</version>
-      </dependency>
-
-      <dependency>
-        <groupId>org.apache.logging.log4j</groupId>
-        <artifactId>log4j-core</artifactId>
-        <type>test-jar</type>
+        <artifactId>log4j-bom</artifactId>
         <version>${log4j2.version}</version>
+        <type>pom</type>
+        <scope>import</scope>
       </dependency>
 
       <dependency>

Reply via email to