Cory Benfield 1aec5d7d0e Add defensive connection closure. (#328) ...

Motivation:

Currently when either we or the server send Connection: close, we
correctly do not return that connection to the pool. However, we rely on
the server actually performing the connection closure: we never call
close() ourselves. This is unnecessarily optimistic: a server may
absolutely fail to close this connection. To protect our own file
descriptors, we should make sure that any connection we do not return
the pool is closed.

Modifications:

If we think a connection is closing when we release it, we now call
close() on it defensively.

Result:

We no longer leak connections when the server fails to close them.

Fixes #324.

2021-01-19 17:27:09 +00:00

..

AsyncHTTPClient

Add defensive connection closure. (#328)

2021-01-19 17:27:09 +00:00