[Monolog] Disable DebugLogger in CLI

lyrixx · lyrixx · commit 17533da49ce7 · 2019-02-24T14:22:28.000+01:00
diff --git a/src/Symfony/Bridge/Monolog/Logger.php b/src/Symfony/Bridge/Monolog/Logger.php
@@ -75,6 +75,21 @@ public function reset()
         $this->clear();
     }
 
+    public function removeDebugLogger()
+    {
+        foreach ($this->processors as $k => $processor) {
+            if ($processor instanceof DebugLoggerInterface) {
+                unset($this->processors[$k]);
+            }
+        }
+
+        foreach ($this->handlers as $k => $handler) {
+            if ($handler instanceof DebugLoggerInterface) {
+                unset($this->handlers[$k]);
+            }
+        }
+    }
+
     /**
      * Returns a DebugLoggerInterface instance if one is registered with this logger.
      *
diff --git a/src/Symfony/Bundle/FrameworkBundle/DependencyInjection/Compiler/AddDebugLogProcessorPass.php b/src/Symfony/Bundle/FrameworkBundle/DependencyInjection/Compiler/AddDebugLogProcessorPass.php
@@ -30,6 +30,14 @@ public function process(ContainerBuilder $container)
         }
 
         $definition = $container->getDefinition('monolog.logger_prototype');
+        $definition->setConfigurator([__CLASS__, 'configureLogger']);
         $definition->addMethodCall('pushProcessor', [new Reference('debug.log_processor')]);
     }
+
+    public static function configureLogger($logger)
+    {
+        if (method_exists($logger, 'removeDebugLogger') && \in_array(\PHP_SAPI, ['cli', 'phpdbg'], true)) {
+            $logger->removeDebugLogger();
+        }
+    }
 }

Original file line number	Diff line number	Diff line change
`@@ -30,6 +30,14 @@ public function process(ContainerBuilder $container)`
`30`	`30`	`}`
`31`	`31`
`32`	`32`	`$definition = $container->getDefinition('monolog.logger_prototype');`
	`33`	`+ $definition->setConfigurator([__CLASS__, 'configureLogger']);`
`33`	`34`	`$definition->addMethodCall('pushProcessor', [new Reference('debug.log_processor')]);`
`34`	`35`	`}`
	`36`	`+`
	`37`	`+ public static function configureLogger($logger)`
	`38`	`+ {`
	`39`	`+ if (method_exists($logger, 'removeDebugLogger') && \in_array(\PHP_SAPI, ['cli', 'phpdbg'], true)) {`
	`40`	`+ $logger->removeDebugLogger();`
	`41`	`+ }`
	`42`	`+ }`
`35`	`43`	`}`