[ 
https://issues.apache.org/jira/browse/ARTEMIS-1420?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16182696#comment-16182696
 ] 

ASF GitHub Bot commented on ARTEMIS-1420:
-----------------------------------------

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.


> limit non-ssl connection hangs up exceeding client(s)
> -----------------------------------------------------
>
>                 Key: ARTEMIS-1420
>                 URL: https://issues.apache.org/jira/browse/ARTEMIS-1420
>             Project: ActiveMQ Artemis
>          Issue Type: Improvement
>    Affects Versions: 2.3.0
>            Reporter: Stanislav Knot
>
> this is a max-negotiate-time. Maximum time a connection can take to send the 
> initial protocol negotiation, so time between creating tcp connection and 
> sending the first frame.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

Reply via email to