aoelvp94 opened a new pull request, #64986: URL: https://github.com/apache/airflow/pull/64986
## Summary The `AirbyteHook` unconditionally requires OAuth client credentials and calls `/v1/applications/token` on every request. This endpoint does not exist on Airbyte OSS v2.x without Keycloak/Enterprise auth, causing a `JSONDecodeError` (the token endpoint returns HTML instead of JSON) and breaking the provider for all self-hosted OSS deployments. This PR makes OAuth credentials optional: - When `client_id` and `client_secret` are provided → existing OAuth flow (Airbyte Cloud/Enterprise) - When omitted → session created with `security=None` (Airbyte OSS without auth) closes: #47161 ## Changes - **`hooks/airbyte.py`**: `create_api_session()` skips OAuth when no credentials are configured; UI labels updated to mark Client ID/Secret as optional - **`docs/connections.rst`**: documents both authenticated and unauthenticated connection setups - **`tests/unit/airbyte/hooks/test_airbyte.py`**: adds tests for no-credentials session, with-credentials session, and sync submission without auth ## Testing All 33 Airbyte provider tests pass (ran via `breeze testing providers-tests`). Verified against a live Airbyte OSS v2.0.1 cluster on EKS — the `airbyte-api` SDK works with `security=None` out of the box. --- ##### Was generative AI tooling used to co-author this PR? - [X] Yes (please specify the tool below) Generated-by: Claude Code following [the guidelines](https://github.com/apache/airflow/blob/main/contributing-docs/05_pull_requests.rst#gen-ai-assisted-contributions) -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
