elixir-mint / mint / defd0a8c5ab9efdecd637dfafca94338a135d505-PR-529

90%

main: 91%

Build Type

Pull #529

github

Committed by ericmj

Commit Message

Fail requests pipelined behind a 101 response

After a 101 response the connection stayed open with the queued
requests in place, so bytes arriving later, which belong to the protocol
the server switched to (RFC 9110 15.2.2), were parsed as HTTP/1
responses to those requests.

Queued requests now fail with the new :connection_upgraded error when
the 101 response completes. Bytes arriving later are unexpected data as
long as no new request is made on the connection. A failed request that
was streaming its body can't stream the rest of it. :unprocessed isn't
used because the pipelined request bytes reached the server as part of
the new protocol, so the request might have been processed. The bytes
after the 101 response in the same packet are still delivered as data
for the upgrade request.

Pull Request Pull Request #529: Fail requests pipelined behind a 101 response

Coverage Stats

5 of 6 new or added lines in 1 file covered. (83.33%)

5 existing lines in 3 files now uncovered.

1819 of 2010 relevant lines covered (90.5%)

712.47 hits per line