elixir-mint / mint / bfc6454f92ade5bb7afeede28b3f741378fc985c

88%

push

github

Close TCP sockets on errors in HTTP/1 (#502)

In some cases, when handling a failure, Mint would only set the conn
state to :closed but would not close the underlying socket. These
sockets stay open until the calling process that owns them exits, and
may therefore accumulate until resources are exhausted.

Every relevant %{conn | state: :closed} inside Mint.HTTP1 is replaced
with internal_close that in addition to setting the state also closes
the socket.

Tests that asserted closed conn now also assert that the socket is
closed. New tests that demonstrate the problem are added. They would
fail without the fix and they pass with the fix in place.

Fixes #501.

6 of 8 new or added lines in 1 file covered. (75.0%)

1502 of 1700 relevant lines covered (88.35%)

517.46 hits per line