DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG· RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT <http://issues.apache.org/bugzilla/show_bug.cgi?id=39564>. ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND· INSERTED IN THE BUG DATABASE.
http://issues.apache.org/bugzilla/show_bug.cgi?id=39564 Summary: literal plus (+) character in path components of HttpURL is not preserved. Product: HttpClient Version: 3.0.1 Platform: Other OS/Version: other Status: NEW Severity: normal Priority: P2 Component: Commons HttpClient AssignedTo: httpclient-dev@jakarta.apache.org ReportedBy: [EMAIL PROTECTED] When a literal plus character is included in the path component of an URL, it is not encoded, but get decoded during getPath() to a space. Reproducible with the following: HttpURL httpURL = new HttpURL("http://localhost/test+test";); System.out.println(httpURL.getPath()); Output: "test test" The following path fixes the issue (This patch does not appear to break anything else): Patch against SVN Repo: URL: http://svn.apache.org/repos/asf/jakarta/commons/proper/httpclient/trunk Repository UUID: 13f79535-47bb-0310-9956-ffa450edef68 Revision: 405803 Index: src/java/org/apache/commons/httpclient/URI.java =================================================================== --- src/java/org/apache/commons/httpclient/URI.java (revision 405803) +++ src/java/org/apache/commons/httpclient/URI.java (working copy) @@ -1552,6 +1552,7 @@ allowed_abs_path.or(abs_path); // allowed_abs_path.set('/'); // aleady included allowed_abs_path.andNot(percent); + allowed_abs_path.clear('+'); } @@ -1563,6 +1564,7 @@ static { allowed_rel_path.or(rel_path); allowed_rel_path.clear('%'); + allowed_rel_path.clear('+'); } -- Configure bugmail: http://issues.apache.org/bugzilla/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the assignee for the bug, or are watching the assignee. --------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
