Fix (*Bucket)PutCopy bugs

AdRoll / goamz

Fork of the GOAMZ version developed within Canonical with additional functionality with DynamoDB

https://wiki.ubuntu.com/goamz

Other

401 stars 213 forks source link

Closed tomclegg closed 9 years ago

tomclegg commented 9 years ago

Adds a PutCopy() to the basic functionality test case, and fixes a couple of bugs needed to make it work.

s3test was sending an empty response body instead of a CopyObjectResult
PutCopy was using url.QueryEscape() to escape the source parameter, which caused the / separator to be incorrectly encoded as %2F

alimoeeny commented 9 years ago

Thanks, @tomclegg
surprised that url.QueryEscape was not working.