jmeter-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Philippe Mouawad <p.moua...@ubik-ingenierie.com>
Subject Re: Regression in JMeter 3.0 : Compressed response break keepalive
Date Wed, 11 May 2016 20:43:30 GMT
Hello,
I created this to track issue:
https://bz.apache.org/bugzilla/show_bug.cgi?id=59489

Regards

On Wed, May 11, 2016 at 10:30 PM, Philippe Mouawad <
p.mouawad@ubik-ingenierie.com> wrote:

> Hello,
> We have an issue in current 3.0 rc4 as seen by Rainer :
> - https://issues.apache.org/jira/browse/HTTPCLIENT-1742
>
> The issue is due to a behavioral change in HttpClient 4.5.2 vs
> httpclient-4.2.6:
>
>    - ResponseContentEncoding removes 3 headers from Response :
>    Content-Encoding, Content-Length, Content-MD5
>    - JMeter uses DefaultRequestDirector which is replaced by
>    MainClientExec
>
>
> Now here is what happens:
>
> At this line, PostProcessor run , leading to removal of 3 headers
>
>    -
>    https://github.com/apache/httpclient/blob/4.5.x/httpclient/src/main/java-deprecated/org/apache/http/impl/client/DefaultRequestDirector.java#L496
>
> At the next line, keepAlive management is run, but it's too late as
> headers have been removed:
>
>    -
>    https://github.com/apache/httpclient/blob/4.5.x/httpclient/src/main/java-deprecated/org/apache/http/impl/client/DefaultRequestDirector.java#L500
>
>
> While in new HC Class ProtocolExec:
>
> The keepAlive is managed in this
>
>
> https://github.com/apache/httpclient/blob/4.5.x/httpclient/src/main/java/org/apache/http/impl/execchain/ProtocolExec.java#L184
>
> Which calls this:
>
>
> https://github.com/apache/httpclient/blob/4.5.x/httpclient/src/main/java/org/apache/http/impl/execchain/MainClientExec.java#L274
>
> And Compression is managed afterwards in :
>
>
> https://github.com/apache/httpclient/blob/4.5.x/httpclient/src/main/java/org/apache/http/impl/execchain/ProtocolExec.java#L189
>
>
> So no issue in HC4.5.2, but in JMeter. I think it's unfortunately a
> stopper for the release.
>
> Breaking KeepAlive is a big issue
>
> --
> Regards.
> Philippe
>



-- 
Cordialement.
Philippe Mouawad.
Ubik-Ingénierie

UBIK LOAD PACK Web Site <http://www.ubikloadpack.com/>

UBIK LOAD PACK on TWITTER <https://twitter.com/ubikloadpack>

Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message