[
https://issues.apache.org/jira/browse/HTTPCLIENT-1923?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16471839#comment-16471839
]
ASF subversion and git services commented on HTTPCLIENT-1923:
-------------------------------------------------------------
Commit c300adb4b05e02bd7bff2bfd24d52fa22ae70c1b in httpcomponents-client's
branch refs/heads/4.6.x from Aleksei Arsenev
[ https://git-wip-us.apache.org/repos/asf?p=httpcomponents-client.git;h=c300adb
]
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: [email protected]
For additional commands, e-mail: [email protected]
