Skip to content

Commit b13e7ce

Browse files
committed
bug #60515 [Mailer] Fix memory leak with mailer.message_logger_listener (andrew-demb)
This PR was squashed before being merged into the 7.3 branch. Discussion ---------- [Mailer] Fix memory leak with `mailer.message_logger_listener` | Q | A | ------------- | --- | Branch? | 7.3 | Bug fix? | yes | New feature? | no | Deprecations? | no <!-- please update UPGRADE-*.md and src/**/CHANGELOG.md files --> | Issues | Fix #50580 | License | MIT Register `mailer.message_logger_listener` only when profiler enabled to prevent memory leaks as suggested in: - symfony/symfony#50580 (comment) - symfony/symfony#50580 (comment) ~~`@stof` can you guide me about the way how to implement your second suggestion too? symfony/symfony#50580 (comment) UPD: resolved _This change may be considered as a minor BC break - DI service won't be available anymore with disabled profiler._ Commits ------- b63317d2cf6 [Mailer] Fix memory leak with `mailer.message_logger_listener`
2 parents f23354a + fb5bb74 commit b13e7ce

File tree

2 files changed

+8
-5
lines changed

2 files changed

+8
-5
lines changed

Resources/config/mailer.php

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -15,7 +15,6 @@
1515
use Symfony\Component\Mailer\EventListener\DkimSignedMessageListener;
1616
use Symfony\Component\Mailer\EventListener\EnvelopeListener;
1717
use Symfony\Component\Mailer\EventListener\MessageListener;
18-
use Symfony\Component\Mailer\EventListener\MessageLoggerListener;
1918
use Symfony\Component\Mailer\EventListener\MessengerTransportListener;
2019
use Symfony\Component\Mailer\EventListener\SmimeEncryptedMessageListener;
2120
use Symfony\Component\Mailer\EventListener\SmimeSignedMessageListener;
@@ -72,10 +71,6 @@
7271
])
7372
->tag('kernel.event_subscriber')
7473

75-
->set('mailer.message_logger_listener', MessageLoggerListener::class)
76-
->tag('kernel.event_subscriber')
77-
->tag('kernel.reset', ['method' => 'reset'])
78-
7974
->set('mailer.messenger_transport_listener', MessengerTransportListener::class)
8075
->tag('kernel.event_subscriber')
8176

Resources/config/mailer_debug.php

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -12,9 +12,17 @@
1212
namespace Symfony\Component\DependencyInjection\Loader\Configurator;
1313

1414
use Symfony\Component\Mailer\DataCollector\MessageDataCollector;
15+
use Symfony\Component\Mailer\EventListener\MessageLoggerListener;
1516

1617
return static function (ContainerConfigurator $container) {
1718
$container->services()
19+
->set('mailer.message_logger_listener', MessageLoggerListener::class)
20+
->args([
21+
service('profiler.is_disabled_state_checker')->nullOnInvalid(),
22+
])
23+
->tag('kernel.event_subscriber')
24+
->tag('kernel.reset', ['method' => 'reset'])
25+
1826
->set('mailer.data_collector', MessageDataCollector::class)
1927
->args([
2028
service('mailer.message_logger_listener'),

0 commit comments

Comments
 (0)