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





------- Additional Comments From [EMAIL PROTECTED]  2006-05-12 16:00 -------
(In reply to comment #2)
> This constructor is flawed hopelessly. We should deprecate it.
> There is no way to escape a non-escaped path after composing the parts with
> reserved characters like /.

Isn't that what the elaborate series of character BitSets is for in URI.java, to
determine which characters to escape in what situations and contexts?  Is there
a problem with the patch that I provided?

> As a workaround use 
> 
> HttpURL httpURL = new HttpURL("http://localhost/test+test".toCharArray());

OK, however according to this constructor I need to escape my input myself. 
This is fine, however the main place where this bug is manifesting itself for me
is in webdav slide client (WebdavResource.setPath() which delegates to
HttpURL.setPath()).  Therefore, the constructor is not the only source of pain
for me.  Again, my patch seems to fix this properly internally so the both the
constructor and setPath() work properly.  Am I missing something?  Does the
patch break other stuff?

-- 
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]

Reply via email to