You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@ranger.apache.org by pr...@apache.org on 2022/01/26 05:00:23 UTC

[ranger] branch ranger-2.3 updated: RANGER-3498 : RANGER : Remove log4j1 dependencies.

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

pradeep pushed a commit to branch ranger-2.3
in repository https://gitbox.apache.org/repos/asf/ranger.git


The following commit(s) were added to refs/heads/ranger-2.3 by this push:
     new 54d491c  RANGER-3498 : RANGER : Remove log4j1 dependencies.
54d491c is described below

commit 54d491cdee6f2704b7862e45c03317fc8536bf68
Author: mateenmansoori <ma...@gmail.com>
AuthorDate: Tue Jan 25 16:23:35 2022 +0530

    RANGER-3498 : RANGER : Remove log4j1 dependencies.
---
 distro/src/main/assembly/kms.xml      | 5 +++--
 kms/config/kms-webapp/kms-logback.xml | 8 +++-----
 kms/pom.xml                           | 1 -
 3 files changed, 6 insertions(+), 8 deletions(-)

diff --git a/distro/src/main/assembly/kms.xml b/distro/src/main/assembly/kms.xml
index b119f50..be8b17b 100755
--- a/distro/src/main/assembly/kms.xml
+++ b/distro/src/main/assembly/kms.xml
@@ -39,9 +39,10 @@
                     <include>org.eclipse.persistence:eclipselink</include>
                     <include>org.eclipse.persistence:javax.persistence</include>
                     <include>com.googlecode.log4jdbc:log4jdbc</include>
-                    <include>log4j:log4j</include>
                     <include>org.slf4j:slf4j-api</include>
-                    <include>org.slf4j:slf4j-log4j12</include>
+                    <include>org.slf4j:log4j-over-slf4j:jar:${slf4j.version}</include>
+                    <include>ch.qos.logback:logback-classic:jar:${logback.version}</include>
+                    <include>ch.qos.logback:logback-core:jar:${logback.version}</include>
                     <include>com.codahale.metrics:metrics-core</include>
                     <include>org.slf4j:jul-to-slf4j</include>
                     <include>commons-logging:commons-logging</include>
diff --git a/kms/config/kms-webapp/kms-logback.xml b/kms/config/kms-webapp/kms-logback.xml
index b7290c0..d47e86c 100644
--- a/kms/config/kms-webapp/kms-logback.xml
+++ b/kms/config/kms-webapp/kms-logback.xml
@@ -27,18 +27,16 @@
       <pattern>%d{ISO8601} %m%n</pattern>
     </encoder>
     <rollingPolicy class="ch.qos.logback.core.rolling.TimeBasedRollingPolicy">
-      <fileNamePattern>${kms.log.dir}/kms-audit-${hostname}-${user}.log.%d{'.'yyyy-MM-dd}</fileNamePattern>
+      <fileNamePattern>${kms.log.dir}/kms-audit-${hostname}-${user}.log.%d{yyyy-MM-dd}</fileNamePattern>
     </rollingPolicy>
   </appender>
-  <appender name="kms-metric" class="ch.qos.logback.core.rolling.RollingFileAppender">
+  <appender name="kms-metric" class="ch.qos.logback.core.FileAppender">
     <!--See also http://logback.qos.ch/manual/appenders.html#RollingFileAppender-->
     <Append>false</Append>
     <File>${kms.log.dir}/ranger_kms_metric_data_for_${metric.type}.log</File>
     <encoder>
       <pattern>%m%n</pattern>
     </encoder>
-    <rollingPolicy class="ch.qos.logback.core.rolling.FixedWindowRollingPolicy"/>
-    <triggeringPolicy class="ch.qos.logback.core.rolling.SizeBasedTriggeringPolicy"/>
   </appender>
   <appender name="kms" class="ch.qos.logback.core.rolling.RollingFileAppender">
     <!--See http://logback.qos.ch/manual/appenders.html#RollingFileAppender-->
@@ -50,7 +48,7 @@
       <pattern>%d{ISO8601} %-5p %c{1} - %m%n</pattern>
     </encoder>
     <rollingPolicy class="ch.qos.logback.core.rolling.TimeBasedRollingPolicy">
-      <fileNamePattern>${kms.log.dir}/ranger-kms-${hostname}-${user}.log.%d{'.'yyyy-MM-dd}</fileNamePattern>
+      <fileNamePattern>${kms.log.dir}/ranger-kms-${hostname}-${user}.log.%d{yyyy-MM-dd}</fileNamePattern>
     </rollingPolicy>
   </appender>
   <logger name="com.sun.jersey.server.wadl.generators.WadlGeneratorJAXBGrammarGenerator" level="OFF"/>
diff --git a/kms/pom.xml b/kms/pom.xml
index 08daea2..3545c33 100644
--- a/kms/pom.xml
+++ b/kms/pom.xml
@@ -47,7 +47,6 @@
             <groupId>org.slf4j</groupId>
             <artifactId>log4j-over-slf4j</artifactId>
             <version>${slf4j.version}</version>
-            <scope>test</scope>
         </dependency>
         <dependency>
             <groupId>ch.qos.logback</groupId>