From fe03bad4a1d114e53a9f355a5976c12e1986f7c7 Mon Sep 17 00:00:00 2001 From: Benjamin Dauvergne Date: Wed, 6 Mar 2019 10:10:35 +0100 Subject: [PATCH 3/5] utils: only log requests/response as info (#31114) --- passerelle/utils/__init__.py | 11 ++--------- 1 file changed, 2 insertions(+), 9 deletions(-) diff --git a/passerelle/utils/__init__.py b/passerelle/utils/__init__.py index 8388ef6..873237f 100644 --- a/passerelle/utils/__init__.py +++ b/passerelle/utils/__init__.py @@ -262,7 +262,6 @@ class Request(RequestSession): extra['request_headers'] = dict(request.headers.items()) if request.body: extra['request_payload'] = repr(request.body[:5000]) - log_function = self.logger.info if response is not None: message = message + ' (=> %s)' % response.status_code extra['response_status'] = response.status_code @@ -271,17 +270,11 @@ class Request(RequestSession): # log body only if content type is allowed if content_type_match(response.headers.get('Content-Type')): content = response.content[:settings.LOGGED_RESPONSES_MAX_SIZE] - extra['response_content'] = repr(content) - if response.status_code // 100 == 3: - log_function = self.logger.warning - elif response.status_code // 100 >= 4: - log_function = self.logger.error + extra['response_content'] = repr(content[:5000]) elif exception: message = message + ' (=> %s)' % repr(exception) extra['response_exception'] = repr(exception) - log_function = self.logger.error - - log_function(message, extra=extra) + self.logger.info(message, extra=extra) class SOAPClient(Client): -- 2.20.1