Skip to content

Commit 86c27b5

Browse files
authored
Fix MSSQLSocketConnection compilation issue (#1638)
Follows-up on latest changes in Vert.x Core `io.vertx.core.internal.net.SslChannelProvider.createClientSslHandler` now requires `HostAndPort` instead of `SocketAddress`. Signed-off-by: Thomas Segismont <tsegismont@gmail.com>
1 parent 2d05041 commit 86c27b5

File tree

1 file changed

+3
-1
lines changed

1 file changed

+3
-1
lines changed

vertx-mssql-client/src/main/java/io/vertx/mssqlclient/impl/MSSQLSocketConnection.java

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -24,6 +24,7 @@
2424
import io.vertx.core.internal.tls.SslContextManager;
2525
import io.vertx.core.net.ClientSSLOptions;
2626
import io.vertx.core.net.HostAndPort;
27+
import io.vertx.core.net.SocketAddress;
2728
import io.vertx.core.spi.metrics.ClientMetrics;
2829
import io.vertx.mssqlclient.MSSQLConnectOptions;
2930
import io.vertx.mssqlclient.MSSQLInfo;
@@ -113,7 +114,8 @@ Future<Void> enableSsl(boolean clientConfigSsl, byte encryptionLevel, MSSQLConne
113114
.resolveSslContextProvider(sslOptions, "", null, context)
114115
.map(provider -> new SslChannelProvider(context.owner(), provider, false));
115116
return f.compose(provider -> {
116-
SslHandler sslHandler = provider.createClientSslHandler(socket.remoteAddress(), null, sslOptions.getApplicationLayerProtocols(), sslOptions.getSslHandshakeTimeout(), sslOptions.getSslHandshakeTimeoutUnit());
117+
SocketAddress socketAddress = socket.remoteAddress();
118+
SslHandler sslHandler = provider.createClientSslHandler(HostAndPort.create(socketAddress.host(), socketAddress.port()), null, sslOptions.getApplicationLayerProtocols(), sslOptions.getSslHandshakeTimeout(), sslOptions.getSslHandshakeTimeoutUnit());
117119

118120
// 3. TdsSslHandshakeCodec manages SSL payload encapsulated in TDS packets
119121
TdsSslHandshakeCodec tdsSslHandshakeCodec = new TdsSslHandshakeCodec();

0 commit comments

Comments
 (0)