phillipleblanc commented on code in PR #6843:
URL: https://github.com/apache/arrow-rs/pull/6843#discussion_r1874350947
##########
object_store/src/client/mod.rs:
##########
@@ -671,6 +671,10 @@ impl ClientOptions {
builder = builder.danger_accept_invalid_certs(true)
}
+ // Reqwest will remove the `Content-Length` header if it is configured
to
+ // transparently decompress the body via the non-default `gzip`
feature.
+ builder = builder.no_gzip();
Review Comment:
Yes, that is correct - sorry I could have made this clearer.
All this affects is what happens when the response has the header
`Content-Encoding: gzip`, which HTTP servers will usually only do when the
request has the header `Accept-Encoding: gzip`. If that is the case, then
reqwest will transparently decode the body as a gzip stream and remove the
`Content-Length` header (if the `gzip` feature is enabled)
For object store APIs, it will just return the bytes of the object as they
are (including objects that are gzipped).
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
