https://issues.apache.org/bugzilla/show_bug.cgi?id=54142

--- Comment #11 from Marek <[email protected]> ---
Created attachment 29742
  --> https://issues.apache.org/bugzilla/attachment.cgi?id=29742&action=edit
Corrected patch - double encoding problem

Hi,

I've noticed that there is problem with encoding url.
For example FireFox will encode spaces but will not encode unwise characters 
("|").
So there is danger of encoding something which was already encoded. To overcome
this problem I'm first decoding uri (path and query parts) and then encode
everything (including unwise characters).
Choice of encoding (UTF-8/ANSI-ASCII/...) is problematic, so defensively
someone should review my patch.

I've notice that this also can be fixed in:
org.apache.jmeter.protocol.http.sampler.HTTPSamplerBase.getUrl
it might be better place to fix it but for me it is hard to evaluate.

BR,

Marek

-- 
You are receiving this mail because:
You are the assignee for the bug.

Reply via email to