Skip to content

Commit

Permalink
update m2m to not include offline_access. Per Jacky, it is not needed…
Browse files Browse the repository at this point in the history
… for m2m, and it's breaking OAuth for AWS.
  • Loading branch information
benc-db committed Sep 11, 2023
1 parent c80c557 commit 0770d41
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions dbt/adapters/databricks/auth.py
Original file line number Diff line number Diff line change
Expand Up @@ -40,12 +40,12 @@ def noop_credentials(_: Any): # type: ignore

config = Config(host=host, credentials_provider=noop_credentials)
oidc = config.oidc_endpoints
scopes = ["offline_access", "all-apis"]
scopes = ["all-apis"]
if not oidc:
raise ValueError(f"{host} does not support OAuth")
if config.is_azure:
# Azure AD only supports full access to Azure Databricks.
scopes = [f"{config.effective_azure_login_app_id}/.default", "offline_access"]
scopes = [f"{config.effective_azure_login_app_id}/.default"]
self._token_source = ClientCredentials(
client_id=client_id,
client_secret=client_secret,
Expand Down

0 comments on commit 0770d41

Please sign in to comment.