LearnBoost/knox — pull request #90 by bsorbo (2012-09-11 03:07:18+00:00)
Title: S3: Remove default canned ACL when access grants are used 

Knox by default adds the canned 'public-read' ACL to PUT and COPY requests. However, if an app manually specifies an access grant such as 'x-amz-grant-read', both the canned ACL and access grants will be present in the headers, and the request will fail with the following error:

'Specifying both Canned ACLs and Header Grants is not allowed'

This behavior is documented on the respective API pages, these are to be used exclusive of each other.

This change simply removes the default x-amz-acl header when any of the access grant headers are specified by the app. Let me know if there is a better way/place to handle this logic, thanks.