From 90c8f18f768b2944ba7ad5d33fbcdb811c2f1e5c Mon Sep 17 00:00:00 2001 From: Matthew Jones Date: Fri, 30 Jan 2015 15:55:14 -0500 Subject: [PATCH] Make error messages less alarming and include ip address in them --- awx/api/generics.py | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/awx/api/generics.py b/awx/api/generics.py index ed60eacaeb..30def04f42 100644 --- a/awx/api/generics.py +++ b/awx/api/generics.py @@ -92,7 +92,12 @@ class APIView(views.APIView): Log warning for 400 requests. Add header with elapsed time. ''' if response.status_code >= 400: - logger.warn("status %s received by user %s attempting to access %s" % (response.status_code, request.user, request.path)) + status_msg = "status %s received by user %s attempting to access %s from %s" % \ + (response.status_code, request.user, request.path, request.META.get('REMOTE_ADDR', None)) + if response.status_code == 401: + logger.info(status_msg) + else: + logger.warn(status_msg) response = super(APIView, self).finalize_response(request, response, *args, **kwargs) time_started = getattr(self, 'time_started', None) if time_started: