minor fix

apple · Sep 29, 2023 · 70be0c1 · 70be0c1
1 parent c746fb4
commit 70be0c1
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 7 deletions.
diff --git a/servicetalk-http-netty/src/main/java/io/servicetalk/http/netty/AlpnChannelSingle.java b/servicetalk-http-netty/src/main/java/io/servicetalk/http/netty/AlpnChannelSingle.java
@@ -45,11 +45,6 @@
 final class AlpnChannelSingle extends ChannelInitSingle<String> {
     private final Consumer<ChannelHandlerContext> onHandlerAdded;
 
-    AlpnChannelSingle(final Channel channel,
-                      final ChannelInitializer channelInitializer) {
-        this(channel, channelInitializer, __ -> { });
-    }
-
     AlpnChannelSingle(final Channel channel,
                       final ChannelInitializer channelInitializer,
                       final Consumer<ChannelHandlerContext> onHandlerAdded) {

diff --git a/...etalk-http-netty/src/main/java/io/servicetalk/http/netty/AlpnLBHttpConnectionFactory.java b/...etalk-http-netty/src/main/java/io/servicetalk/http/netty/AlpnLBHttpConnectionFactory.java
@@ -69,8 +69,8 @@ Single<FilterableStreamingHttpConnection> newFilterableConnection(
     private Single<FilterableStreamingHttpConnection> createConnection(
             final Channel channel, final ConnectionObserver connectionObserver,
             final ReadOnlyTcpClientConfig tcpConfig) {
-        return new AlpnChannelSingle(channel,
-                new TcpClientChannelInitializer(tcpConfig, connectionObserver)).flatMap(protocol -> {
+        return new AlpnChannelSingle(channel, new TcpClientChannelInitializer(tcpConfig, connectionObserver),
+                ctx -> { /* SslHandler will automatically start handshake on channelActive */ }).flatMap(protocol -> {
             switch (protocol) {
                 case HTTP_1_1:
                     final H1ProtocolConfig h1Config = config.h1Config();