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
Turned RequestCleaner into an event subscriber #887
Conversation
6ddc659
to
d63b4f3
Compare
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.
👍
Looking good, can you resolve the conflicts, please? |
7b4c121
to
5df77db
Compare
Done. |
@@ -333,5 +332,8 @@ public function subscribe(Container $app, EventDispatcherInterface $dispatcher) | |||
$app[CsrfValidator::class], | |||
$app['url_generator'] | |||
)); | |||
$dispatcher->addSubscriber(new RequestCleanerListener( | |||
$app['purifier'] |
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.
Ha, sorry - can we sort out the wrapping issue here as well, please?
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.
Sure, done.
5df77db
to
ee5ef08
Compare
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.
👍
Thank you, @derrabus! |
The
RequestCleaner
class is now an event subscriber too.Related to #618.