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

http3: fix check for content length of the response (#3998)

Browse Source 
* fix: check response content-length other than request content-length

* Update http3/client.go

---------

Co-authored-by: Marten Seemann <martenseemann@gmail.com>
This commit is contained in:
roc
2023-07-29 12:18:26 +08:00
committed by GitHub
parent 469a6153b6
commit 32f8b20ae5
1 changed files with 2 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
http3/client.go
View File

@@ -429,8 +429,8 @@ func (c *client) doRequest(req *http.Request, conn quic.EarlyConnection, str qui
// Check that the server doesn't send more data in DATA frames than indicated by the Content-Length header (if set). // Check that the server doesn't send more data in DATA frames than indicated by the Content-Length header (if set).
// See section 4.1.2 of RFC 9114. // See section 4.1.2 of RFC 9114.
var httpStr Stream var httpStr Stream
if _, ok := req.Header["Content-Length"]; ok && req.ContentLength >= 0 { if _, ok := res.Header["Content-Length"]; ok && res.ContentLength >= 0 {
httpStr = newLengthLimitedStream(hstr, req.ContentLength) httpStr = newLengthLimitedStream(hstr, res.ContentLength)
} else { } else {
httpStr = hstr httpStr = hstr
} }