Do not always require an authority

This fixes connections where a local UNIX domain socket path is provided, where the authority contains the full path to the *.sock file or is empty. Signed-off-by: Sascha Grunert <[email protected]>
hyperium · Oct 30, 2020 · 1f99ad3 · 1f99ad3
1 parent cbbdd30
commit 1f99ad3
Showing 1 changed file with 14 additions and 8 deletions.
diff --git a/src/server.rs b/src/server.rs
@@ -1395,14 +1395,20 @@ impl proto::Peer for Peer {
         // A request translated from HTTP/1 must not include the :authority
         // header
         if let Some(authority) = pseudo.authority {
-            let maybe_authority = uri::Authority::from_maybe_shared(authority.clone().into_inner());
-            parts.authority = Some(maybe_authority.or_else(|why| {
-                malformed!(
-                    "malformed headers: malformed authority ({:?}): {}",
-                    authority,
-                    why,
-                )
-            })?);
+            // When connecting to a UNIX Domain Socket (UDS), then we might get a path for the
+            // authority field. If it's a local path and exists, then we do not error in that case
+            // and assume an UDS.
+            if !authority.is_empty() && !authority.ends_with(".sock") {
+                let maybe_authority =
+                    uri::Authority::from_maybe_shared(authority.clone().into_inner());
+                parts.authority = Some(maybe_authority.or_else(|why| {
+                    malformed!(
+                        "malformed headers: malformed authority ({:?}): {}",
+                        authority,
+                        why,
+                    )
+                })?);
+            }
         }
 
         // A :scheme is required, except CONNECT.