Exception handlers registration behavior

Should Starlette raise an exception when there are exception handlers for 500 status code and for Exception class?

For instance, in this case, only one of those handlers will be called.

def handler_for_500(req, exc):
    ...

def handler_for_exception(req, exc):
    ...

app = Starlette(
    exception_handlers={
        500: handler_for_500,
        Exception: handler_for_exception,
    }
)

So my question - is this expected behavior or it is a bug?

[aminalaee]

Sorry I didn’t explain that clearly. What I meant is that we could consider a full stop with a RuntimeError, or have a warning saying already have an error_handler and ignore it.

Totally agree, let’s wait for feedback.

[Kludex]

Or we could also set the first error_handlder and ignore any other after that.

That’s just transferring the problem.

I don’t think we do any sacrifice doing the check, as we’ll run it once, but… Is it necessary?

Well, it’s been a year, let’s ask the issuer. @uriyyo after more than a year, what are your thoughts about this issue? Do you think doing the check is still necessary?