bug #331 Use an HttpClient without logger in all handlers (lyrixx)

This PR was merged into the 3.x-dev branch.

Discussion
----------

Use an HttpClient without logger in all handlers

fixes:

* symfony/symfony#34423
* symfony/symfony#32360 (comment)
* symfony/symfony-docs#12642

Commits
-------

faf6943 Use an HttpClient without logger in all handlers

Author: lyrixx

DependencyInjection/MonologExtension.php

@@ -13,18 +13,21 @@
 
 use Monolog\Logger;
 use Monolog\Processor\ProcessorInterface;
+use Monolog\Handler\HandlerInterface;
 use Monolog\ResettableInterface;
 use Symfony\Bridge\Monolog\Handler\FingersCrossed\HttpCodeActivationStrategy;
 use Symfony\Bridge\Monolog\Processor\TokenProcessor;
 use Symfony\Bridge\Monolog\Processor\WebProcessor;
 use Symfony\Bundle\FullStack;
-use Symfony\Component\HttpKernel\DependencyInjection\Extension;
-use Symfony\Component\DependencyInjection\Loader\XmlFileLoader;
-use Symfony\Component\DependencyInjection\ContainerBuilder;
 use Symfony\Component\Config\FileLocator;
+use Symfony\Component\DependencyInjection\Argument\BoundArgument;
+use Symfony\Component\DependencyInjection\ContainerBuilder;
 use Symfony\Component\DependencyInjection\Definition;
+use Symfony\Component\DependencyInjection\Loader\XmlFileLoader;
 use Symfony\Component\DependencyInjection\Reference;
+use Symfony\Component\HttpKernel\DependencyInjection\Extension;
 use Symfony\Component\HttpKernel\Kernel;
+use Symfony\Contracts\HttpClient\HttpClientInterface;
 
 /**
  * MonologExtension is an extension for the Monolog library.
@@ -126,6 +129,12 @@ public function load(array $configs, ContainerBuilder $container)
             }
             $container->registerForAutoconfiguration(TokenProcessor::class)
                 ->addTag('monolog.processor');
+            if (interface_exists(HttpClientInterface::class)) {
+                $handlerAutoconfiguration = $container->registerForAutoconfiguration(HandlerInterface::class);
+                $handlerAutoconfiguration->setBindings($handlerAutoconfiguration->getBindings() + [
+                    HttpClientInterface::class => new BoundArgument(new Reference('monolog.http_client'), false),
+                ]);
+            }
         }
     }
 

Resources/config/monolog.xml

@@ -36,5 +36,9 @@
         <service id="monolog.formatter.logstash" class="Monolog\Formatter\LogstashFormatter" public="false">
             <argument index="0">app</argument>
         </service>
+
+        <service id="monolog.http_client" class="Symfony\Contracts\HttpClient\HttpClientInterface" public="false">
+            <factory class="Symfony\Component\HttpClient\HttpClient" method="create" />
+        </service>
     </services>
 </container>