xp-robotaxi-push-message-boot 代码问题分析

本文档记录了对该模块代码实现中发现的问题。

问题列表

WebSocketUserConnectionManager.java

内存泄漏潜在风险 (lines 24, 30, 36):
java private static ConcurrentHashMap<Long, ConcurrentHashMap<Long, Session>> userConnections = ... private static ConcurrentHashMap<String, Long> sessionIdUserIdMap = ... private static ConcurrentHashMap<Long, Session> sessionIdSessionMap = ...
- 所有连接 map 都是 static
- 即使组件被重新加载（例如开发环境热重载），static map 仍然保持对所有 session 的引用
- 会导致内存泄漏，因为 session 无法被 GC
- 应该使用实例字段而不是 static

PushMessageWebSocketHandler.java

1. 连接建立时缺少认证:
- 连接立即建立，认证在之后通过认证消息进行
- 如果认证永远不会发生或失败，连接仍然保持打开
- 会导致半开放连接累积，占用资源

2. 异常吞掉 (line 74-77):
java } catch (Exception e) { // treat as plain text }
- 捕获解析 JSON 时的任何异常，当作普通文本处理
- 这会吞掉真正的处理异常，应该适当处理