Actually we don't need to use the TrustAllOptions since it's handled by the SSLHelper

vietj · vietj · commit 12b48d4ee9fa · 2024-03-12T20:18:54.000+01:00
diff --git a/vertx-mssql-client/src/main/java/io/vertx/mssqlclient/impl/MSSQLSocketConnection.java b/vertx-mssql-client/src/main/java/io/vertx/mssqlclient/impl/MSSQLSocketConnection.java
@@ -105,7 +105,7 @@ Future<Void> enableSsl(boolean clientConfigSsl, byte encryptionLevel, MSSQLConne
 
     // Do not perform hostname validation if the client did not require encryption
     if (!clientConfigSsl) {
-      sslOptions.setTrustOptions(TrustAllOptions.INSTANCE);
+      sslOptions.setTrustAll(true);
     }
     sslOptions.setHostnameVerificationAlgorithm("");
 

Original file line number	Diff line number	Diff line change
`@@ -105,7 +105,7 @@ Future<Void> enableSsl(boolean clientConfigSsl, byte encryptionLevel, MSSQLConne`
`105`	`105`
`106`	`106`	`// Do not perform hostname validation if the client did not require encryption`
`107`	`107`	`if (!clientConfigSsl) {`
`108`		`- sslOptions.setTrustOptions(TrustAllOptions.INSTANCE);`
	`108`	`+ sslOptions.setTrustAll(true);`
`109`	`109`	`}`
`110`	`110`	`sslOptions.setHostnameVerificationAlgorithm("");`
`111`	`111`