ba9b432993
* Default error handler now only logs traceback on 500 errors and all responses are HTML formatted. * Tests passing. * Ability to flag any exception object with self.quiet = True following @ashleysommer suggestion. * Refactor HTML formatting into errorpages.py. String escapes for debug tracebacks. * Remove extra includes * Auto-set quiet flag also when decorator is used. * Cleanup, make error pages (probably) HTML5-compliant and similar for debug and non-debug modes. * Fix lookup of non-existant status codes * No logging of 503 errors after all. * lint
214 lines
7.1 KiB
Python
214 lines
7.1 KiB
Python
from traceback import format_exc
|
|
|
|
from sanic.errorpages import exception_response
|
|
from sanic.exceptions import (
|
|
ContentRangeError,
|
|
HeaderNotFound,
|
|
InvalidRangeType,
|
|
)
|
|
from sanic.log import logger
|
|
from sanic.response import text
|
|
|
|
|
|
class ErrorHandler:
|
|
"""
|
|
Provide :class:`sanic.app.Sanic` application with a mechanism to handle
|
|
and process any and all uncaught exceptions in a way the application
|
|
developer will set fit.
|
|
|
|
This error handling framework is built into the core that can be extended
|
|
by the developers to perform a wide range of tasks from recording the error
|
|
stats to reporting them to an external service that can be used for
|
|
realtime alerting system.
|
|
|
|
"""
|
|
|
|
handlers = None
|
|
cached_handlers = None
|
|
_missing = object()
|
|
|
|
def __init__(self):
|
|
self.handlers = []
|
|
self.cached_handlers = {}
|
|
self.debug = False
|
|
|
|
def add(self, exception, handler):
|
|
"""
|
|
Add a new exception handler to an already existing handler object.
|
|
|
|
:param exception: Type of exception that need to be handled
|
|
:param handler: Reference to the method that will handle the exception
|
|
|
|
:type exception: :class:`sanic.exceptions.SanicException` or
|
|
:class:`Exception`
|
|
:type handler: ``function``
|
|
|
|
:return: None
|
|
"""
|
|
self.handlers.append((exception, handler))
|
|
|
|
def lookup(self, exception):
|
|
"""
|
|
Lookup the existing instance of :class:`ErrorHandler` and fetch the
|
|
registered handler for a specific type of exception.
|
|
|
|
This method leverages a dict lookup to speedup the retrieval process.
|
|
|
|
:param exception: Type of exception
|
|
|
|
:type exception: :class:`sanic.exceptions.SanicException` or
|
|
:class:`Exception`
|
|
|
|
:return: Registered function if found ``None`` otherwise
|
|
"""
|
|
handler = self.cached_handlers.get(type(exception), self._missing)
|
|
if handler is self._missing:
|
|
for exception_class, handler in self.handlers:
|
|
if isinstance(exception, exception_class):
|
|
self.cached_handlers[type(exception)] = handler
|
|
return handler
|
|
self.cached_handlers[type(exception)] = None
|
|
handler = None
|
|
return handler
|
|
|
|
def response(self, request, exception):
|
|
"""Fetches and executes an exception handler and returns a response
|
|
object
|
|
|
|
:param request: Instance of :class:`sanic.request.Request`
|
|
:param exception: Exception to handle
|
|
|
|
:type request: :class:`sanic.request.Request`
|
|
:type exception: :class:`sanic.exceptions.SanicException` or
|
|
:class:`Exception`
|
|
|
|
:return: Wrap the return value obtained from :func:`default`
|
|
or registered handler for that type of exception.
|
|
"""
|
|
handler = self.lookup(exception)
|
|
response = None
|
|
try:
|
|
if handler:
|
|
response = handler(request, exception)
|
|
if response is None:
|
|
response = self.default(request, exception)
|
|
except Exception:
|
|
self.log(format_exc())
|
|
try:
|
|
url = repr(request.url)
|
|
except AttributeError:
|
|
url = "unknown"
|
|
response_message = (
|
|
"Exception raised in exception handler " '"%s" for uri: %s'
|
|
)
|
|
logger.exception(response_message, handler.__name__, url)
|
|
|
|
if self.debug:
|
|
return text(response_message % (handler.__name__, url), 500)
|
|
else:
|
|
return text("An error occurred while handling an error", 500)
|
|
return response
|
|
|
|
def log(self, message, level="error"):
|
|
"""
|
|
Deprecated, do not use.
|
|
"""
|
|
|
|
def default(self, request, exception):
|
|
"""
|
|
Provide a default behavior for the objects of :class:`ErrorHandler`.
|
|
If a developer chooses to extent the :class:`ErrorHandler` they can
|
|
provide a custom implementation for this method to behave in a way
|
|
they see fit.
|
|
|
|
:param request: Incoming request
|
|
:param exception: Exception object
|
|
|
|
:type request: :class:`sanic.request.Request`
|
|
:type exception: :class:`sanic.exceptions.SanicException` or
|
|
:class:`Exception`
|
|
:return:
|
|
"""
|
|
quiet = getattr(exception, "quiet", False)
|
|
if quiet is False:
|
|
try:
|
|
url = repr(request.url)
|
|
except AttributeError:
|
|
url = "unknown"
|
|
|
|
self.log(format_exc())
|
|
logger.exception("Exception occurred while handling uri: %s", url)
|
|
|
|
return exception_response(request, exception, self.debug)
|
|
|
|
|
|
class ContentRangeHandler:
|
|
"""
|
|
A mechanism to parse and process the incoming request headers to
|
|
extract the content range information.
|
|
|
|
:param request: Incoming api request
|
|
:param stats: Stats related to the content
|
|
|
|
:type request: :class:`sanic.request.Request`
|
|
:type stats: :class:`posix.stat_result`
|
|
|
|
:ivar start: Content Range start
|
|
:ivar end: Content Range end
|
|
:ivar size: Length of the content
|
|
:ivar total: Total size identified by the :class:`posix.stat_result`
|
|
instance
|
|
:ivar ContentRangeHandler.headers: Content range header ``dict``
|
|
"""
|
|
|
|
__slots__ = ("start", "end", "size", "total", "headers")
|
|
|
|
def __init__(self, request, stats):
|
|
self.total = stats.st_size
|
|
_range = request.headers.get("Range")
|
|
if _range is None:
|
|
raise HeaderNotFound("Range Header Not Found")
|
|
unit, _, value = tuple(map(str.strip, _range.partition("=")))
|
|
if unit != "bytes":
|
|
raise InvalidRangeType(
|
|
"%s is not a valid Range Type" % (unit,), self
|
|
)
|
|
start_b, _, end_b = tuple(map(str.strip, value.partition("-")))
|
|
try:
|
|
self.start = int(start_b) if start_b else None
|
|
except ValueError:
|
|
raise ContentRangeError(
|
|
"'%s' is invalid for Content Range" % (start_b,), self
|
|
)
|
|
try:
|
|
self.end = int(end_b) if end_b else None
|
|
except ValueError:
|
|
raise ContentRangeError(
|
|
"'%s' is invalid for Content Range" % (end_b,), self
|
|
)
|
|
if self.end is None:
|
|
if self.start is None:
|
|
raise ContentRangeError(
|
|
"Invalid for Content Range parameters", self
|
|
)
|
|
else:
|
|
# this case represents `Content-Range: bytes 5-`
|
|
self.end = self.total - 1
|
|
else:
|
|
if self.start is None:
|
|
# this case represents `Content-Range: bytes -5`
|
|
self.start = self.total - self.end
|
|
self.end = self.total - 1
|
|
if self.start >= self.end:
|
|
raise ContentRangeError(
|
|
"Invalid for Content Range parameters", self
|
|
)
|
|
self.size = self.end - self.start + 1
|
|
self.headers = {
|
|
"Content-Range": "bytes %s-%s/%s"
|
|
% (self.start, self.end, self.total)
|
|
}
|
|
|
|
def __bool__(self):
|
|
return self.size > 0
|