Skip to content

Upgrade SDK - to support discovery url on U2M (AWS and GCP flows)#1011

Merged
samikshya-db merged 7 commits into
databricks:mainfrom
samikshya-db:samikshya-chand_data/upgradesdk
Sep 18, 2025
Merged

Upgrade SDK - to support discovery url on U2M (AWS and GCP flows)#1011
samikshya-db merged 7 commits into
databricks:mainfrom
samikshya-db:samikshya-chand_data/upgradesdk

Conversation

@samikshya-db

@samikshya-db samikshya-db commented Sep 17, 2025

Copy link
Copy Markdown
Collaborator

Description

Testing

Tested that discovery URL is being honored in U2M as well. Tested this using the OSS JDBC with the following URL

  1. Success flow test :
String jdbcUrl =
        "jdbc:databricks://e2-dogfood.staging.cloud.databricks.com:443/default;transportMode=http;ssl=1;httpPath=/sql/1.0/warehouses/dd43ee29fedd958d;"
            + "AuthMech=11;Auth_Flow=2;LogLevel=6;UseThriftClient=0;EnableTokenCache=0;OIDCDiscoveryEndpoint=https://e2-dogfood.staging.cloud.databricks.com/oidc/.well-known/oauth-authorization-server";
  1. Failure flow test :
String jdbcUrl =
        "jdbc:databricks://e2-dogfood.staging.cloud.databricks.com:443/default;transportMode=http;ssl=1;httpPath=/sql/1.0/warehouses/dd43ee29fedd958d;"
            + "AuthMech=11;Auth_Flow=2;LogLevel=6;UseThriftClient=0;EnableTokenCache=0;OIDCDiscoveryEndpoint=https://google.com";

Also tested using GCP :

"jdbc:databricks://6177827686947384.4.gcp.databricks.com:443/default;transportMode=http;ssl=1;EnableTokenCache=0;AuthMech=11;Auth_Flow=2;httpPath=/sql/1.0/warehouses/f5e97c8f8dfb56b5;OIDCDiscoveryEndpoint=https://6177827686947384.4.gcp.databricks.com/oidc/.well-known/oauth-authorization-server";
   

@samikshya-db samikshya-db marked this pull request as ready for review September 17, 2025 18:21
@samikshya-db samikshya-db changed the title Upgrade SDK Upgrade SDK - to support discovery url on U2M (AWS and GCP flows) Sep 17, 2025
@samikshya-db samikshya-db enabled auto-merge (squash) September 18, 2025 07:51
@samikshya-db samikshya-db merged commit c0e788f into databricks:main Sep 18, 2025
12 of 13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants