airbyte-source-github 2.1.15.dev202603161151__tar.gz → 2.1.16__tar.gz

{airbyte_source_github-2.1.15.dev202603161151 → airbyte_source_github-2.1.16}/PKG-INFO

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

{airbyte_source_github-2.1.15.dev202603161151 → airbyte_source_github-2.1.16}/pyproject.toml

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

{airbyte_source_github-2.1.15.dev202603161151 → airbyte_source_github-2.1.16}/source_github/source.py

@@ -9,7 +9,6 @@ from urllib.parse import urlparse
 from airbyte_cdk.models import FailureType
 from airbyte_cdk.sources import AbstractSource
 from airbyte_cdk.sources.streams import Stream
-from airbyte_cdk.sources.streams.http.http_client import MessageRepresentationAirbyteTracedErrors
 from airbyte_cdk.sources.streams.http.requests_native_auth import MultipleTokenAuthenticator
 from airbyte_cdk.utils.traced_exception import AirbyteTracedException
 from source_github.utils import MultipleTokenAuthenticatorWithRateLimiter
@@ -204,7 +203,7 @@ class SourceGithub(AbstractSource):
                 )
             return True, None
 
-        except MessageRepresentationAirbyteTracedErrors as e:
+        except AirbyteTracedException as e:
             user_message = self.user_friendly_error_message(e.message)
             return False, user_message or e.message
         except Exception as e:

{airbyte_source_github-2.1.15.dev202603161151 → airbyte_source_github-2.1.16}/source_github/spec.json

@@ -149,6 +149,20 @@
     "predicate_key": ["credentials", "option_title"],
     "predicate_value": "OAuth Credentials",
     "oauth_config_specification": {
+      "oauth_connector_input_specification": {
+        "scopes": [
+          { "scope": "repo" },
+          { "scope": "read:org" },
+          { "scope": "read:repo_hook" },
+          { "scope": "read:user" },
+          { "scope": "read:project" },
+          { "scope": "read:discussion" },
+          { "scope": "workflow" }
+        ],
+        "consent_url": "https://github.com/login/oauth/authorize?{{client_id_param}}&{{redirect_uri_param}}&{{scopes_param}}&{{state_param}}",
+        "access_token_url": "https://github.com/login/oauth/access_token?{{client_id_param}}&{{client_secret_param}}&{{auth_code_param}}&{{redirect_uri_param}}",
+        "extract_output": ["access_token"]
+      },
       "complete_oauth_output_specification": {
         "type": "object",
         "additionalProperties": false,

{airbyte_source_github-2.1.15.dev202603161151 → airbyte_source_github-2.1.16}/source_github/streams.py

@@ -22,7 +22,6 @@ from airbyte_cdk.sources.streams.core import CheckpointMixin, Stream
 from airbyte_cdk.sources.streams.http import HttpStream
 from airbyte_cdk.sources.streams.http.error_handlers import ErrorHandler, ErrorResolution, HttpStatusErrorHandler, ResponseAction
 from airbyte_cdk.sources.streams.http.exceptions import DefaultBackoffException, UserDefinedBackoffException
-from airbyte_cdk.sources.streams.http.http_client import MessageRepresentationAirbyteTracedErrors
 from airbyte_cdk.utils import AirbyteTracedException
 from airbyte_cdk.utils.datetime_helpers import ab_datetime_format, ab_datetime_parse
 
@@ -133,12 +132,12 @@ class GithubStreamABC(HttpStream, ABC):
         # Reading records while handling the errors
         try:
             yield from super().read_records(stream_slice=stream_slice, **kwargs)
-        # HTTP Client wraps DefaultBackoffException into MessageRepresentationAirbyteTracedErrors
-        except MessageRepresentationAirbyteTracedErrors as e:
+        # HTTP Client wraps DefaultBackoffException into AirbyteTracedException
+        except AirbyteTracedException as e:
             # 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") or e._exception.response is None:
+            if not hasattr(e, "_exception") and 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.
@@ -1813,7 +1812,7 @@ class ContributorActivity(GithubStream):
             yield from super().read_records(stream_slice=stream_slice, **kwargs)
         # HTTP Client wraps BackoffException into MessageRepresentationAirbyteTracedErrors
         except MessageRepresentationAirbyteTracedErrors as e:
-            if hasattr(e, "_exception") and hasattr(e._exception, "response") and e._exception.response is not None:
+            if hasattr(e, "_exception") and hasattr(e._exception, "response"):
                 if e._exception.response.status_code == requests.codes.ACCEPTED:
                     yield AirbyteMessage(
                         type=MessageType.LOG,