Merge pull request bottlerocket-os#3667 from etungsten/huhcat

pluto: remove unnecessary auth header in hyper-proxy
yeazelm · Dec 8, 2023 · a7be9c3 · a7be9c3
2 parents 1ebf2df + 8e4869d
commit a7be9c3
Showing 1 changed file with 3 additions and 6 deletions.
diff --git a/sources/api/pluto/src/hyper_proxy/mod.rs b/sources/api/pluto/src/hyper_proxy/mod.rs
@@ -126,15 +126,12 @@ impl Proxy {
 
     /// Set `Proxy` authorization
     pub fn set_authorization<C: Credentials + Clone>(&mut self, credentials: Authorization<C>) {
+        // In pluto, we use custom intercept for HTTPS traffic we might proxy based on the no proxy specification.
         match self.intercept {
-            Intercept::Https => {
-                self.headers.typed_insert(ProxyAuthorization(credentials.0));
-            }
-            _ => {
-                self.headers
-                    .typed_insert(Authorization(credentials.0.clone()));
+            Intercept::Custom(_) | Intercept::Https => {
                 self.headers.typed_insert(ProxyAuthorization(credentials.0));
             }
+            _ => {}
         }
     }
 }