Merge pull request #422 from r0fls/420

move logging to method in ErrorHandler
This commit is contained in:
Eli Uriegas 2017-02-15 21:03:15 -06:00 committed by GitHub
commit a359e11f97
2 changed files with 9 additions and 3 deletions

View File

@ -46,7 +46,7 @@ class ErrorHandler:
try: try:
response = handler(request=request, exception=exception) response = handler(request=request, exception=exception)
except Exception: except Exception:
log.error(format_exc()) self.log(format_exc())
if self.debug: if self.debug:
response_message = ( response_message = (
'Exception raised in exception handler "{}" ' 'Exception raised in exception handler "{}" '
@ -58,8 +58,15 @@ class ErrorHandler:
return text('An error occurred while handling an error', 500) return text('An error occurred while handling an error', 500)
return response return response
def log(self, message, level='error'):
"""
Override this method in an ErrorHandler subclass to prevent
logging exceptions.
"""
getattr(log, level)(message)
def default(self, request, exception): def default(self, request, exception):
log.error(format_exc()) self.log(format_exc())
if issubclass(type(exception), SanicException): if issubclass(type(exception), SanicException):
return text( return text(
'Error: {}'.format(exception), 'Error: {}'.format(exception),

View File

@ -346,7 +346,6 @@ def serve(host, port, request_handler, error_handler, before_start=None,
except NotImplementedError: except NotImplementedError:
log.warn('Sanic tried to use loop.add_signal_handler but it is' log.warn('Sanic tried to use loop.add_signal_handler but it is'
' not implemented on this platform.') ' not implemented on this platform.')
pid = os.getpid() pid = os.getpid()
try: try:
log.info('Starting worker [{}]'.format(pid)) log.info('Starting worker [{}]'.format(pid))