Uploaded image for project: 'Libcloud'
  1. Libcloud
  2. LIBCLOUD-497

Cannot set ACLs on S3 objects

        Details

        • Type: Bug
        • Status: Resolved
        • Priority: Major
        • Resolution: Fixed
        • Affects Version/s: 0.13.3
        • Fix Version/s: None
        • Component/s: Storage
        • Labels:
          None

          Description

          Would be nice if you could set extra=

          {'acl': 'public-read'}

          and get that passed as the x-amz-acl header.

            Activity

            Ascending order - Click to sort in descending order
            Hide
            Permalink
            kami Tomaz Muraus added a comment -

            Yeah, adding support for ACL management to the Amazon S3 driver would be nice.

            We could implement ACL management methods which work on the existing objects and containers via the extension methods.

            And for new containers and object we could add "ex_acl" or a similar argument to the create_container, upload_object and other methods.

            Reference: http://docs.aws.amazon.com/AmazonS3/latest/dev/ACLUsingRESTAPI.html

            Show
            kami Tomaz Muraus added a comment - Yeah, adding support for ACL management to the Amazon S3 driver would be nice. We could implement ACL management methods which work on the existing objects and containers via the extension methods. And for new containers and object we could add "ex_acl" or a similar argument to the create_container, upload_object and other methods. Reference: http://docs.aws.amazon.com/AmazonS3/latest/dev/ACLUsingRESTAPI.html
            Hide
            Permalink
            coderanger Noah Kantrowitz added a comment -

            Patch posted to https://github.com/apache/libcloud/pull/223

            Show
            coderanger Noah Kantrowitz added a comment - Patch posted to https://github.com/apache/libcloud/pull/223
            Hide
            Permalink
            kami Tomaz Muraus added a comment -

            Patch has been merged into trunk, thanks!

            Show
            kami Tomaz Muraus added a comment - Patch has been merged into trunk, thanks!

              People

              • Assignee:
                Unassigned
                Reporter:
                coderanger Noah Kantrowitz
              • Votes:
                0 Vote for this issue
                Watchers:
                2 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved:

                  Development