check sign_payload instead of skip_signature before computing checksum (apache#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]>

Author: mherrerarendon

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();