chilts/awssum

PutObjectAcl and PutBucketAcl missing 'Content-Length' header

laddi opened this issue · 3 comments

laddi commented

When calling PutObjectAcl or PutBucketAcl the Content-Length header is required. These two methods should calculate the content-length from the body part of the put request and add it as a header when the request is made.

laddi commented

Actually figured out a good way to solve this by adding this to bodyAccessControlPolicy in s3-config.js:

var xml = data2xml('AccessControlPolicy', args.AccessControlPolicy);
options.headers['Content-Length'] = xml.length;

return xml;
chilts commented

Hi Laddi,

Fixed the Content-Length issue for you (slightly more generic way than your suggestion, but essentially the same thing).

Will cut a release later today. :)

Cheers,
Andy

laddi commented

Sweet! Then we can add two huge features to our console, instance resizing and ACL permission to storage objects! :)

Sent from my iPhone

On 12.11.2012, at 18:56, Andrew Chilton notifications@github.com wrote:

Hi Laddi,

Fixed the Content-Length issue for you (slightly more generic way than your suggestion, but essentially the same thing).

Will cut a release later today. :)

Cheers,
Andy

—
Reply to this email directly or view it on GitHub.