Vamsi-klu commented on PR #68672:
URL: https://github.com/apache/airflow/pull/68672#issuecomment-4764226473

   There are a few blockers in the new GCP WIF auth path:
   - `_get_token()` calls `_get_wif_databricks_token()` but does not return it, 
so sync calls fall through and `raise_error=True` paths still raise `Token 
authentication isn't configured`.
   - `_a_get_token()` has no `use_gcp_workload_federation` branch, so 
async/deferrable Databricks calls cannot use this auth mode.
   - The Google ID token audience uses raw `databricks_conn.host`; it should 
normalize to the full workspace URL, e.g. `https://{self.host}`, or host values 
stored without a scheme generate the wrong `aud`.
   - The `google.auth` imports need dependency handling/docs, or an 
`AirflowOptionalProviderFeatureException`, because a standard Databricks 
provider install does not include the Google auth dependency.


-- 
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]

Reply via email to