Merge pull request #2768 from bhavya2109sharma/presignedurl-requestpa…

…yer-change Presignedurl requestpayer change
aws · Aug 30, 2024 · 4ed838e · 4ed838e
2 parents 97e2d3f + d4bd42f
commit 4ed838e
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 1 deletion.
diff --git a/.changelog/d85bd7ce7f754e159b33ac402f217a7c.json b/.changelog/d85bd7ce7f754e159b33ac402f217a7c.json
@@ -0,0 +1,8 @@
+{
+    "id": "d85bd7ce-7f75-4e15-9b33-ac402f217a7c",
+    "type": "bugfix",
+    "description": "Change requestPayer behaviour from signedHeader to hoisted query parameter ",
+    "modules": [
+        "."
+    ]
+}
diff --git a/aws/signer/internal/v4/headers.go b/aws/signer/internal/v4/headers.go
@@ -45,7 +45,6 @@ var RequiredSignedHeaders = Rules{
 			"X-Amz-Grant-Write-Acp":                                       struct{}{},
 			"X-Amz-Metadata-Directive":                                    struct{}{},
 			"X-Amz-Mfa":                                                   struct{}{},
-			"X-Amz-Request-Payer":                                         struct{}{},
 			"X-Amz-Server-Side-Encryption":                                struct{}{},
 			"X-Amz-Server-Side-Encryption-Aws-Kms-Key-Id":                 struct{}{},
 			"X-Amz-Server-Side-Encryption-Context":                        struct{}{},