Format

Author: renaudhartert-db

databricks-sdk-java/src/main/java/com/databricks/sdk/core/DatabricksConfig.java

@@ -352,9 +352,9 @@ public DatabricksConfig setOAuthRedirectUrl(String redirectUrl) {
   }
 
   /**
-   * Returns the scopes to use for the current configuration. If no scopes were provided,
-   * returns the default "all-apis" scope.
-   * 
+   * Returns the scopes to use for the current configuration. If no scopes were provided, returns
+   * the default "all-apis" scope.
+   *
    * @return The scopes to use for the current configuration
    */
   public List<String> getScopes() {

databricks-sdk-java/src/main/java/com/databricks/sdk/core/DefaultCredentialsProvider.java

@@ -158,7 +158,7 @@ private synchronized void addDefaultCredentialsProviders(DatabricksConfig config
     }
 
     providers.add(new PatCredentialsProvider());
-    providers.add(new BasicCredentialsProvider());        
+    providers.add(new BasicCredentialsProvider());
     providers.add(new OAuthM2MServicePrincipalCredentialsProvider());
 
     // Add OIDC-based providers

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

@@ -165,9 +165,6 @@ public Token getToken() {
     params.put(GRANT_TYPE_PARAM, GRANT_TYPE);
     params.put(SUBJECT_TOKEN_PARAM, idToken.getValue());
     params.put(SUBJECT_TOKEN_TYPE_PARAM, SUBJECT_TOKEN_TYPE);
-    if (scopes == null) {
-      throw new IllegalArgumentException("Scopes cannot be null. Use .scopes() method on the builder to set scopes.");
-    }
     params.put(SCOPE_PARAM, String.join(" ", scopes));
     params.put(CLIENT_ID_PARAM, clientId);
 

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

@@ -109,7 +109,7 @@ CachedTokenSource performBrowserAuth(
       DatabricksConfig config, String clientId, String clientSecret, TokenCache tokenCache)
       throws IOException {
     LOGGER.debug("Performing browser authentication");
-    
+
     // Get user-provided scopes and add required default scopes.
     Set<String> scopes = new HashSet<>(config.getScopes());
 
@@ -119,7 +119,7 @@ CachedTokenSource performBrowserAuth(
     if (config.isAzure()) {
       scopes.add(config.getEffectiveAzureLoginAppId() + "/user_impersonation");
     }
-    
+
     OAuthClient client =
         new OAuthClient.Builder()
             .withHttpClient(config.getHttpClient())

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

@@ -12,7 +12,6 @@
 import java.io.IOException;
 import java.net.MalformedURLException;
 import java.net.URL;
-import java.util.Arrays;
 import java.util.HashMap;
 import java.util.Map;
 import java.util.stream.Stream;