diff options
author | Erik Olof Gunnar Andersson <eandersson@blizzard.com> | 2021-11-24 14:10:55 -0800 |
---|---|---|
committer | Erik Olof Gunnar Andersson <eandersson@blizzard.com> | 2021-11-24 14:10:55 -0800 |
commit | b9a4e36c0cf09bee7ba64c0b775f293915daefca (patch) | |
tree | 13bf623ec1db3e3d4128672d5c6066218a8fef02 /designate/api | |
parent | bee8dc032e32536d0c7a24e65e9883a7a196b068 (diff) | |
download | designate-b9a4e36c0cf09bee7ba64c0b775f293915daefca.tar.gz |
Remove LOG.exceptions from __call__
Removing the extra LOG.exceptions from call,
as this is already handled under _handle_exception.
> if not getattr(e, 'expected', False):
> LOG.exception(e)
Change-Id: I2ff56bb8e079fa032ca3b211b9d46b3a0906c0c3
Diffstat (limited to 'designate/api')
-rw-r--r-- | designate/api/middleware.py | 4 |
1 files changed, 0 insertions, 4 deletions
diff --git a/designate/api/middleware.py b/designate/api/middleware.py index a4cd5f50..01de871a 100644 --- a/designate/api/middleware.py +++ b/designate/api/middleware.py @@ -237,7 +237,6 @@ class FaultWrapperMiddleware(base.Middleware): try: return request.get_response(self.application) except exceptions.DesignateException as e: - LOG.exception(e) # Handle Designate Exceptions status = e.error_code if hasattr(e, 'error_code') else 500 @@ -257,7 +256,6 @@ class FaultWrapperMiddleware(base.Middleware): return self._handle_exception(request, e, status, response) except messaging.MessagingTimeout as e: - LOG.exception(e) # Special case for RPC timeout's response = { 'code': 504, @@ -266,7 +264,6 @@ class FaultWrapperMiddleware(base.Middleware): return self._handle_exception(request, e, 504, response) except Exception as e: - LOG.exception(e) # Handle all other exception types return self._handle_exception(request, e) @@ -274,7 +271,6 @@ class FaultWrapperMiddleware(base.Middleware): pass def _handle_exception(self, request, e, status=500, response=None): - response = response or {} # Log the exception ASAP unless it is a 404 Not Found if not getattr(e, 'expected', False): |