UMIACS / rgwadmin

Ceph Object Storage Admin API python library bindings.
https://rgwadmin.readthedocs.io/
GNU Lesser General Public License v2.1
85 stars 47 forks source link

new: add access_key kwarg to get_user #55

Closed jahschwa closed 3 years ago

jahschwa commented 3 years ago

Although this is undocumented, it does exist in the code:

https://github.com/ceph/ceph/blob/v15.2.8/src/rgw/rgw_rest_user.cc#L70

And worked on my ceph cluster.

I'll note in my testing access_key overrode uid, which feels unintuitive to me, so personally I think it makes sense to prevent the user from passing both. Passing neither will return a 400 Invalid Argument from ceph.