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=42156>.
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=42156





------- Additional Comments From [EMAIL PROTECTED]  2007-04-18 04:09 -------
Created an attachment (id=19986)
 --> (http://issues.apache.org/bugzilla/attachment.cgi?id=19986&action=view)
Suggested patch

This patch urldecodes parameter value if the httpargument.isAlwaysEncode is
false, so that when it is added to the httpclient postmethod, where it is
urlencoded, the value will be correct.

There is also an commented out section, which uses the
HTTPSamplerBase.getQueryString to construct the post body. If this alternative
is used, then HTTPSampler and HTTPSampler2 will always send the same POST body.

I do not know which alternative you prefer.

The patch also contains more unit tests for
TestHTTPSamplersAgainstHttpMirrorServer, it contains the new tests suggested in
patch for bug 42139, plus some more, to expose this bug.
So if this patch for this bug is applied, then bug 42139 should just be closed.

If only the unit tests are applied, then you will see that one test will fail
for HTTPSampler2, the one with 
titleValue =
"%2FwEPDwULLTE2MzM2OTA0NTYPZBYCAgMPZ%2FrA%2B8DZ2dnZ2dnZ2d%2FGNDar6OshPwdJc%3D";


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