archive/pillar
0
0
Fork 0
Code Issues 42 Pull Requests Projects Releases Wiki Activity

Log as error when project admin group isn't properly configured.

Browse Source
This commit is contained in:
Sybren A. Stüvel 2017-10-25 14:57:55 +02:00
parent e983d0756d
commit a4addbfd22
1 changed files with 5 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
pillar/api/projects/utils.py
View File

@ -5,6 +5,8 @@ from flask import current_app
from werkzeug import exceptions as wz_exceptions from werkzeug import exceptions as wz_exceptions
from werkzeug.exceptions import abort from werkzeug.exceptions import abort
from pillar.auth import current_user
log = logging.getLogger(__name__) log = logging.getLogger(__name__)
@ -58,6 +60,9 @@ def get_admin_group(project: dict) -> dict:
raise ValueError('Unable to handle project without admin group.') raise ValueError('Unable to handle project without admin group.')
if group['name'] != str(project['_id']): if group['name'] != str(project['_id']):
log.error('User %s tries to get admin group for project %s, '
'but that does not have the project ID as group name: %s',
current_user.user_id, project.get('_id', '-unknown-'), group)
return abort_with_error(403) return abort_with_error(403)
return group return group