Bug 60928

Summary: Http Request : Connection Leak when keepalive is used with Embedded Resources
Product: JMeter - Now in Github Reporter: Philippe Mouawad <p.mouawad>
Component: HTTPAssignee: JMeter issues mailing list <issues>
Status: RESOLVED FIXED    
Severity: normal CC: p.mouawad
Priority: P2    
Version: 2.9   
Target Milestone: ---   
Hardware: All   
OS: All   

Description Philippe Mouawad 2017-03-28 10:11:18 UTC
Doing a load test with embedded resources I faced the following issue:

    I have a GET (https://www.foo.com/fragment/ajax/get/identifier/dynamic/) that returns JSON with wrong Content Type (text/html instead of json)
    HTML Parser parses the response that contains escaped html inside JSON so it succeeds extracting links and builds 2 links:
        https://www.foo.com/fragment/ajax/get/identifier/dynamic/%5C%22https:%5C/%5C/www.foo.com%5C/mediap%5C/62%5C/6%5C/3%5C/a%5C/9%5C/10110954_fo_01_web.jpg%5C%22
        https://www.foo.com/fragment/ajax/get/identifier/dynamic/%5C%22https:%5C/%5C/www.foo.com%5C/mediap%5C/62%5C/c%5C/4%5C/9%5C/d%5C/10361071_fo_01_web.jpg%5C%22
    Response to this new request will return the same response so we end up being recursive

Now here is the problem:

    For some recursive processing I end up having this which seems ok:
        java.lang.Exception: Maximum frame/iframe nesting depth exceeded.
    But for some of the requests I end up having :
        Response message: Non HTTP response message: Timeout waiting for connection from pool
        Meaning the Http4 pool has reached the max number of connections, its size is set to the value of "Parallel downloads. Number" which is 6


I have another case, where I have 1 page that returns 1 html page with many links.
The 10th link download also fails with "Timeout waiting for connection from pool".
Comment 1 Philippe Mouawad 2017-03-28 10:15:15 UTC
Author: pmouawad
Date: Tue Mar 28 10:14:49 2017
New Revision: 1789082

URL: http://svn.apache.org/viewvc?rev=1789082&view=rev
Log:
Bug 60928 - Http Request : Connection Leak when keepalive is used with Embedded Resources
Bugzilla Id: 60928

Modified:
    jmeter/trunk/src/protocol/http/org/apache/jmeter/protocol/http/sampler/HTTPHC4Impl.java
    jmeter/trunk/xdocs/changes.xml
Comment 2 Philippe Mouawad 2017-03-28 11:30:44 UTC
Author: pmouawad
Date: Tue Mar 28 11:30:13 2017
New Revision: 1789097

URL: http://svn.apache.org/viewvc?rev=1789097&view=rev
Log:
Fix code smell

Modified:
    jmeter/trunk/src/protocol/http/org/apache/jmeter/protocol/http/sampler/HTTPHC4Impl.java
Comment 3 The ASF infrastructure team 2022-09-24 20:38:08 UTC
This issue has been migrated to GitHub: https://github.com/apache/jmeter/issues/4348