activemq-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From stanlyDoge <...@git.apache.org>
Subject [GitHub] activemq-artemis pull request #1534: ARTEMIS-1420 limit non-ssl connection h...
Date Tue, 19 Sep 2017 11:25:10 GMT
Github user stanlyDoge commented on a diff in the pull request:

    https://github.com/apache/activemq-artemis/pull/1534#discussion_r139663458
  
    --- Diff: tests/integration-tests/src/test/java/org/apache/activemq/artemis/tests/integration/amqp/AmqpClientTestSupport.java
---
    @@ -185,6 +187,7 @@ protected TransportConfiguration addAcceptorConfiguration(ActiveMQServer
server,
           HashMap<String, Object> params = new HashMap<>();
           params.put(TransportConstants.PORT_PROP_NAME, String.valueOf(port));
           params.put(TransportConstants.PROTOCOLS_PROP_NAME, getConfiguredProtocols());
    +      params.put(TransportConstants.NETTY_READ_TIMEOUT, readTimeout);
    --- End diff --
    
    Actually it is 3 seconds. It is just typo in property string. I've tried to make test
as you say. Check it in new commit. And thanks for reviewing! :)


---

Mime
View raw message