From 969aba205e24ae52ab1d9bb202832a423a7b0456 Mon Sep 17 00:00:00 2001 From: Valentin Clavreul Date: Thu, 1 Sep 2022 10:42:28 +0200 Subject: [PATCH] fix(log): avoid duplicated log where there is a subrequest by listening to kernel.terminate --- .../DependencyInjection/M6WebLogBridgeExtension.php | 2 +- .../LogBridgeBundle/EventDispatcher/LogRequestListener.php | 4 ++-- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/src/M6Web/Bundle/LogBridgeBundle/DependencyInjection/M6WebLogBridgeExtension.php b/src/M6Web/Bundle/LogBridgeBundle/DependencyInjection/M6WebLogBridgeExtension.php index f795e5d..adecd15 100644 --- a/src/M6Web/Bundle/LogBridgeBundle/DependencyInjection/M6WebLogBridgeExtension.php +++ b/src/M6Web/Bundle/LogBridgeBundle/DependencyInjection/M6WebLogBridgeExtension.php @@ -73,7 +73,7 @@ protected function loadRequestListener(ContainerBuilder $container): void ->addTag( 'kernel.event_listener', [ - 'event' => 'kernel.response', + 'event' => 'kernel.terminate', 'method' => 'onKernelTerminate', ] ); diff --git a/src/M6Web/Bundle/LogBridgeBundle/EventDispatcher/LogRequestListener.php b/src/M6Web/Bundle/LogBridgeBundle/EventDispatcher/LogRequestListener.php index a347beb..f0be2b7 100644 --- a/src/M6Web/Bundle/LogBridgeBundle/EventDispatcher/LogRequestListener.php +++ b/src/M6Web/Bundle/LogBridgeBundle/EventDispatcher/LogRequestListener.php @@ -7,7 +7,7 @@ use M6Web\Bundle\LogBridgeBundle\Formatter\FormatterInterface; use M6Web\Bundle\LogBridgeBundle\Matcher\MatcherInterface; use Psr\Log\LoggerInterface; -use Symfony\Component\HttpKernel\Event\ResponseEvent; +use Symfony\Component\HttpKernel\Event\TerminateEvent; /** * LogRequestListener @@ -22,7 +22,7 @@ public function __construct(protected FormatterInterface $contentFormatter) { } - public function onKernelTerminate(ResponseEvent $event): void + public function onKernelTerminate(TerminateEvent $event): void { if ($this->matcher === null) { return;