You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@opendal.apache.org by xu...@apache.org on 2026/03/24 04:56:44 UTC

(opendal) branch main updated: fix(s3): respect content encoding for multipart upload initialization (#7274)

This is an automated email from the ASF dual-hosted git repository.

xuanwo pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/opendal.git


The following commit(s) were added to refs/heads/main by this push:
     new 90328cf17 fix(s3): respect content encoding for multipart upload initialization (#7274)
90328cf17 is described below

commit 90328cf176519bf4acd3cb4629a1884655127ba4
Author: dentiny <de...@gmail.com>
AuthorDate: Mon Mar 23 21:56:38 2026 -0700

    fix(s3): respect content encoding for multipart upload initialization (#7274)
---
 core/services/s3/src/core.rs | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

diff --git a/core/services/s3/src/core.rs b/core/services/s3/src/core.rs
index 845e70130..f5482754f 100644
--- a/core/services/s3/src/core.rs
+++ b/core/services/s3/src/core.rs
@@ -805,6 +805,10 @@ impl S3Core {
             req = req.header(CACHE_CONTROL, cache_control)
         }
 
+        if let Some(content_encoding) = args.content_encoding() {
+            req = req.header(CONTENT_ENCODING, content_encoding)
+        }
+
         // Set storage class header
         if let Some(v) = &self.default_storage_class {
             req = req.header(HeaderName::from_static(constants::X_AMZ_STORAGE_CLASS), v);
@@ -828,7 +832,7 @@ impl S3Core {
         // Set SSE headers.
         req = self.insert_sse_headers(req, true);
 
-        // Set SSE headers.
+        // Set checksum type headers.
         req = self.insert_checksum_type_header(req);
 
         // Inject operation to the request.