Merge pull request #189 from delyriand/fix/elasticsearch-checker-host…

…-config Add config with url in checker to avoid error couldn’t resolve host
monsieurbiz · Aug 30, 2023 · 0393453 · 0393453
2 parents 1930ec3 + f326f58
commit 0393453
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 1 deletion.
diff --git a/src/Checker/ElasticsearchChecker.php b/src/Checker/ElasticsearchChecker.php
@@ -19,10 +19,17 @@ class ElasticsearchChecker implements ElasticsearchCheckerInterface
 {
     private ?bool $isAvailable = null;
 
+    private array $config;
+
+    public function __construct(array $config = [])
+    {
+        $this->config = $config;
+    }
+
     public function check(): bool
     {
         if (null === $this->isAvailable) {
-            $client = (new Factory())->buildClient();
+            $client = (new Factory($this->config))->buildClient();
 
             // Check client response
             try {

diff --git a/src/Resources/config/services.yaml b/src/Resources/config/services.yaml
@@ -255,6 +255,9 @@ services:
 
   monsieurbiz.search.checker.elasticsearch_checker:
     class: MonsieurBiz\SyliusSearchPlugin\Checker\ElasticsearchChecker
+    arguments:
+      $config:
+        url: '%monsieurbiz_search_elasticsearch_url%'
 
   MonsieurBiz\SyliusSearchPlugin\Twig\Extension\RenderSearchForm:
     arguments: