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.