fix(services/gcs): Remove HOST header to avoid gcs RESET connection (#…

…2139)

fix(services/gcs): Remove HOST header to avoid gcs RESET

Signed-off-by: Xuanwo <github@xuanwo.io>

.github/workflows/service_test_gcs.yml

@@ -47,10 +47,7 @@ jobs:
       - name: Test
         shell: bash
         working-directory: core
-        # Workaround
-        # gcs presign test failed for https://github.com/apache/incubator-opendal/issues/2125
-        # We should remove this feature until the issue is fixed.
-        run: cargo test gcs --features=native-tls -- --show-output
+        run: cargo test gcs -- --show-output
         env:
           RUST_BACKTRACE: full
           RUST_LOG: debug

core/src/services/gcs/backend.rs

@@ -21,6 +21,7 @@ use std::fmt::Formatter;
 use std::sync::Arc;
 
 use async_trait::async_trait;
+use http::header::HOST;
 use http::StatusCode;
 use log::debug;
 use reqsign::GoogleCredentialLoader;
@@ -544,7 +545,14 @@ impl Accessor for GcsBackend {
         self.core.sign_query(&mut req, args.expire()).await?;
 
         // We don't need this request anymore, consume it directly.
-        let (parts, _) = req.into_parts();
+        let (mut parts, _) = req.into_parts();
+        // Always remove host header, let users' client to set it based on HTTP
+        // version.
+        //
+        // As discussed in <https://github.com/seanmonstar/reqwest/issues/1809>,
+        // google server could send RST_STREAM of PROTOCOL_ERROR if our request
+        // contains host header.
+        parts.headers.remove(HOST);
 
         Ok(RpPresign::new(PresignedRequest::new(
             parts.method,