check sign_payload instead of skip_signature before computing checksum (

#6698) * check sign_payload instead of skip_signature before computing checksum * Update object_store/src/aws/client.rs Co-authored-by: Raphael Taylor-Davies <[email protected]> * fix format --------- Co-authored-by: Raphael Taylor-Davies <[email protected]>
apache · Nov 8, 2024 · 0e9abcd · 0e9abcd
1 parent a2a80ca
commit 0e9abcd
Showing 1 changed file with 3 additions and 1 deletion.
diff --git a/object_store/src/aws/client.rs b/object_store/src/aws/client.rs
@@ -372,7 +372,9 @@ impl<'a> Request<'a> {
     }
 
     pub(crate) fn with_payload(mut self, payload: PutPayload) -> Self {
-        if !self.config.skip_signature || self.config.checksum.is_some() {
+        if (!self.config.skip_signature && self.config.sign_payload)
+            || self.config.checksum.is_some()
+        {
             let mut sha256 = Context::new(&digest::SHA256);
             payload.iter().for_each(|x| sha256.update(x));
             let payload_sha256 = sha256.finish();