[
https://issues.apache.org/jira/browse/ARTEMIS-1420?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16211087#comment-16211087
]
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_r145710804
--- Diff:
artemis-server/src/main/java/org/apache/activemq/artemis/core/protocol/ProtocolHandler.java
---
@@ -99,9 +101,28 @@ public ProtocolManager getProtocol(String name) {
private final boolean httpEnabled;
+ private ScheduledFuture timeoutFuture;
+
+ private int handshakeTimeout;
+
+
ProtocolDecoder(boolean http, boolean httpEnabled) {
this.http = http;
this.httpEnabled = httpEnabled;
+ this.handshakeTimeout =
ConfigurationHelper.getIntProperty(TransportConstants.NETTY_HANDSHAKE_TIMEOUT,
TransportConstants.DEFAULT_NETTY_HANDSHAKE_TIMEOUT,
nettyAcceptor.getConfiguration());
+ }
+
+ @Override
+ public void channelActive(ChannelHandlerContext ctx) throws
Exception {
+ if (handshakeTimeout > 0) {
+ timeoutFuture = scheduledThreadPool.schedule(new Runnable() {
+ @Override
+ public void run() {
+ ActiveMQClientLogger.LOGGER.readTimeout();
--- End diff --
The method should probably be renamed something like handshakeTimeout() in
addition to being moved to the ActiveMQServerLogger.
> 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)
