Skip to content

Commit 8f1810d

Browse files
hectorcast-dbclaude
andcommitted
Fix OIDC endpoints detection
Fix oidc_endpoints() property. The method was incorrectly returning Azure OIDC endpoints when ARM_CLIENT_ID is set, but oidc_endpoints() is never called for Azure OIDC authentication. This logic actually caused Databricks M2M OAuth to use the wrong endpoint when the user set ARM_CLIENT_ID for other purposes. NOTE: The new logic matches the behavior of the Go SDK and Java SDK. Mirrors: databricks/databricks-sdk-java#657 Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
1 parent dd68e95 commit 8f1810d

1 file changed

Lines changed: 0 additions & 4 deletions

File tree

databricks/sdk/config.py

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -20,7 +20,6 @@
2020
from .environments import (ALL_ENVS, AzureEnvironment, Cloud,
2121
DatabricksEnvironment, get_environment_for_hostname)
2222
from .oauth import (OidcEndpoints, Token, get_account_endpoints,
23-
get_azure_entra_id_workspace_endpoints,
2423
get_unified_endpoints, get_workspace_endpoints)
2524

2625
logger = logging.getLogger("databricks.sdk")
@@ -490,9 +489,6 @@ def oidc_endpoints(self) -> Optional[OidcEndpoints]:
490489
self._fix_host_if_needed()
491490
if not self.host:
492491
return None
493-
if self.is_azure and self.azure_client_id:
494-
return get_azure_entra_id_workspace_endpoints(self.host)
495-
496492
# Handle unified hosts
497493
if self.host_type == HostType.UNIFIED:
498494
if not self.account_id:

0 commit comments

Comments
 (0)