From a99c795319b27e09d63ddf5793ec5f7d9d160e52 Mon Sep 17 00:00:00 2001 From: Montana Date: Tue, 30 Apr 2019 10:12:05 -0400 Subject: [PATCH] Update logger message for application user not found --- atst/routes/applications/settings.py | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/atst/routes/applications/settings.py b/atst/routes/applications/settings.py index ad68a8d4..aa017fa2 100644 --- a/atst/routes/applications/settings.py +++ b/atst/routes/applications/settings.py @@ -47,7 +47,7 @@ def serialize_env_member_form_data(application): def check_users_are_in_application(user_ids, application): for user_id in user_ids: if not application.has_member(user_id): - raise NotFoundError("application user {}".format(user_id)) + raise NotFoundError("application user", user_id) return True @@ -115,8 +115,8 @@ def update_env_roles(environment_id): check_users_are_in_application(user_ids, application) except NotFoundError as err: app.logger.warning( - "Security violation for user {}, {} {}".format( - g.current_user.id, request.method, request.path + "User {} requested environment role change for unauthorized user {} in application {}.".format( + g.current_user.id, err.resource_id, application.id ), extra={"tags": ["update", "failure"], "security_warning": True}, )