[GitHub] activemq-artemis pull request #1534: ARTEMIS-1420 limit non-ssl connection h...

[michaelandrepearce]

Github user michaelandrepearce commented on a diff in the pull request:

    https://github.com/apache/activemq-artemis/pull/1534#discussion_r138713895
  
    --- Diff: 
artemis-server/src/main/java/org/apache/activemq/artemis/core/remoting/impl/netty/NettyAcceptor.java
 ---
    @@ -206,6 +207,8 @@
     
        final AtomicBoolean warningPrinted = new AtomicBoolean(false);
     
    +   private static int communicationTimeout = 30; //seconds
    --- End diff --
    
    This should be named read time out similar to netty naming


---