You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@activemq.apache.org by jbertram <gi...@git.apache.org> on 2017/10/19 14:07:07 UTC

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

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

    https://github.com/apache/activemq-artemis/pull/1534#discussion_r145710225
  
    --- Diff: artemis-server/src/main/java/org/apache/activemq/artemis/core/protocol/ProtocolHandler.java ---
    @@ -38,6 +39,7 @@
     import io.netty.handler.codec.http.HttpResponseEncoder;
     import org.apache.activemq.artemis.api.core.client.ActiveMQClient;
     import org.apache.activemq.artemis.core.buffers.impl.ChannelBufferWrapper;
    +import org.apache.activemq.artemis.core.client.ActiveMQClientLogger;
    --- End diff --
    
    Since the timeout is being identified by the server the ActiveMQServerLogger should be used here, not the client logger.


---