Lower "no session" message log level in SubProtocolWSH

rstoyanchev · rstoyanchev · commit b0e6091cadea · 2014-09-24T15:04:57.000-04:00
Issue: SPR-12247
diff --git a/spring-websocket/src/main/java/org/springframework/web/socket/messaging/SubProtocolWebSocketHandler.java b/spring-websocket/src/main/java/org/springframework/web/socket/messaging/SubProtocolWebSocketHandler.java
@@ -324,7 +324,10 @@ public void handleMessage(Message<?> message) throws MessagingException {
 		}
 		WebSocketSessionHolder holder = this.sessions.get(sessionId);
 		if (holder == null) {
-			logger.error("No session for " + message);
+			if (logger.isDebugEnabled()) {
+				// The broker may not have removed the session yet
+				logger.debug("No session for " + message);
+			}
 			return;
 		}
 		WebSocketSession session = holder.getSession();

Original file line number	Diff line number	Diff line change
`@@ -324,7 +324,10 @@ public void handleMessage(Message<?> message) throws MessagingException {`
`324`	`324`	`}`
`325`	`325`	`WebSocketSessionHolder holder = this.sessions.get(sessionId);`
`326`	`326`	`if (holder == null) {`
`327`		`- logger.error("No session for " + message);`
	`327`	`+ if (logger.isDebugEnabled()) {`
	`328`	`+ // The broker may not have removed the session yet`
	`329`	`+ logger.debug("No session for " + message);`
	`330`	`+ }`
`328`	`331`	`return;`
`329`	`332`	`}`
`330`	`333`	`WebSocketSession session = holder.getSession();`