feature #43322 [MonologBridge] Deprecates ResetLoggersWorkerSubscribe…

…r (lyrixx) This PR was merged into the 5.4 branch. Discussion ---------- [MonologBridge] Deprecates ResetLoggersWorkerSubscriber | Q | A | ------------- | --- | Branch? | 5.4 | Bug fix? | no | New feature? | yes | Deprecations? | yes | Tickets | Fix #43271 | License | MIT | Doc PR | Commits ------- 641e50a [MonologBridge] Deprecates ResetLoggersWorkerSubscriber
symfony · Oct 6, 2021 · 4868dd0 · 4868dd0
2 parents 02e6f4d + 641e50a
commit 4868dd0
Show file tree

Hide file tree

Showing 3 changed files with 11 additions and 0 deletions.
diff --git a/src/Symfony/Bridge/Monolog/CHANGELOG.md b/src/Symfony/Bridge/Monolog/CHANGELOG.md
@@ -1,6 +1,12 @@
 CHANGELOG
 =========
 
+5.4
+---
+
+ * Deprecate `ResetLoggersWorkerSubscriber` to reset buffered logs in messenger
+   workers, use "reset_on_message" option in messenger configuration instead.
+
 5.3
 ---
 

diff --git a/src/Symfony/Bridge/Monolog/Messenger/ResetLoggersWorkerSubscriber.php b/src/Symfony/Bridge/Monolog/Messenger/ResetLoggersWorkerSubscriber.php
@@ -16,10 +16,14 @@
 use Symfony\Component\Messenger\Event\WorkerMessageFailedEvent;
 use Symfony\Component\Messenger\Event\WorkerMessageHandledEvent;
 
+trigger_deprecation('symfony/monolog-bridge', '5.4', 'The "%s" class is deprecated, use "reset_on_message" option in messenger configuration instead.', ResetLoggersWorkerSubscriber::class);
+
 /**
  * Reset loggers between messages being handled to release buffered handler logs.
  *
  * @author Laurent VOULLEMIER <laurent.voullemier@gmail.com>
+ *
+ * @deprecated since Symfony 5.4, use "reset_on_message" option in messenger configuration instead.
  */
 class ResetLoggersWorkerSubscriber implements EventSubscriberInterface
 {

diff --git a/src/Symfony/Bridge/Monolog/Tests/Messenger/ResetLoggersWorkerSubscriberTest.php b/src/Symfony/Bridge/Monolog/Tests/Messenger/ResetLoggersWorkerSubscriberTest.php
@@ -25,6 +25,7 @@
 use Symfony\Component\Messenger\Transport\Receiver\ReceiverInterface;
 use Symfony\Component\Messenger\Worker;
 
+/** @group legacy */
 class ResetLoggersWorkerSubscriberTest extends TestCase
 {
     public function testLogsAreFlushed()