Skip to content

Commit

Permalink
Add x-amz-expected-bucket-owner as a header the SDK needs to sign (#…
Browse files Browse the repository at this point in the history
…5062)

* Add x-amz-expected-bucket-owner to headers to sign in presigned URL

* changelog added

* updated changelog

---------

Co-authored-by: Ran Vaknin <username@users.noreply.github.com>
  • Loading branch information
RanVaknin and Ran Vaknin committed Nov 10, 2023
1 parent a359223 commit d05d03f
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 0 deletions.
1 change: 1 addition & 0 deletions CHANGELOG_PENDING.md
Original file line number Diff line number Diff line change
@@ -1,5 +1,6 @@
### SDK Features

### SDK Enhancements
"x-amz-expected-bucket-owner" header was added to the list of headers that need to be presigned

### SDK Bugs
1 change: 1 addition & 0 deletions aws/signer/v4/v4.go
Original file line number Diff line number Diff line change
Expand Up @@ -125,6 +125,7 @@ var requiredSignedHeaders = rules{
"X-Amz-Copy-Source-Server-Side-Encryption-Customer-Algorithm": struct{}{},
"X-Amz-Copy-Source-Server-Side-Encryption-Customer-Key": struct{}{},
"X-Amz-Copy-Source-Server-Side-Encryption-Customer-Key-Md5": struct{}{},
"X-Amz-Expected-Bucket-Owner": struct{}{},
"X-Amz-Grant-Full-control": struct{}{},
"X-Amz-Grant-Read": struct{}{},
"X-Amz-Grant-Read-Acp": struct{}{},
Expand Down

0 comments on commit d05d03f

Please sign in to comment.