minor #62067 [MonologBridge] Improve error in logstash handler when the HttpClient component is not installed (GromNaN)

nicolas-grekas · nicolas-grekas · commit 3e5fc65536ba · 2025-10-14T17:03:43.000+02:00
This PR was merged into the 6.4 branch. Discussion ---------- [MonologBridge] Improve error in logstash handler when the HttpClient component is not installed | Q | A | ------------- | --- | Branch? | 6.4 | Bug fix? | yes | New feature? | no | Deprecations? | no | Issues | Fix symfony/monolog-bundle#498 | License | MIT When the `symfony/http-client-contracts` is installed, but not `symfony/http-client` component, the user doesn't get the informative error message. Fixed the message by checking if `HttpClient` class is available when `$client` parameter is not provider. Commits ------- c47df37 [MonologBridge] Improve error when HttpClient contract is installed but not the component
diff --git a/src/Symfony/Bridge/Monolog/Handler/ElasticsearchLogstashHandler.php b/src/Symfony/Bridge/Monolog/Handler/ElasticsearchLogstashHandler.php
@@ -63,7 +63,7 @@ class ElasticsearchLogstashHandler extends AbstractHandler
 
     public function __construct(string $endpoint = 'http://127.0.0.1:9200', string $index = 'monolog', ?HttpClientInterface $client = null, string|int|Level $level = Logger::DEBUG, bool $bubble = true, string $elasticsearchVersion = '1.0.0')
     {
-        if (!interface_exists(HttpClientInterface::class)) {
+        if (!$client && !class_exists(HttpClient::class)) {
             throw new \LogicException(\sprintf('The "%s" handler needs an HTTP client. Try running "composer require symfony/http-client".', __CLASS__));
         }
 
