-
-
Notifications
You must be signed in to change notification settings - Fork 9.4k
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
[Security] Added AbstractExceptionListener #11103
Conversation
This allows to use a custom implementations of exception listeners.
*/ | ||
public function add(RequestMatcherInterface $requestMatcher = null, array $listeners = array(), ExceptionListener $exceptionListener = null) | ||
public function add( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
please keep method declaration on a single line
Will this get merged? |
use Symfony\Component\HttpKernel\KernelEvents; | ||
|
||
/** | ||
* Interface that must be implemented by firewall exception listener |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is an abstract class, not an interface.
I'm 👎 on this one. The logic in the exception listener is quite complex and I fail to see how you would reimplement something different but still compatible with the default one. So, that's not an extension point I want to expose to the users. But of course, you can still just extend the existing class and implement your logic. |
I want to send |
This allows to use a custom implementations of exception listeners.