-
Notifications
You must be signed in to change notification settings - Fork 4
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Change priority of kernel exception subscribers to be able to process security exceptions #82
Conversation
… security exceptions
This comment has been minimized.
This comment has been minimized.
11b911e
to
8a0a548
Compare
Code Coverage Report: Summary: Nijens\OpenapiBundle\Controller\CatchAllController |
This change is necessary to fix transforming security exceptions to
ProblemException
instances.Symfony's
ErrorListener
converts eg. anAccessDeniedException
to anAccessDeniedHttpException
even though the exception is already transformed to aProblemException
instance.By changing the priority of the subscribers to execute after Symfony's
ErrorListener
theAccessDeniedHttpException
(instead of theAccessDeniedException
) is transformed toProblemException
instance and then successfully converted to a response.