From cd941d33cdbd0810e281f3b11a7ecc85c393ee07 Mon Sep 17 00:00:00 2001 From: Andrii Date: Thu, 22 May 2025 17:35:16 +0200 Subject: [PATCH] :package: [#50580] Register `mailer.message_logger_listener` only on when profiler enabled to prevent memory leaks --- src/Symfony/Bundle/FrameworkBundle/CHANGELOG.md | 1 + .../Bundle/FrameworkBundle/Resources/config/mailer.php | 4 ---- .../Bundle/FrameworkBundle/Resources/config/mailer_debug.php | 5 +++++ 3 files changed, 6 insertions(+), 4 deletions(-) diff --git a/src/Symfony/Bundle/FrameworkBundle/CHANGELOG.md b/src/Symfony/Bundle/FrameworkBundle/CHANGELOG.md index ce62c9cdf836b..c33e9d2ee2c8f 100644 --- a/src/Symfony/Bundle/FrameworkBundle/CHANGELOG.md +++ b/src/Symfony/Bundle/FrameworkBundle/CHANGELOG.md @@ -4,6 +4,7 @@ CHANGELOG 7.3 --- + * `mailer.message_logger_listener` now registered in DI only when profiler enabled to prevent memory leaks * Add `errors.php` and `webhook.php` routing configuration files (use them instead of their XML equivalent) Before: diff --git a/src/Symfony/Bundle/FrameworkBundle/Resources/config/mailer.php b/src/Symfony/Bundle/FrameworkBundle/Resources/config/mailer.php index 43e7fb9a5e4cb..e7ad8a5a5cb25 100644 --- a/src/Symfony/Bundle/FrameworkBundle/Resources/config/mailer.php +++ b/src/Symfony/Bundle/FrameworkBundle/Resources/config/mailer.php @@ -72,10 +72,6 @@ ]) ->tag('kernel.event_subscriber') - ->set('mailer.message_logger_listener', MessageLoggerListener::class) - ->tag('kernel.event_subscriber') - ->tag('kernel.reset', ['method' => 'reset']) - ->set('mailer.messenger_transport_listener', MessengerTransportListener::class) ->tag('kernel.event_subscriber') diff --git a/src/Symfony/Bundle/FrameworkBundle/Resources/config/mailer_debug.php b/src/Symfony/Bundle/FrameworkBundle/Resources/config/mailer_debug.php index cdb205750f05d..85f1fff5894cc 100644 --- a/src/Symfony/Bundle/FrameworkBundle/Resources/config/mailer_debug.php +++ b/src/Symfony/Bundle/FrameworkBundle/Resources/config/mailer_debug.php @@ -12,9 +12,14 @@ namespace Symfony\Component\DependencyInjection\Loader\Configurator; use Symfony\Component\Mailer\DataCollector\MessageDataCollector; +use Symfony\Component\Mailer\EventListener\MessageLoggerListener; return static function (ContainerConfigurator $container) { $container->services() + ->set('mailer.message_logger_listener', MessageLoggerListener::class) + ->tag('kernel.event_subscriber') + ->tag('kernel.reset', ['method' => 'reset']) + ->set('mailer.data_collector', MessageDataCollector::class) ->args([ service('mailer.message_logger_listener'),