[#6] Only send x-amz-metadata-directive on CopyObject

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 8, 2020 · 07f4478 · 07f4478
1 parent 290c88a
commit 07f4478
Showing 1 changed file with 6 additions and 3 deletions.
diff --git a/src/request.c b/src/request.c
@@ -401,9 +401,12 @@ static S3Status compose_amz_headers(const RequestParams *params,
                            (unsigned long long) params->startByte,
                            (unsigned long long) (params->startByte + params->byteCount - 1) );
         }
-        // And the x-amz-metadata-directive header
-        if (properties) {
-            headers_append(1, "%s", "x-amz-metadata-directive: REPLACE");
+        else {
+            // Add the x-amz-metadata-directive header
+            // Only for CopyObject, not UploadPartCopy (above)
+            if (properties) {
+                headers_append(1, "%s", "x-amz-metadata-directive: REPLACE");
+            }
         }
     }