curben
/
cloudflared-mirror
mirror of https://github.com/cloudflare/cloudflared.git
1
0
Fork 0
Code Issues Projects Releases Wiki Activity

Merge 2d7b01259d into 0d2a7a0385

This commit is contained in:
Çağrı 2025-11-22 22:58:30 -05:00 committed by GitHub
parent 0d2a7a0385 2d7b01259d
commit 548000c0ff
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
1 changed files with 4 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
connection/quic_connection.go
View File

@ -377,7 +377,7 @@ func buildHTTPRequest(
// * the method doesn't usually have a body (GET, HEAD, DELETE, ...) // * the method doesn't usually have a body (GET, HEAD, DELETE, ...)
// * there is no transfer-encoding=chunked already set. // * there is no transfer-encoding=chunked already set.
// So, if transfer cannot be chunked and content length is 0, we dont set a request body. // So, if transfer cannot be chunked and content length is 0, we dont set a request body.
if !isWebsocket && !isTransferEncodingChunked(req) && req.ContentLength == 0 { if !isWebsocket && !isEncodingChunked(req) && req.ContentLength == 0 {
req.Body = http.NoBody req.Body = http.NoBody
} }
stripWebsocketUpgradeHeader(req) stripWebsocketUpgradeHeader(req)
@ -395,11 +395,12 @@ func setContentLength(req *http.Request) error {
return err return err
} }
func isTransferEncodingChunked(req *http.Request) bool { func isEncodingChunked(req *http.Request) bool {
contentEncodingVal := req.Header.Get("Content-Encoding") // AWS S3 uses Content-Encoding: aws-chunked
transferEncodingVal := req.Header.Get("Transfer-Encoding") transferEncodingVal := req.Header.Get("Transfer-Encoding")
// https://developer.mozilla.org/en-US/docs/Web/HTTP/Headers/Transfer-Encoding suggests that this can be a comma // https://developer.mozilla.org/en-US/docs/Web/HTTP/Headers/Transfer-Encoding suggests that this can be a comma
// separated value as well. // separated value as well.
return strings.Contains(strings.ToLower(transferEncodingVal), "chunked") return strings.Contains(strings.ToLower(transferEncodingVal), "chunked") || strings.Contains(strings.ToLower(contentEncodingVal), "chunked")
} }
// A helper struct that guarantees a call to close only affects read side, but not write side. // A helper struct that guarantees a call to close only affects read side, but not write side.