Re: Handling of redirects in HttpClient

2013-08-07 Thread Oleg Kalnichevski
On Tue, 2013-08-06 at 22:58 +0200, Philippe Mouawad wrote: Hello, We have an issue with redirection handling in HttpClient 4.2.5 We have a JSP that contains the following code: %@ page session=false contentType=text/html buffer=8kb % html body %

Re: Handling of redirects in HttpClient

2013-08-07 Thread Philippe Mouawad
Hello Oleg, Sorry it's not clear. Using Httpclient, we call this jsp which ad you can see does a redirection with unsafz characters in url, redirection fails with the mentionned exception. This is due to the way hc4 impl builds a Uri from it not checking for unsafe characters. my question is , is

Re: Handling of redirects in HttpClient

2013-08-07 Thread Oleg Kalnichevski
On Wed, 2013-08-07 at 09:41 +0200, Philippe Mouawad wrote: Hello Oleg, Sorry it's not clear. Using Httpclient, we call this jsp which ad you can see does a redirection with unsafz characters in url, redirection fails with the mentionned exception. This is due to the way hc4 impl builds a Uri

Re: Handling of redirects in HttpClient

2013-08-07 Thread Philippe Mouawad
Hello Oleg, In the example Url contained too much garbage to see if all issues were fixed. You can see whole context here: - https://issues.apache.org/bugzilla/show_bug.cgi?id=54482 It seems encoding of these characters is not mandatory although rfc says it's better to encode. So this could be

Re: Handling of redirects in HttpClient

2013-08-07 Thread Oleg Kalnichevski
On Wed, 2013-08-07 at 10:54 +0200, Philippe Mouawad wrote: Hello Oleg, In the example Url contained too much garbage to see if all issues were fixed. You can see whole context here: - https://issues.apache.org/bugzilla/show_bug.cgi?id=54482 It seems encoding of these characters is not

Handling of redirects in HttpClient

2013-08-06 Thread Philippe Mouawad
Hello, We have an issue with redirection handling in HttpClient 4.2.5 We have a JSP that contains the following code: %@ page session=false contentType=text/html buffer=8kb % html body % response.sendRedirect(http://localhost:8080/?[]!@#$%^*()); % /body /html As you can see it contains