proxy: use the same data for CONNECT and Host header

The CONNECT and Host header should both point to the proxied host that we want to eventually reach, not to the proxy itself.
algesten · Jan 30, 2025 · 89d593c · 89d593c
1 parent e2e27ea
commit 89d593c
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 1 deletion.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -1,5 +1,6 @@
 # Unreleased
 
+  * Use the same data in CONNECT and Host header for proxied requests (#967)
   * Set default scheme in proxy uri (#966)
   * Redact URI and Location header on debug level (#964)
   * Downgrade all logging to debug and below (#964)

diff --git a/src/proxy.rs b/src/proxy.rs
@@ -244,7 +244,7 @@ impl<In: Transport> Connector<In> for ConnectProxyConnector {
             .unwrap_or(proxied.scheme().unwrap().default_port().unwrap());
 
         write!(w, "CONNECT {}:{} HTTP/1.1\r\n", proxied_host, proxied_port)?;
-        write!(w, "Host: {}:{}\r\n", proxy.host(), proxy.port())?;
+        write!(w, "Host: {}:{}\r\n", proxied_host, proxied_port)?;
         if let Some(v) = details.config.user_agent().as_str(DEFAULT_USER_AGENT) {
             write!(w, "User-Agent: {}\r\n", v)?;
         }