Only send x-amz-metadata-directive on CopyObject

Fixes bji#80 See: https://docs.aws.amazon.com/AmazonS3/latest/API/API_CopyObject.html and https://docs.aws.amazon.com/AmazonS3/latest/API/API_UploadPartCopy.html
irods · Dec 9, 2020 · 054a76a · 054a76a
1 parent 287e4be
commit 054a76a
Showing 1 changed file with 6 additions and 3 deletions.
diff --git a/src/request.c b/src/request.c
@@ -405,9 +405,12 @@ static S3Status compose_amz_headers(const RequestParams *params,
                      params->startByte, params->startByte + params->byteCount);
             append_amz_header(values, 0, "x-amz-copy-source-range", byteRange);
         }
-        // And the x-amz-metadata-directive header
-        if (properties) {
-            append_amz_header(values, 0, "x-amz-metadata-directive", "REPLACE");
+        else {
+            // Add the x-amz-metadata-directive header
+            // Only for CopyObject, not UploadPartCopy (above)
+            if (properties) {
+                append_amz_header(values, 0, "x-amz-metadata-directive", "REPLACE");
+            }
         }
     }