Infra
/
lemur
5
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

Merge pull request #1589 from castrapel/issue_1570_ignorecasev2 

Allow case insensitive role matching for cert permissions
This commit is contained in:
Curtis 2018-08-20 09:03:55 -07:00 committed by GitHub
parent 075b18e5db 6abf274680
commit a21b71a0e2
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 3 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
lemur/auth/permissions.py
View File

@ -29,6 +29,9 @@ class CertificatePermission(Permission):
needs = [RoleNeed('admin'), RoleNeed(owner), RoleNeed('creator')] needs = [RoleNeed('admin'), RoleNeed(owner), RoleNeed('creator')]
for r in roles: for r in roles:
needs.append(CertificateOwnerNeed(str(r))) needs.append(CertificateOwnerNeed(str(r)))
# Backwards compatibility with mixed-case role names
if str(r) != str(r).lower():
needs.append(CertificateOwnerNeed(str(r).lower()))
super(CertificatePermission, self).__init__(*needs) super(CertificatePermission, self).__init__(*needs)