Setting to invalid because the patch is abandoned and because of
comments in it.
** Changed in: keystone
Status: In Progress => Invalid
--
You received this bug notification because you are a member of Yahoo!
Engineering Team, which is subscribed to Keystone.
https://bugs.launchpad.net/bugs/1414876
Title:
unnecessary checks in assignment/controllers.py
Status in OpenStack Identity (Keystone):
Invalid
Bug description:
There are checks that only one of project/domain or user/group are passed to
create/list/check/revoke_grant methods,
https://github.com/openstack/keystone/blob/master/keystone/assignment/controllers.py#L496-L504.
However,
1. only router calls these methods
2. URL scheme does not accept both project and user simultaneously.
If a user specify a url such as
/domains/{domain_id}/projects/{project_id}/users/{user_id}/roles or
/domains/{domain_id}/users/{user_id}/groups/{group_id}/roles or etc, he would
get a 404 error from router.
That is to say, the cases these methods check would never happen. So, I think
these checks can be removed.
To manage notifications about this bug go to:
https://bugs.launchpad.net/keystone/+bug/1414876/+subscriptions
--
Mailing list: https://launchpad.net/~yahoo-eng-team
Post to : [email protected]
Unsubscribe : https://launchpad.net/~yahoo-eng-team
More help : https://help.launchpad.net/ListHelp
