gr1ffon/quic-go
1
0
Fork 0
forked from quic-go/quic-go
Code Pull Requests Activity

http3: reset stream if the server sends too many 1xx responses (#5144)

Browse Source
This commit is contained in:
Marten Seemann
2025-05-17 12:18:18 +08:00
committed by GitHub
parent 8ef0a71581
commit 06e8ee1bcf
2 changed files with 6 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
http3/client.go
View File

@@ -345,6 +345,8 @@ func (c *ClientConn) doRequest(req *http.Request, str *requestStream) (*http.Res
if is1xxNonTerminal {
num1xx++
if num1xx > max1xxResponses {
str.CancelRead(quic.StreamErrorCode(ErrCodeExcessiveLoad))
str.CancelWrite(quic.StreamErrorCode(ErrCodeExcessiveLoad))
return nil, errors.New("http3: too many 1xx informational responses")
}
traceGot1xxResponse(trace, resCode, textproto.MIMEHeader(res.Header))

4
http3/client_test.go
View File

@@ -412,6 +412,10 @@ func testClient1xxHandling(t *testing.T, numEarlyHints int, terminalStatus int,
str.EXPECT().Write(gomock.Any()).DoAndReturn(buf.Write).AnyTimes()
str.EXPECT().Close()
str.EXPECT().Read(gomock.Any()).DoAndReturn(bytes.NewReader(rspBytes).Read).AnyTimes()
if tooMany {
str.EXPECT().CancelRead(quic.StreamErrorCode(ErrCodeExcessiveLoad))
str.EXPECT().CancelWrite(quic.StreamErrorCode(ErrCodeExcessiveLoad))
}
conn.EXPECT().OpenStreamSync(gomock.Any()).Return(str, nil)
cc := (&Transport{}).NewClientConn(conn)