root
/
jmeter
mirror of https://github.com/apache/jmeter.git
1
0
Fork 0
Code Issues Actions 4 Packages Projects Releases Wiki Activity

Bug 60689 - "httpclient4.validate_after_inactivity" has no impact leading to usage of potentially stale/closed connections 

Bugzilla Id: 60689

git-svn-id: https://svn.apache.org/repos/asf/jmeter/trunk@1781673 13f79535-47bb-0310-9956-ffa450edef68

Former-commit-id: ed2dc1c609
This commit is contained in:
Philippe Mouawad 2017-02-04 14:28:03 +00:00
parent 77e3a9c270
commit 13556fda47
2 changed files with 11 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

11
src/protocol/http/org/apache/http/impl/conn/JMeterPoolingClientConnectionManager.java
View File

@ -106,7 +106,16 @@ public class JMeterPoolingClientConnectionManager implements ClientConnectionMan
this.schemeRegistry = schemeRegistry; this.schemeRegistry = schemeRegistry;
this.dnsResolver = dnsResolver; this.dnsResolver = dnsResolver;
this.operator = createConnectionOperator(schemeRegistry); this.operator = createConnectionOperator(schemeRegistry);
this.pool = new HttpConnPool(this.log, this.operator, 2, 20, timeToLive, tunit); this.pool = new HttpConnPool(this.log, this.operator, 2, 20, timeToLive, tunit) {
/**
* @see org.apache.http.pool.AbstractConnPool#validate(org.apache.http.pool.PoolEntry)
*/
@Override
protected boolean validate(HttpPoolEntry entry) {
return !entry.getConnection().isStale();
}
};
pool.setValidateAfterInactivity(validateAfterInactivity); pool.setValidateAfterInactivity(validateAfterInactivity);
} }

1
xdocs/changes.xml
View File

@ -204,6 +204,7 @@ JMeter now requires Java 8. Ensure you use most up to date version.
<li><bug>60575</bug>HTTP GET Requests could have a content-type header without a body.</li> <li><bug>60575</bug>HTTP GET Requests could have a content-type header without a body.</li>
<li><bug>60643</bug>HTTP(S) Test Script Recorder doesn't correctly handle restart or start after stop. Contributed by Ubik Load Pack (support at ubikloadpack.com)</li> <li><bug>60643</bug>HTTP(S) Test Script Recorder doesn't correctly handle restart or start after stop. Contributed by Ubik Load Pack (support at ubikloadpack.com)</li>
<li><bug>60652</bug>PUT might leak file descriptors.</li> <li><bug>60652</bug>PUT might leak file descriptors.</li>
<li><bug>60689</bug><code>httpclient4.validate_after_inactivity</code> has no impact leading to usage of potentially stale/closed connections</li>
</ul> </ul>
<h3>Other Samplers</h3> <h3>Other Samplers</h3>