How to respond with HTTP 500 on any unhandled exception in Falcon framework
Asked Answered
S

3

8

Is there a way in Falcon framework to respond with HTTP 500 status on any unspecific exception that is not handled in resource handler? I've tried to add following handler for Exception:

api.add_error_handler(Exception, 
                      handler=lambda e, 
                      *_: exec('raise falcon.HTTPInternalServerError("Internal Server Error", "Some error")'))

But this makes impossible to throw, for example, falcon.HTTPNotFound — it is handled by the handler above and I receive 500 instead of 404.

Savick answered 19/9, 2016 at 10:16 Comment(0)
S
8

Yes, it is possible. You need to define a generic error handler, check if the exception is instance of any falcon error, and if it is not, then raise your HTTP_500.

This example shows a way of doing it.

def generic_error_handler(ex, req, resp, params):
    if not isinstance(ex, HTTPError):
        raise HTTPInternalServerError("Internal Server Error", "Some error")
    else:  # reraise :ex otherwise it will gobble actual HTTPError returned from the application code ref. https://mcmap.net/q/1299688/-how-to-respond-with-http-500-on-any-unhandled-exception-in-falcon-framework
        raise ex

app = falcon.API()
app.add_error_handler(Exception, generic_error_handler)
Sealey answered 23/9, 2018 at 9:48 Comment(3)
and remember that any other, more specific error handler must be registered after generic_error_handler, which will otherwise gobble it up (see here)Semblable
This, however, suppresses Exceptions being raised into the Console. Is there a way to raise both internally, and respond with any (i.e. generic Exception) as falcon request-response?Spry
You can log exception ex before raising HTTPInternalError here. For example, using this method: docs.python.org/3/library/logging.html#logging.exceptionTablespoon
T
2

Accepted answer seems to gobble actual HTTPError returned from the application code. This is what worked for me:

def generic_error_handler(ex, req, resp, params):
    if not isinstance(ex, HTTPError):
        logger.exception("Internal server error")
        raise HTTPInternalServerError("Internal Server Error")
    else:
        raise ex

Tablespoon answered 9/3, 2020 at 18:59 Comment(0)
M
1

I am not sure whether I understand your question properly or not.

But you can use following approach to return respond with HTTP 500 status on any unspecific exception:

class MyFirstAPI:
    def on_post(self, req, res):
        try:
            json_data = json.loads(req.stream.read().decode('utf8'))
            # some task
            res.status = falcon.HTTP_200
            res.body = json.dumps({'status': 1, 'message': "success"})

        except Exception as e:
            res.status = falcon.HTTP_500
            res.body = json.dumps({'status': 0,
                               'message': 'Something went wrong, Please try again'
                               })
app = falcon.API()
app.add_route("/my-api/", MyFirstAPI())

Or you can also use Decorators in python as follow:

def my_500_error_decorator(func):
    def wrapper(*args):
        try:
            func(*args)
        except Exception as e:
            resp.status = falcon.HTTP_500
            resp.body = json.dumps({'status': 0, 'message': 'Server Error'})

return wrapper

class MyFirstAPI:
    @my_500_error_decorator
    def on_post(self, req, res):
        try:
            json_data = json.loads(req.stream.read().decode('utf8'))
            # some task
            res.status = falcon.HTTP_200
            res.body = json.dumps({'status': 1, 'message': "success"})
app = falcon.API()
app.add_route("/my-api/", MyFirstAPI())
Meretricious answered 5/3, 2017 at 17:25 Comment(0)

© 2022 - 2024 — McMap. All rights reserved.