[Mailchimp] use Httpv2

parti-renaissance · Nov 4, 2024 · 42da3c4 · 42da3c4
1 parent 092cabd
commit 42da3c4
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 4 deletions.
diff --git a/config/packages/http_client.yaml b/config/packages/http_client.yaml
@@ -29,7 +29,6 @@ framework:
                 headers:
                     'User-Agent': 'Renaissance'
                 timeout: 60
-                http_version: '1.1'
             ohme.client:
                 base_uri: 'https://api-ohme.oneheart.fr/api/v1/'
                 timeout: 60

diff --git a/src/Mailchimp/Driver.php b/src/Mailchimp/Driver.php
@@ -231,10 +231,14 @@ public function getLastError(): ?string
 
     public function isSuccessfulResponse(?ResponseInterface $response, bool $log = false): bool
     {
-        $isSuccessful = $response && 200 <= $response->getStatusCode() && $response->getStatusCode() < 300;
+        try {
+            $isSuccessful = $response && 200 <= $response->getStatusCode() && $response->getStatusCode() < 300;
+        } catch (TransportExceptionInterface $e) {
+            $isSuccessful = false;
+        }
 
-        if (!$isSuccessful && $log && $response) {
-            $this->logger->error(\sprintf('[API] Error: %s', $response->getContent(false)));
+        if (!$isSuccessful && $log) {
+            $this->logger->error(\sprintf('[API] Error: %s', $response ? $response->getContent(false) : 'unknown'));
         }
 
         return $isSuccessful;