airbyte-source-github 2.1.17__tar.gz → 2.1.19__tar.gz

{airbyte_source_github-2.1.17 → airbyte_source_github-2.1.19}/PKG-INFO

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

{airbyte_source_github-2.1.17 → airbyte_source_github-2.1.19}/pyproject.toml

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

{airbyte_source_github-2.1.17 → airbyte_source_github-2.1.19}/source_github/errors_handlers.py

@@ -21,9 +21,9 @@ GITHUB_DEFAULT_ERROR_MAPPING = DEFAULT_ERROR_MAPPING | {
         error_message="Conflict.",
     ),
     403: ErrorResolution(
-        response_action=ResponseAction.RETRY,
+        response_action=ResponseAction.FAIL,
         failure_type=FailureType.config_error,
-        error_message="Conflict.",
+        error_message="Access denied due to insufficient permissions.",
     ),
     404: ErrorResolution(
         response_action=ResponseAction.RETRY,
@@ -106,7 +106,7 @@ class GithubStreamABCErrorHandler(HttpStatusErrorHandler):
         return super().interpret_response(response_or_exception)
 
 
-class ContributorActivityErrorHandler(HttpStatusErrorHandler):
+class ContributorActivityErrorHandler(GithubStreamABCErrorHandler):
     """
     This custom error handler is needed for streams based on repository statistics endpoints like ContributorActivity because
     when requesting data that hasn't been cached yet when the request is made, you'll receive a 202 response. And these requests

{airbyte_source_github-2.1.17 → airbyte_source_github-2.1.19}/source_github/streams.py

@@ -3,6 +3,7 @@
 #
 
 import base64
+import binascii
 import re
 import struct
 from abc import ABC, abstractmethod
@@ -137,7 +138,7 @@ 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") and not hasattr(e._exception, "response"):
+            if not hasattr(e, "_exception") or not hasattr(e._exception, "response"):
                 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.
@@ -827,7 +828,7 @@ class Releases(SemiIncrementalMixin, GitHubGraphQLStream):
             decoded = base64.urlsafe_b64decode(encoded + "==")
             if len(decoded) >= 4:
                 return struct.unpack(">I", decoded[-4:])[0]
-        except Exception:
+        except (ValueError, struct.error, binascii.Error):
             return None
         return None
 
@@ -1787,7 +1788,7 @@ class ContributorActivity(GithubStream):
         return record
 
     def get_error_handler(self) -> Optional[ErrorHandler]:
-        return ContributorActivityErrorHandler(logger=self.logger, max_retries=5, error_mapping=GITHUB_DEFAULT_ERROR_MAPPING)
+        return ContributorActivityErrorHandler(stream=self, logger=self.logger, max_retries=5, error_mapping=GITHUB_DEFAULT_ERROR_MAPPING)
 
     def get_backoff_strategy(self) -> Optional[Union[BackoffStrategy, List[BackoffStrategy]]]:
         return ContributorActivityBackoffStrategy()
@@ -1828,6 +1829,8 @@ class ContributorActivity(GithubStream):
                     partition_obj = stream_slice.get("partition")
                     if self.cursor and partition_obj:
                         self.cursor.close_slice(StreamSlice(cursor_slice={}, partition=partition_obj))
+                else:
+                    raise e
             else:
                 raise e