Provide http-config option for curl low-speed-limit

Fixes #5717

low-speed-time was removed because the corresponding curl-option is already set via http.timeout.

removed unnecessary Option.

Author: lostiniceland

src/cargo/ops/registry.rs

@@ -367,12 +367,7 @@ pub fn configure_http_handle(config: &Config, handle: &mut Easy) -> CargoResult<
     // connect phase as well as a "low speed" timeout so if we don't receive
     // many bytes in a large-ish period of time then we time out.
     handle.connect_timeout(Duration::new(30, 0))?;
-    if let Some(config_low_speed_limit) = low_speed_limit(config)? {
-        handle.low_speed_limit(config_low_speed_limit)?;
-    }
-    if let Some(config_low_speed_time) = low_speed_time(config)? {
-        handle.low_speed_time(config_low_speed_time)?;
-    }
+    handle.low_speed_limit(http_low_speed_limit(config)?)?;
     if let Some(proxy) = http_proxy(config)? {
         handle.proxy(&proxy)?;
     }
@@ -395,19 +390,11 @@ pub fn configure_http_handle(config: &Config, handle: &mut Easy) -> CargoResult<
 }
 
 /// Find an override from config for curl low-speed-limit option, otherwise use default value
-fn low_speed_limit(config: &Config) -> CargoResult<Option<u32>> {
+fn http_low_speed_limit(config: &Config) -> CargoResult<u32> {
     if let Some(s) = config.get::<Option<u32>>("http.low-speed-limit")? {
-        return Ok(Some(s));
-    }
-    Ok(Some(10))
-}
-
-/// Find an override from config for curl low-speed-time option, otherwise use default value
-fn low_speed_time(config: &Config) -> CargoResult<Option<Duration>> {
-    if let Some(s) = config.get::<Option<u64>>("http.low-speed-time")? {
-        return Ok(Some(Duration::new(s, 0)));
+        return Ok(s);
     }
-    Ok(Some(Duration::new(30, 0)))
+    Ok(10)
 }
 
 /// Find an explicit HTTP proxy if one is available.

src/doc/src/reference/config.md

@@ -95,6 +95,7 @@ proxy = "host:port" # HTTP proxy to use for HTTP requests (defaults to none)
 timeout = 60000     # Timeout for each HTTP request, in milliseconds
 cainfo = "cert.pem" # Path to Certificate Authority (CA) bundle (optional)
 check-revoke = true # Indicates whether SSL certs are checked for revocation
+low-speed-limit = 0 # Lower threshold for bytes/sec (10 = default, 0 = disabled)
 
 [build]
 jobs = 1                  # number of parallel jobs, defaults to # of CPUs