airbyte-source-github 2.1.22__tar.gz → 2.1.23__tar.gz

{airbyte_source_github-2.1.22 → airbyte_source_github-2.1.23}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.1
 Name: airbyte-source-github
-Version: 2.1.22
+Version: 2.1.23
 Summary: Source implementation for GitHub.
 Home-page: https://airbyte.com
 License: ELv2

{airbyte_source_github-2.1.22 → airbyte_source_github-2.1.23}/pyproject.toml

@@ -3,7 +3,7 @@ requires = [ "poetry-core>=1.0.0",]
 build-backend = "poetry.core.masonry.api"
 
 [tool.poetry]
-version = "2.1.22"
+version = "2.1.23"
 name = "airbyte-source-github"
 description = "Source implementation for GitHub."
 authors = [ "Airbyte <contact@airbyte.io>",]

{airbyte_source_github-2.1.22 → airbyte_source_github-2.1.23}/source_github/streams.py

@@ -138,7 +138,11 @@ class GithubStreamABC(HttpStream, ABC):
             # This whole try/except situation in `read_records()` isn't good but right now in `self._send_request()`
             # function we have `response.raise_for_status()` so we don't have much choice on how to handle errors.
             # Bocked on https://github.com/airbytehq/airbyte/issues/3514.
-            if not hasattr(e, "_exception") or not hasattr(e._exception, "response"):
+            # `requests.RequestException` subclasses always expose a `response` attribute, but it
+            # defaults to `None` for transport-layer failures (ConnectionError, ConnectTimeout,
+            # ReadTimeout, SSLError, DNS failures, etc.). Treat a missing or `None` response as
+            # something this handler cannot classify, and let the CDK surface it.
+            if not hasattr(e, "_exception") or getattr(e._exception, "response", None) is None:
                 raise e
             if e._exception.response.status_code == requests.codes.NOT_FOUND:
                 # A lot of streams are not available for repositories owned by a user instead of an organization.