Add --noproxy argument to ignore proxy for hosts or IP addresses.

Related issue: ducaale#296

Author: Stefan Zwanenburg

src/cli.rs

@@ -295,6 +295,14 @@ Example: --print=Hb"
     #[clap(long, value_name = "PROTOCOL:URL", number_of_values = 1)]
     pub proxy: Vec<Proxy>,
 
+    /// Comma-separated list of hosts for which not to use a proxy, if one is specified.
+    ///
+    /// - A '*' matches all hosts, and effectively disables proxies altogether.
+    /// - IP addresses are allowed, as are subnets (in CIDR notation, i.e.: '127.0.0.0/8').
+    /// - Any other entry in the list is assumed to be a hostname.
+    #[clap(long, value_name = "no-proxy-list", value_delimiter = ',')]
+    pub noproxy: Vec<String>,
+
     /// If "no", skip SSL verification. If a file path, use it as a CA bundle.
     ///
     /// Specifying a CA bundle will disable the system's built-in root certificates.
@@ -1087,6 +1095,24 @@ impl FromStr for Proxy {
     }
 }
 
+impl Proxy {
+    pub fn into_reqwest_proxy(self, noproxy: &[String]) -> anyhow::Result<reqwest::Proxy> {
+        let proxy = match self {
+            Proxy::Http(url) => reqwest::Proxy::http(url),
+            Proxy::Https(url) => reqwest::Proxy::https(url),
+            Proxy::All(url) => reqwest::Proxy::all(url),
+        }?;
+
+        let mut noproxy_comma_delimited = noproxy.join(",");
+        if noproxy.contains(&"*".to_string()) {
+            // reqwest's NoProxy wildcard doesn't apply to IP addresses, while curl's does
+            noproxy_comma_delimited.push_str(",0.0.0.0/0,::/0");
+        }
+
+        Ok(proxy.no_proxy(reqwest::NoProxy::from_string(&noproxy_comma_delimited)))
+    }
+}
+
 #[derive(Debug, Clone)]
 pub struct Resolve {
     pub domain: String,

src/main.rs

@@ -41,7 +41,7 @@ use utils::reason_phrase;
 
 use crate::auth::{Auth, DigestAuthMiddleware};
 use crate::buffer::Buffer;
-use crate::cli::{Cli, FormatOptions, HttpVersion, Print, Proxy, Verify};
+use crate::cli::{Cli, FormatOptions, HttpVersion, Print, Verify};
 use crate::download::{download_file, get_file_size};
 use crate::middleware::ClientWithMiddleware;
 use crate::printer::Printer;
@@ -274,11 +274,7 @@ fn run(args: Cli) -> Result<i32> {
     }
 
     for proxy in args.proxy.into_iter().rev() {
-        client = client.proxy(match proxy {
-            Proxy::Http(url) => reqwest::Proxy::http(url),
-            Proxy::Https(url) => reqwest::Proxy::https(url),
-            Proxy::All(url) => reqwest::Proxy::all(url),
-        }?);
+        client = client.proxy(proxy.into_reqwest_proxy(&args.noproxy)?);
     }
 
     client = match args.http_version {

src/to_curl.rs

@@ -233,6 +233,10 @@ pub fn translate(args: Cli) -> Result<Command> {
             }
         }
     }
+    if !args.noproxy.is_empty() {
+        cmd.arg("--noproxy");
+        cmd.arg(args.noproxy.join(","));
+    }
     if let Some(timeout) = args.timeout.and_then(|t| t.as_duration()) {
         cmd.arg("--max-time");
         cmd.arg(timeout.as_secs_f64().to_string());

tests/cli.rs

@@ -1136,6 +1136,53 @@ fn proxy_multiple_valid_proxies() {
     cmd.assert().success();
 }
 
+fn noproxy_test(noproxy_arg: &str) {
+    let mut proxy_server = server::http(|_| async move {
+        hyper::Response::builder()
+            .status(200)
+            .body("Proxy shouldn't have been used.".into())
+            .unwrap()
+    });
+    let actual_server = server::http(|_| async move {
+        hyper::Response::builder()
+            .status(200)
+            .body("".into())
+            .unwrap()
+    });
+
+    get_command()
+        .arg(format!("--proxy=http:{}", proxy_server.base_url()))
+        .arg(format!("--noproxy={}", noproxy_arg))
+        .arg("GET")
+        .arg(actual_server.base_url().as_str())
+        .assert()
+        .success();
+
+    proxy_server.disable_hit_checks();
+    proxy_server.assert_hits(0);
+    actual_server.assert_hits(1);
+}
+
+#[test]
+fn noproxy_wildcard() {
+    noproxy_test("*");
+}
+
+#[test]
+fn noproxy_ip() {
+    noproxy_test("127.0.0.1");
+}
+
+#[test]
+fn noproxy_ip_cidr() {
+    noproxy_test("127.0.0.0/8");
+}
+
+#[test]
+fn noproxy_multiple() {
+    noproxy_test("127.0.0.2,127.0.0.1");
+}
+
 // temporarily disabled for builds not using rustls
 #[cfg(all(feature = "online-tests", feature = "rustls"))]
 #[ignore = "endpoint is randomly timing out"]