pk/atst
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Update logger message for application user not found

Browse Source
This commit is contained in:
Montana 2019-04-30 10:12:05 -04:00
parent 78a8201323
commit a99c795319
1 changed files with 3 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
atst/routes/applications/settings.py
View File

@ -47,7 +47,7 @@ def serialize_env_member_form_data(application):
def check_users_are_in_application(user_ids, application): def check_users_are_in_application(user_ids, application):
for user_id in user_ids: for user_id in user_ids:
if not application.has_member(user_id): if not application.has_member(user_id):
raise NotFoundError("application user {}".format(user_id)) raise NotFoundError("application user", user_id)
return True return True
@ -115,8 +115,8 @@ def update_env_roles(environment_id):
check_users_are_in_application(user_ids, application) check_users_are_in_application(user_ids, application)
except NotFoundError as err: except NotFoundError as err:
app.logger.warning( app.logger.warning(
"Security violation for user {}, {} {}".format( "User {} requested environment role change for unauthorized user {} in application {}.".format(
g.current_user.id, request.method, request.path g.current_user.id, err.resource_id, application.id
), ),
extra={"tags": ["update", "failure"], "security_warning": True}, extra={"tags": ["update", "failure"], "security_warning": True},
) )