Mailing-List: contact dev-help@jmeter.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@jmeter.apache.org
MIME-Version: 1.0
In-Reply-To: <CAH9fUpZEx3TwXpkcPEkdc8fX+x4Nfvh13hDOfjMUQ9+QkKQ6wQ@mail.gmail.com>
References: <20170825190239.B559C3A009D@svn01-us-west.apache.org>
 <CAH9fUpa-h+jSX-8ZWJ3SNdymLTMsn_Chokz69QiyN7yKvV6KSQ@mail.gmail.com>
 <2BE5B043-868E-44DD-9912-00CFAC33D798@internetallee.de> <CAH9fUpZaTd1A7s1oneg_55yEm0enXMxuX6MK9pjP9G9f6nvgDQ@mail.gmail.com>
 <CAH9fUpbuFKfR1Md509TPwwpox0qZKhRdcH45gSNPEJZCq=6r8A@mail.gmail.com> <CAH9fUpZEx3TwXpkcPEkdc8fX+x4Nfvh13hDOfjMUQ9+QkKQ6wQ@mail.gmail.com>
From: Philippe Mouawad <philippe.mouawad@gmail.com>
Date: Sun, 27 Aug 2017 22:23:20 +0200
Message-ID: <CAH9fUpYHq4NiGfWcHTAWF8naRwfSWo--7PBoMf-cyQqPECzP4Q@mail.gmail.com>
Subject: Re: svn commit: r1806215 - in /jmeter/trunk: src/protocol/http/org/apache/jmeter/protocol/http/sampler/HTTPHC4Impl.java
 xdocs/changes.xml
To: "dev@jmeter.apache.org" <dev@jmeter.apache.org>
Content-Type: multipart/alternative; boundary="001a114106780276380557c1f37f"
archived-at: Sun, 27 Aug 2017 20:23:34 -0000

--001a114106780276380557c1f37f
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable

Might be interesting :
https://stackoverflow.com/questions/20591599/why-arent-post-names-with-unic=
ode-sent-correctly-when-using-multipart-form-data/20592910#20592910

On Sun, Aug 27, 2017 at 8:04 PM, Philippe Mouawad <
philippe.mouawad@gmail.com> wrote:

> Hi Felix,
> I noticed while testing that Java Implementation also corrupts Parameter
> name, so I think it's a bug also, but  I have a doubt regarding the
> encoding of parameter names, does RFC force a particular encoding for the=
m
> (US-ASCII) or can they be encoded in whatever encoding we want ?
>
> If you look at current code, I have added a check for Java Implementation
> to check for corrupt "?_param" instead of "=E5=AE=89_param"
>
> Please review and give your feedback.
>
> Thanks
>
> On Sun, Aug 27, 2017 at 2:41 PM, Philippe Mouawad <
> philippe.mouawad@gmail.com> wrote:
>
>> Hi Felix,
>> Look also at this report for Aka HTTP following their fix to
>> https://github.com/akka/akka-http/issues/338
>>
>>    - https://github.com/akka/akka-http/issues/647
>>
>> I confirmed current trunk has a similar issue, see
>> https://bz.apache.org/bugzilla/show_bug.cgi?id=3D61384#c6.
>>
>> So I committed my alternative patch, please review.
>>
>> Still , I don't think it fixes https://bz.apache.org/bugzilla
>> /show_bug.cgi?id=3D56141
>>
>>
>> Regards
>>
>> On Sun, Aug 27, 2017 at 12:21 PM, Philippe Mouawad <
>> philippe.mouawad@gmail.com> wrote:
>>
>>> Hi Felix,
>>> I attached an alternative patch which :
>>>
>>>    - set surrounding header only if we have a charset
>>>    - same for parameters
>>>
>>> I have asked a question on HC client mailing list:
>>>
>>>    - http://mail-archives.apache.org/mod_mbox/hc-httpclient-users
>>>    /201704.mbox/%3CCAH9fUpbxye8-rydo143Bk%3Dr6q2QDJTEndhPmd5GQ3
>>>    TjxtLpDxA%40mail.gmail.com%3E
>>>    <http://mail-archives.apache.org/mod_mbox/hc-httpclient-users/201704=
.mbox/%3CCAH9fUpbxye8-rydo143Bk%3Dr6q2QDJTEndhPmd5GQ3TjxtLpDxA%40mail.gmail=
.com%3E>
>>>
>>> I think the following bugs have potentially the same root cause:
>>>
>>>    - https://bz.apache.org/bugzilla/show_bug.cgi?id=3D61384
>>>    - https://bz.apache.org/bugzilla/show_bug.cgi?id=3D60800
>>>    - https://bz.apache.org/bugzilla/show_bug.cgi?id=3D56141
>>>
>>> See this interesting comment also:
>>>
>>>    - https://bz.apache.org/bugzilla/show_bug.cgi?id=3D56141#c4
>>>
>>>
>>> Regards
>>>
>>> On Sun, Aug 27, 2017 at 10:59 AM, Felix Schumacher <
>>> felix.schumacher@internetallee.de> wrote:
>>>
>>>>
>>>>
>>>> Am 26. August 2017 15:11:19 MESZ schrieb Philippe Mouawad <
>>>> philippe.mouawad@gmail.com>:
>>>> >Hi Felix,
>>>> >Are we sure that when encoding is UTF-8 there is no need to set chars=
et
>>>> >?
>>>>
>>>> We keep the charset. We only remove it from the surrounding header.
>>>>
>>>> >
>>>> >AFAIK, there were already issue with Multipart forms even before
>>>> >refactoring.
>>>>
>>>> Right. The most questions I found stated that they had problems when
>>>> the charset was set.
>>>>
>>>> What do you think would be the correct way?
>>>>
>>>> Felix
>>>>
>>>> >
>>>> >Thanks
>>>> >Thanks
>>>> >
>>>> >On Fri, Aug 25, 2017 at 9:02 PM, <fschumacher@apache.org> wrote:
>>>> >
>>>> >> Author: fschumacher
>>>> >> Date: Fri Aug 25 19:02:36 2017
>>>> >> New Revision: 1806215
>>>> >>
>>>> >> URL: http://svn.apache.org/viewvc?rev=3D1806215&view=3Drev
>>>> >> Log:
>>>> >> Don't set the charset on enclosing multipart/form-data header. It
>>>> >> irritates some servers.
>>>> >>
>>>> >> The charset was added sometime back while refactoring to use a newe=
r
>>>> >api
>>>> >> of http client.
>>>> >> See https://bz.apache.org/bugzilla/show_bug.cgi?id=3D56141 for more
>>>> >info.
>>>> >>
>>>> >> Bugzilla Id: 61384
>>>> >>
>>>> >>
>>>> >> Modified:
>>>> >>     jmeter/trunk/src/protocol/http/org/apache/jmeter/
>>>> >> protocol/http/sampler/HTTPHC4Impl.java
>>>> >>     jmeter/trunk/xdocs/changes.xml
>>>> >>
>>>> >> Modified: jmeter/trunk/src/protocol/http/org/apache/jmeter/
>>>> >> protocol/http/sampler/HTTPHC4Impl.java
>>>> >> URL: http://svn.apache.org/viewvc/jmeter/trunk/src/protocol/
>>>> >>
>>>> >http/org/apache/jmeter/protocol/http/sampler/HTTPHC4Impl.ja
>>>> va?rev=3D1806215&
>>>> >> r1=3D1806214&r2=3D1806215&view=3Ddiff
>>>> >> =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=
=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=
=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D
>>>> >> =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D
>>>> >> --- jmeter/trunk/src/protocol/http/org/apache/jmeter/
>>>> >> protocol/http/sampler/HTTPHC4Impl.java (original)
>>>> >> +++ jmeter/trunk/src/protocol/http/org/apache/jmeter/
>>>> >> protocol/http/sampler/HTTPHC4Impl.java Fri Aug 25 19:02:36 2017
>>>> >> @@ -1242,7 +1242,7 @@ public class HTTPHC4Impl extends HTTPHCA
>>>> >>          if(getUseMultipartForPost()) {
>>>> >>              // If a content encoding is specified, we use that as
>>>> >the
>>>> >>              // encoding of any parameter values
>>>> >> -            Charset charset =3D null;
>>>> >> +            Charset charset;
>>>> >>              if(haveContentEncoding) {
>>>> >>                  charset =3D Charset.forName(contentEncoding);
>>>> >>              } else {
>>>> >> @@ -1254,8 +1254,7 @@ public class HTTPHC4Impl extends HTTPHCA
>>>> >>                          getDoBrowserCompatibleMultipart(), charset=
,
>>>> >> haveContentEncoding);
>>>> >>              }
>>>> >>              // Write the request to our own stream
>>>> >> -            MultipartEntityBuilder multipartEntityBuilder =3D
>>>> >> MultipartEntityBuilder.create()
>>>> >> -                    .setCharset(charset);
>>>> >> +            MultipartEntityBuilder multipartEntityBuilder =3D
>>>> >> MultipartEntityBuilder.create();
>>>> >>              if(getDoBrowserCompatibleMultipart()) {
>>>> >>                  multipartEntityBuilder.setLaxMode();
>>>> >>              } else {
>>>> >>
>>>> >> Modified: jmeter/trunk/xdocs/changes.xml
>>>> >> URL: http://svn.apache.org/viewvc/jmeter/trunk/xdocs/changes.
>>>> >> xml?rev=3D1806215&r1=3D1806214&r2=3D1806215&view=3Ddiff
>>>> >> =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=
=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=
=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D
>>>> >> =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D
>>>> >> --- jmeter/trunk/xdocs/changes.xml [utf-8] (original)
>>>> >> +++ jmeter/trunk/xdocs/changes.xml [utf-8] Fri Aug 25 19:02:36 2017
>>>> >> @@ -167,6 +167,9 @@ Incorporated feed back about unclear doc
>>>> >>
>>>> >>  <h3>HTTP Samplers and Test Script Recorder</h3>
>>>> >>  <ul>
>>>> >> +  <li><bug>61384</bug>Don't set the charset on enclosing
>>>> >> <code>multipart/form-data</code> header. It irritates some
>>>> >servers.<br/>
>>>> >> +     The charset was added sometime back while refactoring to use =
a
>>>> >newer
>>>> >> api of http client.
>>>> >> +     See <bug>56141</bug> for more info.</li>
>>>> >>  </ul>
>>>> >>
>>>> >>  <h3>Other Samplers</h3>
>>>> >>
>>>> >>
>>>> >>
>>>>
>>>
>>>
>>>
>>> --
>>> Cordialement.
>>> Philippe Mouawad.
>>>
>>>
>>>
>>
>>
>> --
>> Cordialement.
>> Philippe Mouawad.
>>
>>
>>
>
>
> --
> Cordialement.
> Philippe Mouawad.
>
>
>


--=20
Cordialement.
Philippe Mouawad.

--001a114106780276380557c1f37f--