move the nginx HTTP2 bug workaround

We remove the `host` header from the request if present because (some versions
of?) nginx have a known bug such that they can't handle `host` headers in HTTP/2
requests.  However, the request interceptor, if present, may have its own
opinions about whether a `host` header should be set, and if it insists on
setting one we should honor that.  So I've moved the workaround to run _before_
the interceptor is run.

Signed-off-by: Joel Dice <[email protected]>

Author: dicej

crates/factor-outbound-http/src/wasi.rs

@@ -425,6 +425,10 @@ async fn send_request_impl(
         *request.uri_mut() = origin.into_uri(path_and_query);
     }
 
+    // Some servers (looking at you nginx) don't like a host header even though
+    // http/2 allows it: https://github.com/hyperium/hyper/issues/3298
+    request.headers_mut().remove(HOST);
+
     if let Some(interceptor) = request_interceptor {
         let intercept_request = std::mem::take(&mut request).into();
         match interceptor.intercept(intercept_request).await? {
@@ -459,7 +463,7 @@ async fn send_request_impl(
 }
 
 async fn send_request_handler(
-    mut request: http::Request<HyperOutgoingBody>,
+    request: http::Request<HyperOutgoingBody>,
     wasmtime_wasi_http::types::OutgoingRequestConfig {
         use_tls,
         connect_timeout,
@@ -470,10 +474,6 @@ async fn send_request_handler(
     blocked_networks: BlockedNetworks,
     http_clients: HttpClients,
 ) -> Result<wasmtime_wasi_http::types::IncomingResponse, ErrorCode> {
-    // Some servers (looking at you nginx) don't like a host header even though
-    // http/2 allows it: https://github.com/hyperium/hyper/issues/3298
-    request.headers_mut().remove(HOST);
-
     let resp = CONNECT_OPTIONS.scope(
         ConnectOptions {
             blocked_networks,