openstack/oslo.policy
Code Issues Proposed changes

Merge "Don't crash on RoleCheck when roles not present"

Browse Source
This commit is contained in:
Jenkins 2016-01-07 12:50:29 +00:00 committed by Gerrit Code Review
commit 8d65ff6de5
2 changed files with 8 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
oslo_policy/_checks.py
View File

@ -218,7 +218,9 @@ class RoleCheck(Check):
# While doing RoleCheck if key not # While doing RoleCheck if key not
# present in Target return false # present in Target return false
return False return False
return match.lower() in [x.lower() for x in creds['roles']] if 'roles' in creds:
return match.lower() in [x.lower() for x in creds['roles']]
return False
@register('http') @register('http')

5
oslo_policy/tests/test_checks.py
View File

@ -92,6 +92,11 @@ class RoleCheckTestCase(base.PolicyBaseTestCase):
target_dict = dict(target=dict(role=dict())) target_dict = dict(target=dict(role=dict()))
self.assertFalse(check(target_dict, cred_dict, self.enforcer)) self.assertFalse(check(target_dict, cred_dict, self.enforcer))
def test_no_roles_case(self):
check = _checks.RoleCheck('role', 'spam')
self.assertFalse(check({}, {}, self.enforcer))
class HttpCheckTestCase(base.PolicyBaseTestCase): class HttpCheckTestCase(base.PolicyBaseTestCase):