You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@logging.apache.org by gg...@apache.org on 2016/09/13 19:10:40 UTC

logging-log4j2 git commit: [LOG4J2-1320] Custom plugins are not loaded, URL protocol vfs is not supported. Apply patch to URLStreamHandlerFactoryRule, just a comment.

Repository: logging-log4j2
Updated Branches:
  refs/heads/master 5fa1ac1dd -> cf7cce726


[LOG4J2-1320] Custom plugins are not loaded, URL protocol vfs is not
supported. Apply patch to URLStreamHandlerFactoryRule, just a comment.

Project: http://git-wip-us.apache.org/repos/asf/logging-log4j2/repo
Commit: http://git-wip-us.apache.org/repos/asf/logging-log4j2/commit/cf7cce72
Tree: http://git-wip-us.apache.org/repos/asf/logging-log4j2/tree/cf7cce72
Diff: http://git-wip-us.apache.org/repos/asf/logging-log4j2/diff/cf7cce72

Branch: refs/heads/master
Commit: cf7cce726f0360de16b11bd79924674ad3cc5191
Parents: 5fa1ac1
Author: Gary Gregory <gg...@apache.org>
Authored: Tue Sep 13 12:10:36 2016 -0700
Committer: Gary Gregory <gg...@apache.org>
Committed: Tue Sep 13 12:10:36 2016 -0700

----------------------------------------------------------------------
 .../apache/logging/log4j/junit/URLStreamHandlerFactoryRule.java    | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/logging-log4j2/blob/cf7cce72/log4j-core/src/test/java/org/apache/logging/log4j/junit/URLStreamHandlerFactoryRule.java
----------------------------------------------------------------------
diff --git a/log4j-core/src/test/java/org/apache/logging/log4j/junit/URLStreamHandlerFactoryRule.java b/log4j-core/src/test/java/org/apache/logging/log4j/junit/URLStreamHandlerFactoryRule.java
index f6433ed..2d68b58 100644
--- a/log4j-core/src/test/java/org/apache/logging/log4j/junit/URLStreamHandlerFactoryRule.java
+++ b/log4j-core/src/test/java/org/apache/logging/log4j/junit/URLStreamHandlerFactoryRule.java
@@ -82,7 +82,7 @@ public class URLStreamHandlerFactoryRule implements TestRule {
                 Assert.assertNotNull("java.net URL does not declare a java.net.URLStreamHandlerFactory field",
                         factoryField);
                 Assert.assertEquals("java.net.URL declares multiple java.net.URLStreamHandlerFactory fields.", 1,
-                        matches);
+                        matches); // FIXME There is a break in the loop so always 0 or 1
                 URL.setURLStreamHandlerFactory(newURLStreamHandlerFactory);
                 try {
                     base.evaluate();