LITE-28626 reduce default connect timeout and retry also on exceptions (no response)

Author: Francesco Faraone

connect/client/fluent.py

@@ -36,7 +36,7 @@ def __init__(
         default_limit=100,
         max_retries=3,
         logger=None,
-        timeout=(180.0, 180.0),
+        timeout=(15.0, 180.0),
         resourceset_append=True,
     ):
         if default_headers and 'Authorization' in default_headers:

connect/client/mixins.py

@@ -102,7 +102,7 @@ def _execute_http_call(self, method, url, kwargs):  # noqa: CCR001
                 self.response = self._session.request(method, url, **kwargs)
                 if self.logger:
                     self.logger.log_response(self.response)
-            except Timeout:
+            except RequestException:
                 if retry_count < self.max_retries:
                     retry_count += 1
                     time.sleep(1)
@@ -208,18 +208,25 @@ async def _execute_http_call(self, method, url, kwargs):
             if self.logger:
                 self.logger.log_request(method, url, kwargs)
 
-            self.response = await self._session.request(method, url, **kwargs)
-
-            if self.logger:
-                self.logger.log_response(self.response)
+            try:
+                self.response = await self._session.request(method, url, **kwargs)
 
+                if self.logger:
+                    self.logger.log_response(self.response)
+            except HTTPError:
+                if retry_count < self.max_retries:
+                    retry_count += 1
+                    time.sleep(1)
+                    continue
+                raise
             if (  # pragma: no branch
                 self.response.status_code >= 500 and retry_count < self.max_retries
             ):
                 retry_count += 1
                 time.sleep(1)
                 continue
             break  # pragma: no cover
+
         if self.response.status_code >= 400:
             self.response.raise_for_status()