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 2015/05/04 20:05:46 UTC

logging-log4j2 git commit: [LOG4J2-1008] org.apache.logging.log4j.core.config.plugins.util.ResolverUtil.extractPath(URL) incorrectly converts '+' characters to spaces. Simplify.

Repository: logging-log4j2
Updated Branches:
  refs/heads/master 0c20bfd8a -> d4df900ca


[LOG4J2-1008]
org.apache.logging.log4j.core.config.plugins.util.ResolverUtil.extractPath(URL)
incorrectly converts '+' characters to spaces. Simplify.

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

Branch: refs/heads/master
Commit: d4df900cab07c0e1c370a3daaada1057720c2179
Parents: 0c20bfd
Author: Gary Gregory <ga...@gmail.com>
Authored: Mon May 4 11:05:43 2015 -0700
Committer: Gary Gregory <ga...@gmail.com>
Committed: Mon May 4 11:05:43 2015 -0700

----------------------------------------------------------------------
 .../logging/log4j/core/config/plugins/util/ResolverUtil.java      | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/logging-log4j2/blob/d4df900c/log4j-core/src/main/java/org/apache/logging/log4j/core/config/plugins/util/ResolverUtil.java
----------------------------------------------------------------------
diff --git a/log4j-core/src/main/java/org/apache/logging/log4j/core/config/plugins/util/ResolverUtil.java b/log4j-core/src/main/java/org/apache/logging/log4j/core/config/plugins/util/ResolverUtil.java
index 65da5bc..7001487 100644
--- a/log4j-core/src/main/java/org/apache/logging/log4j/core/config/plugins/util/ResolverUtil.java
+++ b/log4j-core/src/main/java/org/apache/logging/log4j/core/config/plugins/util/ResolverUtil.java
@@ -243,8 +243,7 @@ public class ResolverUtil {
             // if URL-encoded file exists, don't decode it
             return cleanPath;
         }
-        urlPath = URLDecoder.decode(urlPath, Constants.UTF_8.name());
-        return urlPath;
+        return URLDecoder.decode(urlPath, Constants.UTF_8.name());
     }
 
     private void loadImplementationsInBundle(final Test test, final String packageName) {