[geode] branch develop updated: GEODE-5001: For log4j >= 2.10 we need to ignore the log4j jars from the PerTestClassLoaderRunner (#1938)

[jensdeppe]

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

jensdeppe pushed a commit to branch develop
in repository https://gitbox.apache.org/repos/asf/geode.git


The following commit(s) were added to refs/heads/develop by this push:
     new a6cdf6b  GEODE-5001: For log4j >= 2.10 we need to ignore the log4j 
jars from the PerTestClassLoaderRunner (#1938)
a6cdf6b is described below

commit a6cdf6b414e8baf45d9685c4ba358610b242d2f9
Author: Jens Deppe <jde...@pivotal.io>
AuthorDate: Wed May 9 12:56:07 2018 -0700

    GEODE-5001: For log4j >= 2.10 we need to ignore the log4j jars from the 
PerTestClassLoaderRunner (#1938)
    
    - See LOG4J2-2266 and LOG4J-2152
---
 .../geode/modules/session/junit/PerTestClassLoaderRunner.java       | 6 ++++++
 1 file changed, 6 insertions(+)

diff --git 
a/extensions/geode-modules-session/src/test/java/org/apache/geode/modules/session/junit/PerTestClassLoaderRunner.java
 
b/extensions/geode-modules-session/src/test/java/org/apache/geode/modules/session/junit/PerTestClassLoaderRunner.java
index c237e11..37ba6e0 100644
--- 
a/extensions/geode-modules-session/src/test/java/org/apache/geode/modules/session/junit/PerTestClassLoaderRunner.java
+++ 
b/extensions/geode-modules-session/src/test/java/org/apache/geode/modules/session/junit/PerTestClassLoaderRunner.java
@@ -79,6 +79,12 @@ public class PerTestClassLoaderRunner extends NamedRunner {
     List<URL> urls = new ArrayList<>();
     while (st.hasMoreTokens()) {
       String u = st.nextToken();
+      // Because of LOG4J2-2266 and LOG4J-2152 ignore all log4j related jars 
and let them be picked
+      // up from the system classloader. Although it's supposed to be fixed in 
log4j 2.10, it does
+      // not seem to be.
+      if (u.contains("log4j")) {
+        continue;
+      }
       try {
         if (!u.endsWith(".jar")) {
           u += "/";

-- 
To stop receiving notification emails like this one, please contact
jensde...@apache.org.