[pve-devel] [PATCH proxmox-openid-rs] fix Open ID with Azure as provider
Mira Limbeck
m.limbeck at proxmox.com
Fri Apr 1 10:57:49 CEST 2022
On 4/1/22 06:21, Dietmar Maurer wrote:
>> let response = if let Method::POST = request.method {
>> - req.send(&*request.body)
>> + let bytes = request.body.as_slice();
>> + req.send_bytes(bytes)
> Does this have the side effect of changing the transfer encoding? If so, it is worth to add an inline comment.
This makes sure that `Transfer-Encoding` is not set at all, but instead
`Content-Length` is set.
AFAIK either one of those is required, but not both at the same time.
And some endpoints don't seem to accept `Transfer-Encoding: chunked`.
I'll add an inline comment mentioning that, thanks!
More information about the pve-devel
mailing list