From 88d5fe56617db18f1682f671497a86774740201d Mon Sep 17 00:00:00 2001 From: "koo.taejin" Date: Tue, 18 Oct 2016 17:09:23 +0900 Subject: [PATCH] [PinpiontWeb] fix a ActiveThread concurrency problem #1482 If webSocketSession status is not opened, then do not binding to flush aggregator. --- .../pinpoint/web/websocket/ActiveThreadCountHandler.java | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/web/src/main/java/com/navercorp/pinpoint/web/websocket/ActiveThreadCountHandler.java b/web/src/main/java/com/navercorp/pinpoint/web/websocket/ActiveThreadCountHandler.java index 2b865789c33f..8da38eea614e 100644 --- a/web/src/main/java/com/navercorp/pinpoint/web/websocket/ActiveThreadCountHandler.java +++ b/web/src/main/java/com/navercorp/pinpoint/web/websocket/ActiveThreadCountHandler.java @@ -214,7 +214,11 @@ private void handleRequestMessage0(WebSocketSession webSocketSession, RequestMes } unbindingResponseAggregator(webSocketSession); - bindingResponseAggregator(webSocketSession, applicationName); + if (webSocketSession.isOpen()) { + bindingResponseAggregator(webSocketSession, applicationName); + } else { + logger.warn("WebSocketSession is not opened. skip binding."); + } } } }