Format

Author: renaudhartert-db

databricks-sdk-java/src/main/java/com/databricks/sdk/core/oauth/OAuthClient.java

@@ -191,7 +191,7 @@ private static byte[] sha256(byte[] input) {
   }
 
   protected static String urlEncode(String urlBase, Map<String, String> params) {
-    if (params.isEmpty()) {
+    if (params == null || params.isEmpty()) {
       return urlBase;
     }
 
@@ -217,6 +217,11 @@ private static String encodeParam(String value) {
   }
 
   public Consent initiateConsent() throws MalformedURLException {
+    if (authUrl == null) {
+      throw new DatabricksException(
+          "Authorization URL is not configured. OAuth endpoints may not be available.");
+    }
+
     String state = tokenUrlSafe(16);
     String verifier = tokenUrlSafe(32);
     byte[] digest = sha256(verifier.getBytes(StandardCharsets.UTF_8));

databricks-sdk-java/src/test/java/com/databricks/sdk/core/oauth/ExternalBrowserCredentialsProviderTest.java

@@ -29,7 +29,9 @@ void clientAndConsentTest() throws IOException {
         new FixtureServer.FixtureMapping.Builder()
             .validateMethod("GET")
             .validatePath("/oidc/.well-known/oauth-authorization-server")
-            .withResponse("{\"token_endpoint\": \"tokenEndPointFromServer\"}", 200)
+            .withResponse(
+                "{\"token_endpoint\": \"tokenEndPointFromServer\", \"authorization_endpoint\": \"authEndPointFromServer\"}",
+                200)
             .build();
     try (FixtureServer fixtures = new FixtureServer()) {
       fixtures.with(fixture).with(fixture);
@@ -73,7 +75,9 @@ void clientAndConsentTestWithCustomRedirectUrl() throws IOException {
         new FixtureServer.FixtureMapping.Builder()
             .validateMethod("GET")
             .validatePath("/oidc/.well-known/oauth-authorization-server")
-            .withResponse("{\"token_endpoint\": \"tokenEndPointFromServer\"}", 200)
+            .withResponse(
+                "{\"token_endpoint\": \"tokenEndPointFromServer\", \"authorization_endpoint\": \"authEndPointFromServer\"}",
+                200)
             .build();
     try (FixtureServer fixtures = new FixtureServer()) {
       fixtures.with(fixture).with(fixture);