← Back to team overview

yahoo-eng-team team mailing list archive

[Bug 1406938] [NEW] Check of targets and actors on Assignment Controller should be generic and report a better message

 

Public bug reported:

The XOR checking of targets  and actors on RoleV3 controller [1] should
be generic and placed at V3Controller, so that it could be used on other
controllers, such as RoleAssignmentV3 on list_role_assignments method,
which needs to report 400 for invalid filters combination [2].

In addition, it has to report a better error message, because it says *both* entities were provided, even if, in fact, *none* were:
'Specify a user or group, not both'; 'Specify a user or group, not both'

[1] https://github.com/openstack/keystone/blob/master/keystone/assignment/controllers.py#L496-L504
[2] https://bugs.launchpad.net/keystone/+bug/1406667

** Affects: keystone
     Importance: Undecided
         Status: New

-- 
You received this bug notification because you are a member of Yahoo!
Engineering Team, which is subscribed to Keystone.
https://bugs.launchpad.net/bugs/1406938

Title:
  Check of targets and actors on Assignment Controller should be generic
  and report a better message

Status in OpenStack Identity (Keystone):
  New

Bug description:
  The XOR checking of targets  and actors on RoleV3 controller [1]
  should be generic and placed at V3Controller, so that it could be used
  on other controllers, such as RoleAssignmentV3 on
  list_role_assignments method, which needs to report 400 for invalid
  filters combination [2].

  In addition, it has to report a better error message, because it says *both* entities were provided, even if, in fact, *none* were:
  'Specify a user or group, not both'; 'Specify a user or group, not both'

  [1] https://github.com/openstack/keystone/blob/master/keystone/assignment/controllers.py#L496-L504
  [2] https://bugs.launchpad.net/keystone/+bug/1406667

To manage notifications about this bug go to:
https://bugs.launchpad.net/keystone/+bug/1406938/+subscriptions


Follow ups

References