Github user jbertram commented on a diff in the pull request:
https://github.com/apache/activemq-artemis/pull/1534#discussion_r141371305
--- Diff:
artemis-server/src/main/java/org/apache/activemq/artemis/core/remoting/impl/netty/NettyAcceptor.java
---
@@ -729,6 +731,7 @@ public NettyServerConnection createConnection(final
ChannelHandlerContext ctx,
public void operationComplete(final
io.netty.util.concurrent.Future<Channel> future) throws Exception {
if (future.isSuccess()) {
active = true;
+
ctx.channel().pipeline().remove("readTimeoutHandler");
--- End diff --
This is only valid for SSL connections. The "readTimeoutHandler" should be
removed for non-SSL connections as well.
---
