Auto merge of #11292 - arlosi:compression, r=epage

bors · bors · commit 1985caf190b6 · 2022-10-27T01:37:25.000Z
Add Accept-Encoding request header to enable compression

### What does this PR try to resolve?

Cargo does not request compression from servers. Enabling compression can save bandwidth and improve performance.

### How should we test and review this PR?

I validated locally that the header was being sent using a local proxy (Fiddler). It sent `Accept-Encoding: deflate, gzip` on Windows.
diff --git a/src/cargo/ops/registry.rs b/src/cargo/ops/registry.rs
@@ -643,6 +643,9 @@ pub fn configure_http_handle(config: &Config, handle: &mut Easy) -> CargoResult<
         handle.useragent(&format!("cargo {}", version()))?;
     }
 
+    // Empty string accept encoding expands to the encodings supported by the current libcurl.
+    handle.accept_encoding("")?;
+
     fn to_ssl_version(s: &str) -> CargoResult<SslVersion> {
         let version = match s {
             "default" => SslVersion::Default,

Original file line number	Diff line number	Diff line change
`@@ -643,6 +643,9 @@ pub fn configure_http_handle(config: &Config, handle: &mut Easy) -> CargoResult<`
`643`	`643`	`handle.useragent(&format!("cargo {}", version()))?;`
`644`	`644`	`}`
`645`	`645`
	`646`	`+ // Empty string accept encoding expands to the encodings supported by the current libcurl.`
	`647`	`+ handle.accept_encoding("")?;`
	`648`	`+`
`646`	`649`	`fn to_ssl_version(s: &str) -> CargoResult<SslVersion> {`
`647`	`650`	`let version = match s {`
`648`	`651`	`"default" => SslVersion::Default,`