-
Notifications
You must be signed in to change notification settings - Fork 6k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
rgw: Adding 'iam' namespace for Role and User Policy related REST APIs. #27178
Conversation
Signed-off-by: Pritha Srivastava <prsrivas@redhat.com>
|
||
if (post_body.find("Action") != string::npos) { | ||
boost::char_separator<char> sep("&"); | ||
boost::tokenizer<boost::char_separator<char>> tokens(post_body, sep); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
iirc boost::tokenizer isn't thought to be very efficient--maybe not a concern for now
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@adamemerson thoughts?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@mattbenjamin @adamemerson @cbodley : I have the exact same method for parsing sts_input also, with a few differences. I could have made this into a common method that parses output for both. But later when we decide to have STS, IAM served at a different hostname:portname, then these methods will already be separate. Is this ok for now?
fine with me |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm, needs-qa :)
The teuthology run is here: http://pulpito.ceph.com/prsrivas-2019-04-12_08:57:26-rgw-wip-rgw-iam-distro-basic-smithi/ |
Signed-off-by: Pritha Srivastava prsrivas@redhat.com