On Fri, Apr 12, 2013 at 11:06 PM, sebb <[email protected]> wrote: > On 12 April 2013 21:17, Philippe Mouawad <[email protected]> > wrote: > > > On Fri, Apr 12, 2013 at 12:11 PM, sebb <[email protected]> wrote: > > > > > On 11 April 2013 22:15, Philippe Mouawad <[email protected]> > > > wrote: > > > > > > > Hello, > > > > > > > > I wonder what's the best option to fix these 3 bugs: > > > > > > > > - 54482- HC fails to follow redirects with non-encoded chars > > > > > > > > > > Is that a bug in HC? > > > > > > > > > > No I don't think so. All is due to non clean URLs and sanitize method > > fixes these. > > > > > What sanitize method? > > org.apache.jmeter.protocol.http.util.ConversionUtils#sanitizeUrl
> > > > > > > - 54293- JMeter rejects html tags '<' in query params as > invalid > > > when > > > > they are accepted by the browser > > > > > > > > > > I think that is a bug in IE; the other browsers all encode < and > > > > > > Same as before > > > > > Again, what method? > org.apache.jmeter.protocol.http.util.ConversionUtils#sanitizeUrl > > > > > > > > - 54142- HTTP Proxy Server throws an exception when path contains > > "|" > > > > character > > > > > > > > > > > Is it not possible to catch the Exception, and then try the > decode-encode > > > proposed method? > > > > > > > I also thought about this method, but it will badly impact performances > due > > to Exception throwing / Catching, don't you think ? > > > > > What does it matter if the ProxyServer is a bit slower? > > It's not used in performance tests. > > Misread your proposition. I thought you were proposing that in HttpHC3Impl#sample (as reverted code was there). Agree with this > > > Or maybe you think we could cache the result ? but what if there is some > > variable in it ? > > > > Not relevant. > > > > > > > Alternatively, if it is only | that causes the problem, maybe that > alone > > > could be fixed. > > > > > > In both cases, a warning should be logged. > > > > > > > > > Should we add a checkbox saying URL is sanitized or not as in fact: > > > > > > > > - Sanitizing everytime may break current test plan where recording > > > > through URL record ? > > > > - > > > > > > > > > > > > > > http://www.titi.fr/toto/CatalogData/ItemImages\IJ\Items_1152\07_06_015_Na_0_0_0_Aucune_0_Na_Na_Batik_s.jpg > > > > to > > > > > > > > > > > > > > > > > > http://www.titi.fr/toto/CatalogData/ItemImages%5CIJ%5CItems_1152%5C07_06_015_Na_0_0_0_Aucune_0_Na_Na_Batik_s.jpg > > > > - No sanitizing generates the bugs 54482, 54293, 54142 > > > > - Regarding redirect url, we would need to handle this within > > redirect > > > > processing > > > > > > > > > > > > Any ideas ? > > > > -- > > > > Regards > > > > Philippe > > > > > > > > > > > > > > > -- > > Cordialement. > > Philippe Mouawad. > > > -- Cordialement. Philippe Mouawad.
