Each uri component needs to be uri-encoded.

smarty-archives / go-aws-auth

[DEPRECATED] Signs requests to Amazon Web Services (AWS) using IAM roles or signed signature versions 2, 3, and 4. Supports S3 and STS.

Other

216 stars 71 forks source link

Closed pxi closed 10 years ago

pxi commented 10 years ago

AWS documentation states that each path component needs to be uri-encoded. This patch will fix that for signature V4.

mholt commented 10 years ago

Thanks @pxi, can you add a test case or two to ensure proper behavior? Make sure it fails without your fix. Then I'll be happy to merge this in!

pxi commented 10 years ago

I added a couple test cases from aws4 test suite to verify the functionality. I also fixed the escaping functionality a bit.