Skip to content

Commit 90c9492

Browse files
authored
Remove custom provider as sdk is now fixed (#1148)
## Description - The custom azure u2m was a pain because we had to make the sdk flow for aws inline with the custom flow : this is not possible. Now that sdk has fixed their flows, we should start using it for azure too. ## Testing - Tested it manually using Azure U2M
1 parent ca6ace6 commit 90c9492

7 files changed

Lines changed: 68 additions & 1342 deletions

File tree

NEXT_CHANGELOG.md

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -15,7 +15,7 @@
1515
- Removes dangerous global timezone modification that caused race conditions.
1616
- Fixed `Statement.getLargeUpdateCount()` to return -1 instead of throwing Exception when there were no more results or result is not an update count.
1717
- CVE-2025-66566. Updated lz4-java dependency to 1.10.1.
18-
- Fix `INVALID_IDENTIFIER` error when using catalog/schema/table names for SQL Exec API with hyphens or special characters in metadata operations (`getSchemas()`, `getTables()`, `getColumns()`, etc.) and connection methods (`setCatalog()`, `setSchema()`). Per Databricks identifier rules, special characters are now properly enclosed in backticks.
19-
18+
- Fix `INVALID_IDENTIFIER` error when using catalog/schema/table names for SQL Exec API with hyphens or special characters in metadata operations (`getSchemas()`, `getTables()`, `getColumns()`, etc.) and connection methods (`setCatalog()`, `setSchema()`). Per Databricks identifier rules, special characters are now properly enclosed in backticks.
19+
- Fix Auth_Scope handling inconsistency in Azure U2M OAuth.
2020
---
2121
*Note: When making changes, please add your change under the appropriate section with a brief description.*

src/main/java/com/databricks/jdbc/api/impl/DatabricksConnectionContext.java

Lines changed: 3 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -333,13 +333,9 @@ public List<String> getOAuthScopesForU2M() throws DatabricksParsingException {
333333
if (getParameter(AUTH_SCOPE) != null) {
334334
return Collections.singletonList(getAuthScope());
335335
}
336-
if (getCloud() == Cloud.AWS || getCloud() == Cloud.GCP) {
337-
return Arrays.asList(
338-
DatabricksJdbcConstants.SQL_SCOPE, DatabricksJdbcConstants.OFFLINE_ACCESS_SCOPE);
339-
} else {
340-
// Default scope is already being set for Azure in databricks-sdk.
341-
return null;
342-
}
336+
// Use uniform default scopes for all clouds: sql and offline_access
337+
return Arrays.asList(
338+
DatabricksJdbcConstants.SQL_SCOPE, DatabricksJdbcConstants.OFFLINE_ACCESS_SCOPE);
343339
}
344340

345341
@Override

0 commit comments

Comments
 (0)