Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Include Content-MD5 in signed headers #17

Open
USSRLivesOn opened this issue Jun 22, 2016 · 2 comments
Open

Include Content-MD5 in signed headers #17

USSRLivesOn opened this issue Jun 22, 2016 · 2 comments

Comments

@USSRLivesOn
Copy link
Contributor

USSRLivesOn commented Jun 22, 2016

Per the AWS docs, "The headers used for request signing are: content-md5, content-type, date, and anything that starts with x-amz-." It looks like content-md5 is excluded currently from the set of headers that is signed by requests-aws4auth, but it should be included. Our current workaround is just to pass the full set in include_hdrs kwarg, but would be good to have the defaults updated.

@KatTraxler
Copy link

I'm running into this issue now, can you provide an example of this workaround?

@viniciuspinto
Copy link

I'm running into this issue now, can you provide an example of this workaround?

I got it working by passing the include_hdrs argument to the constructor:

AWS4Auth('<ACCESS ID>', '<ACCESS KEY>', 'eu-west-1', 's3', include_hdrs={'host', 'content-type', 'date', 'x-amz-*', 'content-md5'})

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants