Enhance WebSocket session management by tracking principal names and improving…

Enhance WebSocket session management by tracking principal names and improving closed session handling

sample/src/main/java/com/dji/sample/component/websocket/config/MyWebSocketHandler.java

@@ -32,6 +32,7 @@ public class MyWebSocketHandler extends WebSocketDefaultHandler {
     private ObjectMapper objectMapper = new ObjectMapper();
     private Map<String, Boolean> authenticatedSessions = new ConcurrentHashMap<>();
     private Map<String, CustomClaim> sessionClaims = new ConcurrentHashMap<>();
+    private Map<String, String> sessionPrincipalNames = new ConcurrentHashMap<>();
 
     MyWebSocketHandler(WebSocketHandler delegate, IWebSocketManageService webSocketManageService) {
         super(delegate);
@@ -46,6 +47,8 @@ public class MyWebSocketHandler extends WebSocketDefaultHandler {
         if (StringUtils.hasText(principalName) && !principalName.startsWith("temp-")) {
             webSocketManageService.put(principalName, new MyConcurrentWebSocketSession(session));
             authenticatedSessions.put(session.getId(), true);
+            // 记录 principalName 用于 afterConnectionClosed 时清理
+            sessionPrincipalNames.put(session.getId(), principalName);
             log.debug("{} is connected (pre-authenticated). ID: {}. WebSocketSession[current count: {}]",
                     principalName, session.getId(), webSocketManageService.getConnectedCount());
         } else {
@@ -60,18 +63,26 @@ public class MyWebSocketHandler extends WebSocketDefaultHandler {
         Boolean isAuthenticated = authenticatedSessions.get(sessionId);
 
         if (Boolean.TRUE.equals(isAuthenticated)) {
+            // 优先用 sessionClaims（消息认证方式），其次用 sessionPrincipalNames（URL token 认证方式）
             CustomClaim claim = sessionClaims.get(sessionId);
             if (claim != null) {
                 String key = claim.getWorkspaceId() + "/" + claim.getUserType() + "/" + claim.getId();
                 webSocketManageService.remove(key, sessionId);
                 log.debug("{} is disconnected. ID: {}. WebSocketSession[current count: {}]",
                         key, sessionId, webSocketManageService.getConnectedCount());
+            } else {
+                String principalName = sessionPrincipalNames.get(sessionId);
+                if (principalName != null) {
+                    webSocketManageService.remove(principalName, sessionId);
+                    log.debug("{} is disconnected (pre-auth). ID: {}. WebSocketSession[current count: {}]",
+                            principalName, sessionId, webSocketManageService.getConnectedCount());
+                }
             }
         }
 
         authenticatedSessions.remove(sessionId);
         sessionClaims.remove(sessionId);
-
+        sessionPrincipalNames.remove(sessionId);
     }
 
     @Override

sample/src/main/java/com/dji/sample/component/websocket/service/impl/WebSocketMessageServiceImpl.java

@@ -63,9 +63,12 @@ public class WebSocketMessageServiceImpl implements IWebSocketMessageService {
 
             for (MyConcurrentWebSocketSession session : sessions) {
                 if (!session.isOpen()) {
-                    session.close();
-                    log.debug("This session is closed.");
-                    return;
+                    try {
+                        session.close();
+                    } catch (Exception ignored) {
+                    }
+                    log.debug("Skipping closed session: {}", session.getId());
+                    continue;
                 }
                 session.sendMessage(data);
             }