fix(client): Proxy Matcher to handle proxy auth without password (#241)

Author: karanabe

src/client/proxy/matcher.rs

@@ -331,6 +331,8 @@ fn get_first_env(names: &[&str]) -> String {
 }
 
 fn parse_env_uri(val: &str) -> Option<Intercept> {
+    use std::borrow::Cow;
+
     let uri = val.parse::<http::Uri>().ok()?;
     let mut builder = http::Uri::builder();
     let mut is_httpish = false;
@@ -358,13 +360,19 @@ fn parse_env_uri(val: &str) -> Option<Intercept> {
     let authority = uri.authority()?;
 
     if let Some((userinfo, host_port)) = authority.as_str().split_once('@') {
-        let (user, pass) = userinfo.split_once(':')?;
+        let (user, pass) = match userinfo.split_once(':') {
+            Some((user, pass)) => (user, Some(pass)),
+            None => (userinfo, None),
+        };
         let user = percent_decode_str(user).decode_utf8_lossy();
-        let pass = percent_decode_str(pass).decode_utf8_lossy();
+        let pass = pass.map(|pass| percent_decode_str(pass).decode_utf8_lossy());
         if is_httpish {
-            auth = Auth::Basic(encode_basic_auth(&user, Some(&pass)));
+            auth = Auth::Basic(encode_basic_auth(&user, pass.as_deref()));
         } else {
-            auth = Auth::Raw(user.into(), pass.into());
+            auth = Auth::Raw(
+                user.into_owned(),
+                pass.map_or_else(String::new, Cow::into_owned),
+            );
         }
         builder = builder.authority(host_port);
     } else {
@@ -822,6 +830,19 @@ mod tests {
         );
     }
 
+    #[test]
+    fn test_parse_http_auth_without_password() {
+        let p = p! {
+            all = "http://Aladdin@y.ep",
+        };
+        let proxy = intercept(&p, "https://example.local");
+        assert_eq!(proxy.uri(), "http://y.ep");
+        assert_eq!(
+            proxy.basic_auth().expect("basic_auth"),
+            "Basic QWxhZGRpbjo="
+        );
+    }
+
     #[test]
     fn test_parse_http_auth_without_scheme() {
         let p = p! {