[PinpiontWeb] fix a ActiveThread concurrency problem pinpoint-apm#1482

If webSocketSession status is not opened, then do not binding to flush aggregator.
koo-taejin · Oct 18, 2016 · 88d5fe5 · 88d5fe5
1 parent 56f53d5
commit 88d5fe5
Showing 1 changed file with 5 additions and 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
@@ -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.");
+                    }
                 }
             }
         }