[ https://issues.apache.org/jira/browse/HTTPCLIENT-1923?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16471836#comment-16471836 ]
ASF subversion and git services commented on HTTPCLIENT-1923: ------------------------------------------------------------- Commit eb27f9ee2946e6aa0741fa089e1188b38fe3c530 in httpcomponents-client's branch refs/heads/4.5.x from Aleksei Arsenev [ https://git-wip-us.apache.org/repos/asf?p=httpcomponents-client.git;h=eb27f9e ] HTTPCLIENT-1923: fixed incorrect connection close on shutdown + fixed corresponding test > Not properly closing connections in BasicHttpClientConnectionManager > -------------------------------------------------------------------- > > Key: HTTPCLIENT-1923 > URL: https://issues.apache.org/jira/browse/HTTPCLIENT-1923 > Project: HttpComponents HttpClient > Issue Type: Bug > Components: HttpClient (classic) > Affects Versions: 4.5.5 > Reporter: Aleksei > Priority: Minor > > Seems we are hitting something similiar to the HTTPCLIENT-1655 issue, where > established connections were not properly closed (client sends Reset instead > of FIN+ACK) > We are using > BasicHttpClientConnectionManager (4.5.5 httpclient version) > and when we are calling client.close(), we see packet with Reset flag enabled > in tcpdump > > Looks like here: > [https://github.com/apache/httpcomponents-client/blob/4.5.x/httpclient/src/main/java/org/apache/http/impl/conn/BasicHttpClientConnectionManager.java#L378] > must be closeConnection() > [https://github.com/apache/httpcomponents-client/blob/4.5.x/httpclient/src/main/java/org/apache/http/impl/conn/BasicHttpClientConnectionManager.java#L209] > instead of shutdownConnection > > This issue is not present if we are using PoolingHttpClientConnectionManager > (in shutdown() there are pool.shutdown: > [https://github.com/apache/httpcomponents-client/blob/4.5.x/httpclient/src/main/java/org/apache/http/impl/conn/PoolingHttpClientConnectionManager.java#L410] > where for each entry runs: entry.close(){color:#cc7832} -> > this{color}.closeConnection(){color:#cc7832};{color}) - we see normal > connection close (FIN+ACK) > > What do you think about it? > Thank you -- This message was sent by Atlassian JIRA (v7.6.3#76005) --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@hc.apache.org For additional commands, e-mail: dev-h...@hc.apache.org