You are viewing a plain text version of this content. The canonical link for it is here.
Posted to common-commits@hadoop.apache.org by cn...@apache.org on 2014/04/18 23:56:36 UTC
svn commit: r1588578 - in /hadoop/common/branches/branch-1: CHANGES.txt
src/hdfs/org/apache/hadoop/hdfs/web/WebHdfsFileSystem.java
src/test/org/apache/hadoop/hdfs/web/TestWebHdfsFileSystemContract.java
Author: cnauroth
Date: Fri Apr 18 21:56:35 2014
New Revision: 1588578
URL: http://svn.apache.org/r1588578
Log:
HDFS-6141. WebHdfsFileSystem#toUrl does not perform character escaping. Contributed by Chris Nauroth.
Modified:
hadoop/common/branches/branch-1/CHANGES.txt
hadoop/common/branches/branch-1/src/hdfs/org/apache/hadoop/hdfs/web/WebHdfsFileSystem.java
hadoop/common/branches/branch-1/src/test/org/apache/hadoop/hdfs/web/TestWebHdfsFileSystemContract.java
Modified: hadoop/common/branches/branch-1/CHANGES.txt
URL: http://svn.apache.org/viewvc/hadoop/common/branches/branch-1/CHANGES.txt?rev=1588578&r1=1588577&r2=1588578&view=diff
==============================================================================
--- hadoop/common/branches/branch-1/CHANGES.txt (original)
+++ hadoop/common/branches/branch-1/CHANGES.txt Fri Apr 18 21:56:35 2014
@@ -208,6 +208,9 @@ Release 1.3.0 - unreleased
MAPREDUCE-5822. FairScheduler does not preempt due to fairshare-starvation
when fairshare is 1. (Anubhav Dhoot via kasha)
+ HDFS-6141. WebHdfsFileSystem#toUrl does not perform character escaping.
+ (cnauroth)
+
Release 1.2.2 - unreleased
INCOMPATIBLE CHANGES
Modified: hadoop/common/branches/branch-1/src/hdfs/org/apache/hadoop/hdfs/web/WebHdfsFileSystem.java
URL: http://svn.apache.org/viewvc/hadoop/common/branches/branch-1/src/hdfs/org/apache/hadoop/hdfs/web/WebHdfsFileSystem.java?rev=1588578&r1=1588577&r2=1588578&view=diff
==============================================================================
--- hadoop/common/branches/branch-1/src/hdfs/org/apache/hadoop/hdfs/web/WebHdfsFileSystem.java (original)
+++ hadoop/common/branches/branch-1/src/hdfs/org/apache/hadoop/hdfs/web/WebHdfsFileSystem.java Fri Apr 18 21:56:35 2014
@@ -373,7 +373,7 @@ public class WebHdfsFileSystem extends F
final Param<?,?>... parameters) throws IOException {
//initialize URI path and query
final String path = PATH_PREFIX
- + (fspath == null? "/": makeQualified(fspath).toUri().getPath());
+ + (fspath == null? "/": makeQualified(fspath).toUri().getRawPath());
final String query = op.toQueryString()
+ '&' + new UserParam(ugi)
+ Param.toSortedString("&", parameters);
Modified: hadoop/common/branches/branch-1/src/test/org/apache/hadoop/hdfs/web/TestWebHdfsFileSystemContract.java
URL: http://svn.apache.org/viewvc/hadoop/common/branches/branch-1/src/test/org/apache/hadoop/hdfs/web/TestWebHdfsFileSystemContract.java?rev=1588578&r1=1588577&r2=1588578&view=diff
==============================================================================
--- hadoop/common/branches/branch-1/src/test/org/apache/hadoop/hdfs/web/TestWebHdfsFileSystemContract.java (original)
+++ hadoop/common/branches/branch-1/src/test/org/apache/hadoop/hdfs/web/TestWebHdfsFileSystemContract.java Fri Apr 18 21:56:35 2014
@@ -383,7 +383,7 @@ public class TestWebHdfsFileSystemContra
{//test create with path containing spaces
HttpOpParam.Op op = PutOpParam.Op.CREATE;
- Path path = new Path("/test/path%20with%20spaces");
+ Path path = new Path("/test/path with spaces");
URL url = webhdfs.toUrl(op, path);
HttpURLConnection conn = (HttpURLConnection)url.openConnection();
conn.setRequestMethod(op.getType().toString());