]> Cypherpunks repositories - gostls13.git/commit
net/http: set Content-Length:0 for empty PATCH requests as with POST, PATCH
authorSegflow <asselmeher@gmail.com>
Mon, 24 Aug 2020 13:42:45 +0000 (13:42 +0000)
committerEmmanuel Odeke <emm.odeke@gmail.com>
Mon, 24 Aug 2020 16:53:02 +0000 (16:53 +0000)
commitfb5c3eabd16bbeea28ee8a11d29a31cf5ed124dd
tree1d807a3fddceb63edb1e836e469d06692cfd6294
parent494ec85d9fd6deea388c454dfd836d9f845f7153
net/http: set Content-Length:0 for empty PATCH requests as with POST, PATCH

Sets Content-Length:0 for nil bodies in PATCH requests, as we already do for  POST and PUT requests.

RFC 2616 mentions that unless a method’s Content-Length is forbidden it can send one.
In the wild, we’ve found that Microsoft Azure’s DataLake Gen2 storage API https://docs.microsoft.com/en-us/rest/api/storageservices/datalakestoragegen2/path/update deliberately rejects PATCH requests without a Content-Length, yet there is no workaround for setting that header when trying to flush the content of a file which was uploaded in a previous request.

Fixes #40978

Change-Id: Ib0a623b907d827a1c5ee431dca3c41024fa291c5
GitHub-Last-Rev: 12a3903f2bc22bcc4f5f8e2abcc3922b612b8871
GitHub-Pull-Request: golang/go#40991
Reviewed-on: https://go-review.googlesource.com/c/go/+/250039
Run-TryBot: Emmanuel Odeke <emm.odeke@gmail.com>
TryBot-Result: Gobot Gobot <gobot@golang.org>
Reviewed-by: Emmanuel Odeke <emm.odeke@gmail.com>
src/net/http/requestwrite_test.go
src/net/http/transfer.go