add support for ACLs in swift

ruby-openstack / ruby-openstack

Ruby binding for Openstack API - supports Keystone, Nova, Swift, Cinder and Neutron

Other

127 stars 80 forks source link

add support for ACLs in swift #50

Open stensonb opened 10 years ago

stensonb commented 10 years ago

I can set the user-metadata, but I cannot see how to set ACLs (read, write) with this client.

It would be great to support that...something like:

# getting
container.read_acl

#setting
container.read_acl = (foo)

# getting
container.write_acl

# setting
container.write_acl = (foo)

See the dev API docs for details: http://developer.openstack.org/api-ref-objectstorage-v1.html

bpieck commented 10 years ago

Hi, If you need this asap, you can use our fork. I added your four methods there in my last commit: https://github.com/bpieck/ruby-openstack/commit/6453057042add2182d278c8e6f888d2dd521fcf7

stensonb commented 10 years ago

Excellent! Thanks @bpieck ...

Did you submit a PR to this? I'd love to get this via rubygems and a more formal release process...

bpieck commented 10 years ago

No, I did not yet. Sorry since we have tested with rspec in our fork and the original developers are using test unit, I can't really submit a PR for it. But perhaps someone who works more with test unit than me, likes to pick my commit and submit a PR.

mrkamel commented 9 years ago

f0086 commented 8 years ago

@bpieck Thanks! What do you think about migrating all tests to rspec? There are not much tests available, so this will be a win win for everyone :+1: