airbyte-cdk 6.7.1rc3__tar.gz → 6.7.1rc4__tar.gz

{airbyte_cdk-6.7.1rc3 → airbyte_cdk-6.7.1rc4}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.1
 Name: airbyte-cdk
-Version: 6.7.1rc3
+Version: 6.7.1rc4
 Summary: A framework for writing Airbyte Connectors.
 Home-page: https://airbyte.com
 License: MIT

{airbyte_cdk-6.7.1rc3 → airbyte_cdk-6.7.1rc4}/airbyte_cdk/sources/declarative/retrievers/simple_retriever.py

@@ -360,6 +360,9 @@ class SimpleRetriever(Retriever):
                 next_page_token = self._next_page_token(response)
                 if not next_page_token:
                     pagination_complete = True
+                # Closing the response to avoid memory issues. Note that this assumes the caller as completely consumed the response before
+                # iterating on another one
+                response.close()
 
         # Always return an empty generator just in case no records were ever yielded
         yield from []

{airbyte_cdk-6.7.1rc3 → airbyte_cdk-6.7.1rc4}/airbyte_cdk/sources/streams/http/http_client.py

@@ -6,7 +6,7 @@ import logging
 import os
 import urllib
 from pathlib import Path
-from typing import Any, Callable, Dict, List, Mapping, Optional, Tuple, Union
+from typing import Any, Callable, List, Mapping, Optional, Tuple, Union
 
 import orjson
 import requests
@@ -43,6 +43,7 @@ from airbyte_cdk.sources.streams.http.rate_limiting import (
     rate_limit_default_backoff_handler,
     user_defined_backoff_handler,
 )
+from airbyte_cdk.utils.airbyte_secrets_utils import filter_secrets
 from airbyte_cdk.utils.constants import ENV_REQUEST_CACHE_PATH
 from airbyte_cdk.utils.stream_status_utils import (
     as_airbyte_message as stream_status_as_airbyte_message,
@@ -90,9 +91,11 @@ class HttpClient:
     ):
         self._name = name
         self._api_budget: APIBudget = api_budget or APIBudget(policies=[])
+        self._is_session_owner = False
         if session:
             self._session = session
         else:
+            self._is_session_owner = True
             self._use_cache = use_cache
             self._session = self._request_session()
             self._session.mount(
@@ -113,10 +116,13 @@ class HttpClient:
         else:
             self._backoff_strategies = [DefaultBackoffStrategy()]
         self._error_message_parser = error_message_parser or JsonErrorMessageParser()
-        self._request_attempt_count: Dict[requests.PreparedRequest, int] = {}
         self._disable_retries = disable_retries
         self._message_repository = message_repository
 
+    def __del__(self):
+        if self._is_session_owner:
+            self._session.close()
+
     @property
     def cache_filename(self) -> str:
         """
@@ -269,13 +275,6 @@ class HttpClient:
         log_formatter: Optional[Callable[[requests.Response], Any]] = None,
         exit_on_rate_limit: Optional[bool] = False,
     ) -> requests.Response:
-        if request not in self._request_attempt_count:
-            self._request_attempt_count[request] = 1
-        else:
-            self._request_attempt_count[request] += 1
-            if hasattr(self._session, "auth") and isinstance(self._session.auth, AuthBase):
-                self._session.auth(request)
-
         self._logger.debug(
             "Making outbound API request",
             extra={"headers": request.headers, "url": request.url, "request_body": request.body},
@@ -361,12 +360,13 @@ class HttpClient:
 
         if error_resolution.response_action == ResponseAction.FAIL:
             if response is not None:
-                error_message = f"'{request.method}' request to '{request.url}' failed with status code '{response.status_code}' and error message '{self._error_message_parser.parse_response_error_message(response)}'"
+                error_message = f"'{request.method}' request to '{request.url}' failed with status code '{response.status_code}' and error message '{response.content}'"
             else:
                 error_message = (
                     f"'{request.method}' request to '{request.url}' failed with exception: '{exc}'"
                 )
 
+            self._logger.warning(filter_secrets(error_message))
             raise MessageRepresentationAirbyteTracedErrors(
                 internal_message=error_message,
                 message=error_resolution.error_message or error_message,
@@ -390,7 +390,7 @@ class HttpClient:
             for backoff_strategy in self._backoff_strategies:
                 backoff_time = backoff_strategy.backoff_time(
                     response_or_exception=response if response is not None else exc,
-                    attempt_count=self._request_attempt_count[request],
+                    attempt_count=0,
                 )
                 if backoff_time:
                     user_defined_backoff_time = backoff_time

{airbyte_cdk-6.7.1rc3 → airbyte_cdk-6.7.1rc4}/pyproject.toml

@@ -23,7 +23,7 @@ classifiers = [
 keywords = ["airbyte", "connector-development-kit", "cdk"]
 
 # Python CDK uses dynamic versioning: https://github.com/mtkennerly/poetry-dynamic-versioning
-version = "6.7.1rc3"  # Version will be calculated dynamically.
+version = "6.7.1rc4"  # Version will be calculated dynamically.
 
 [tool.poetry-dynamic-versioning]
 enable = false