Apply MongoDBFormatter to each "mongodb" handler by default

Author: jmikola

src/DependencyInjection/MonologExtension.php

@@ -324,6 +324,12 @@ private function buildHandler(ContainerBuilder $container, $name, array $handler
                     $handler['level'],
                     $handler['bubble'],
                 ]);
+
+                if (empty($handler['formatter'])) {
+                    $formatter = new Definition('Monolog\Formatter\MongoDBFormatter');
+                    $formatter->setPublic(false);
+                    $definition->addMethodCall('setFormatter', [$formatter]);
+                }
                 break;
 
             case 'elasticsearch':

tests/DependencyInjection/MonologExtensionTest.php

@@ -1048,15 +1048,21 @@ public function testMongoDB()
         $this->assertTrue($container->hasDefinition('monolog.handler.mongodb_with_uri'));
         $this->assertTrue($container->hasDefinition('monolog.handler.mongodb_with_uri_and_default_args'));
 
+        // A MongoDBFormatter will be applied to each handler by default
+        $formatter = new Definition('Monolog\Formatter\MongoDBFormatter');
+        $formatter->setPublic(false);
+
         // MongoDB handler should receive the mongodb.client as first argument
         $handler = $container->getDefinition('monolog.handler.mongodb_with_id');
         $this->assertDICDefinitionClass($handler, MongoDBHandler::class);
         $this->assertDICConstructorArguments($handler, [new Reference('mongodb.client'), 'monolog', 'logs', 'DEBUG', true]);
+        $this->assertDICDefinitionMethodCallAt(1, $handler, 'setFormatter', [$formatter]);
 
         // MongoDB handler should receive the mongodb.client as first argument
         $handler = $container->getDefinition('monolog.handler.mongodb_with_string_id');
         $this->assertDICDefinitionClass($handler, MongoDBHandler::class);
         $this->assertDICConstructorArguments($handler, [new Reference('mongodb.client'), 'monolog', 'logs', 'DEBUG', true]);
+        $this->assertDICDefinitionMethodCallAt(1, $handler, 'setFormatter', [$formatter]);
 
         // MongoDB handler with arguments
         $handler = $container->getDefinition('monolog.handler.mongodb_with_uri');
@@ -1065,6 +1071,7 @@ public function testMongoDB()
         $this->assertDICDefinitionClass($client, 'MongoDB\Client');
         $this->assertDICConstructorArguments($client, ['mongodb://localhost:27018', ['appname' => 'monolog-bundle', 'username' => 'username', 'password' => 'password']]);
         $this->assertDICConstructorArguments($handler, [$client, 'db', 'coll', 'DEBUG', true]);
+        $this->assertDICDefinitionMethodCallAt(1, $handler, 'setFormatter', [$formatter]);
 
         // MongoDB handler with host and default arguments
         $handler = $container->getDefinition('monolog.handler.mongodb_with_uri_and_default_args');
@@ -1073,6 +1080,7 @@ public function testMongoDB()
         $this->assertDICDefinitionClass($client, 'MongoDB\Client');
         $this->assertDICConstructorArguments($client, ['mongodb://localhost:27018', ['appname' => 'monolog-bundle']]);
         $this->assertDICConstructorArguments($handler, [$client, 'monolog', 'logs', 'DEBUG', true]);
+        $this->assertDICDefinitionMethodCallAt(1, $handler, 'setFormatter', [$formatter]);
     }
 
     protected function getContainer(array $config = [], array $thirdPartyDefinitions = []): ContainerBuilder