Remove dependency on com.microsoft.azure:adal4j

Follow-up to #22584
quarkusio · Mar 20, 2022 · ef6b48f · ef6b48f
1 parent 8af7cb4
commit ef6b48f
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 27 deletions.
diff --git a/extensions/jdbc/jdbc-mssql/runtime/pom.xml b/extensions/jdbc/jdbc-mssql/runtime/pom.xml
@@ -27,13 +27,6 @@
             <groupId>com.microsoft.sqlserver</groupId>
             <artifactId>mssql-jdbc</artifactId>
         </dependency>
-        <!--
-            adal4j isn't used directly but you need to have it around for compilation to succeed.
-        -->
-        <dependency>
-            <groupId>com.microsoft.azure</groupId>
-            <artifactId>adal4j</artifactId>
-        </dependency>
         <dependency>
             <groupId>org.graalvm.nativeimage</groupId>
             <artifactId>svm</artifactId>

diff --git a/...kus/jdbc/mssql/runtime/graal/com/microsoft/sqlserver/jdbc/SQLServerJDBCSubstitutions.java b/...kus/jdbc/mssql/runtime/graal/com/microsoft/sqlserver/jdbc/SQLServerJDBCSubstitutions.java
@@ -1,29 +1,12 @@
 package io.quarkus.jdbc.mssql.runtime.graal.com.microsoft.sqlserver.jdbc;
 
-import com.oracle.svm.core.annotate.Alias;
-import com.oracle.svm.core.annotate.RecomputeFieldValue;
+import com.oracle.svm.core.annotate.Delete;
 import com.oracle.svm.core.annotate.Substitute;
 import com.oracle.svm.core.annotate.TargetClass;
 
 @TargetClass(className = "com.microsoft.sqlserver.jdbc.SQLServerADAL4JUtils")
-@Substitute
+@Delete
 final class SQLServerADAL4JUtils {
-
-    @Alias
-    @RecomputeFieldValue(kind = RecomputeFieldValue.Kind.Reset)
-    static final private java.util.logging.Logger adal4jLogger = null;
-
-    @Substitute
-    static QuarkusSqlFedAuthToken getSqlFedAuthToken(QuarkusSqlFedAuthInfo fedAuthInfo, String user, String password,
-            String authenticationString) {
-        throw new IllegalStateException("Quarkus does not support Active Directory based authentication");
-    }
-
-    @Substitute
-    static QuarkusSqlFedAuthToken getSqlFedAuthTokenIntegrated(QuarkusSqlFedAuthInfo fedAuthInfo, String authenticationString) {
-        throw new IllegalStateException("Quarkus does not support Active Directory based authentication");
-    }
-
 }
 
 @TargetClass(className = "com.microsoft.sqlserver.jdbc.SqlFedAuthToken")
@@ -39,6 +22,16 @@ final class QuarkusSqlFedAuthInfo {
 @TargetClass(className = "com.microsoft.sqlserver.jdbc.SQLServerConnection")
 final class QuarkusSQLServerConnection {
 
+    @Substitute
+    private void validateAdalLibrary(String errorMessage) {
+        throw new IllegalStateException("Quarkus does not support Active Directory based authentication");
+    }
+
+    @Substitute
+    private QuarkusSqlFedAuthToken getFedAuthToken(QuarkusSqlFedAuthInfo fedAuthInfo) {
+        throw new IllegalStateException("Quarkus does not support Active Directory based authentication");
+    }
+
     @Substitute
     private QuarkusSqlFedAuthToken getMSIAuthToken(String resource, String msiClientId) {
         throw new IllegalStateException("Quarkus does not support MSI based authentication");
@@ -48,4 +41,4 @@ private QuarkusSqlFedAuthToken getMSIAuthToken(String resource, String msiClient
 
 class SQLServerJDBCSubstitutions {
 
-}
+}