danyang-match

165 Commits 2 Branches 8 Tags

1 Commits

Include renames

Author	SHA1	Message	Date
WorkBuddy	03a03c7c11	fix(chat): Phase 09 复验 —— 修复 WebSocket 下行死链路 / Unmatch 后可发消息 / Block 未落库 / 聊天 IDOR 对 Phase 09（Text Chat）执行 KEEP-VERIFY/EXTEND 复验：不采信历史 ✅，逐条核对 REST 发送 + WebSocket 接收、幂等、重连、未读计数、非 active Match 禁聊、Unmatch/ Block 即时禁聊、活动双确认会话复用。共发现并修复 4 处真实缺陷 + 1 处规范不符。 D1（Critical）WebSocket 下行整条链路是死代码 - 无 HandshakeInterceptor，无任一环节把 userId 写入 session attributes， extractUserId 恒返回 null；sendToUser/sendToConversation 全仓库零调用点。 - 新增 ChatHandshakeInterceptor：从 ?token= 或 Authorization: Bearer 解析 JWT，校验通过后写 session attr userId；失败 fail-closed 拒绝握手；仅接受 type=access。 - 重写 ChatWebSocketHandler：按规范 {event,data} 信封推送；未认证连接直接 close(NOT_ACCEPTABLE)；同账号重连由新连接接管、旧连接 close(NORMAL)；上行仅支持 ping->pong，禁止客户端自订阅。 - ChatService.sendText 落库+递增未读后推送 message.new；RiskNotice 同推。 - 移除 WebSocketConfig 的 .withSockJS()（SockJS 改写端点致原生 WS 无法直连）。 D2（Critical）Unmatch 后仍可发消息 - unmatch 只改 Match，不联动会话；sendText 仅校验 conv.status==active。 - 新增 ChatService.assertMatchAlive(conv)：按 matchId 查 Match，状态非 {active,getting_to_know} 抛 "匹配已解除，无法发送消息"。sendText 在 conv.status 校验之后调用。 - MatchController.unmatch 落库后调 chatService.syncFromMatch 推 conversation.closed。 - syncFromMatch 重写：alive 恢复 active；非 alive 只推 conversation.closed，不改 status（会话仍以灰显态留在列表，属未授权产品决策，登记待决）。 D3（Critical）RelationService.block 调静态 MatchService.unmatch 只改内存态不落库 - 结果：Block 后 matches.status 仍为 active。 - 补 matchMapper.updateById(m) 持久化 + chatService.closeConversationForMatch( CONVERSATION_STATUS_BLOCKED) 通知对端。RelationshipService.block 同步补会话关闭通知。 D4（Critical 安全）聊天 4 个接口无成员校验 -> IDOR 越权读私聊 - MessageController.load/markRead、ConversationController.detail/unread 补 chatService.isMember(conversationId, userId) 校验；非成员返回 ApiResponse.notFound()（body 级 success=false code=NOT_FOUND，HTTP 200，与 MatchController.authorizedMatch 同约定），不区分"不存在"与"无权"以防探测。 D5（Medium）上行订阅/信封不符规范 - REST API 接口详细设计规范.md 第124-126章要求禁止客户端自订阅，事件名 message.new/message.read/conversation.closed，信封 {event,data}。已对齐。测试保真度 + 测试补齐 - schema-h2.sql 新增 uk_message_client_id(sender_id, client_message_id)，对齐生产 V004:71；H2 不支持过滤索引故省略 WHERE（NULL 语义等价）。否则 service 的 SELECT-then-INSERT 竞态在测试中无 DB 兜底，形成测试盲区。 - 新增 ChatPhase09IntegrationTest（13 用例）：幂等重试不重复投递、推送信封字段、未读增减清零、Match unmatched 拒发、Unmatch/Block 端点即时禁聊+收 closed 事件、活动双确认复用同一消息体系、未形成 Match 无法建会话、非成员读消息/详情/未读被拒且不泄露正文、DB 唯一索引兜底重复 clientMessageId、无幂等键系统消息可共存。 - 新增 ChatWebSocketHandlerTest（7 用例）：userId 路由、信封格式、未认证关连、断开不推、重连替换旧连接、上行仅 ping。验证证据 - 全量回归（离线容器 danyang-match-backend-test，mvn -o test）： 328 tests / 0 failures / 0 errors / 3 skipped / BUILD SUCCESS（基线 308，+20）。 - 负向验证：临时注释 assertMatchAlive + 两处 isMember，恰好 3 个用例变红，证明守卫有效；随后还原并 grep 确认无残留。文档 - 新增 Phase-09-V11-Recovery-Completion-Report-2026-09-16.md（完整报告）。 - Phase-09-V11-Completion-Report.md 顶部加状态注记，推翻其 4 条 WebSocket 结论。 - 计划/STATUS/JIRA_BACKLOG 同步 Phase 09 完成态与测试总数（328）。 Phase 10 未启动，按计划每阶段出报告后停止等待确认。	2026-09-16 16:42:48 +08:00

Author

SHA1

Message

Date

WorkBuddy

03a03c7c11

fix(chat): Phase 09 复验 —— 修复 WebSocket 下行死链路 / Unmatch 后可发消息 / Block 未落库 / 聊天 IDOR

对 Phase 09（Text Chat）执行 KEEP-VERIFY/EXTEND 复验：不采信历史 ✅，逐条核对
REST 发送 + WebSocket 接收、幂等、重连、未读计数、非 active Match 禁聊、Unmatch/
Block 即时禁聊、活动双确认会话复用。共发现并修复 4 处真实缺陷 + 1 处规范不符。

D1（Critical）WebSocket 下行整条链路是死代码
  - 无 HandshakeInterceptor，无任一环节把 userId 写入 session attributes，
    extractUserId 恒返回 null；sendToUser/sendToConversation 全仓库零调用点。
  - 新增 ChatHandshakeInterceptor：从 ?token= 或 Authorization: Bearer 解析 JWT，
    校验通过后写 session attr userId；失败 fail-closed 拒绝握手；仅接受 type=access。
  - 重写 ChatWebSocketHandler：按规范 {event,data} 信封推送；未认证连接直接
    close(NOT_ACCEPTABLE)；同账号重连由新连接接管、旧连接 close(NORMAL)；
    上行仅支持 ping->pong，禁止客户端自订阅。
  - ChatService.sendText 落库+递增未读后推送 message.new；RiskNotice 同推。
  - 移除 WebSocketConfig 的 .withSockJS()（SockJS 改写端点致原生 WS 无法直连）。

D2（Critical）Unmatch 后仍可发消息
  - unmatch 只改 Match，不联动会话；sendText 仅校验 conv.status==active。
  - 新增 ChatService.assertMatchAlive(conv)：按 matchId 查 Match，状态非
    {active,getting_to_know} 抛 "匹配已解除，无法发送消息"。sendText 在 conv.status
    校验之后调用。
  - MatchController.unmatch 落库后调 chatService.syncFromMatch 推 conversation.closed。
  - syncFromMatch 重写：alive 恢复 active；非 alive 只推 conversation.closed，
    不改 status（会话仍以灰显态留在列表，属未授权产品决策，登记待决）。

D3（Critical）RelationService.block 调静态 MatchService.unmatch 只改内存态不落库
  - 结果：Block 后 matches.status 仍为 active。
  - 补 matchMapper.updateById(m) 持久化 + chatService.closeConversationForMatch(
    CONVERSATION_STATUS_BLOCKED) 通知对端。RelationshipService.block 同步补会话关闭通知。

D4（Critical 安全）聊天 4 个接口无成员校验 -> IDOR 越权读私聊
  - MessageController.load/markRead、ConversationController.detail/unread 补
    chatService.isMember(conversationId, userId) 校验；非成员返回
    ApiResponse.notFound()（body 级 success=false code=NOT_FOUND，HTTP 200，
    与 MatchController.authorizedMatch 同约定），不区分"不存在"与"无权"以防探测。

D5（Medium）上行订阅/信封不符规范
  - REST API 接口详细设计规范.md 第124-126章要求禁止客户端自订阅，事件名
    message.new/message.read/conversation.closed，信封 {event,data}。已对齐。

测试保真度 + 测试补齐
  - schema-h2.sql 新增 uk_message_client_id(sender_id, client_message_id)，
    对齐生产 V004:71；H2 不支持过滤索引故省略 WHERE（NULL 语义等价）。
    否则 service 的 SELECT-then-INSERT 竞态在测试中无 DB 兜底，形成测试盲区。
  - 新增 ChatPhase09IntegrationTest（13 用例）：幂等重试不重复投递、推送信封字段、
    未读增减清零、Match unmatched 拒发、Unmatch/Block 端点即时禁聊+收 closed 事件、
    活动双确认复用同一消息体系、未形成 Match 无法建会话、非成员读消息/详情/未读
    被拒且不泄露正文、DB 唯一索引兜底重复 clientMessageId、无幂等键系统消息可共存。
  - 新增 ChatWebSocketHandlerTest（7 用例）：userId 路由、信封格式、未认证关连、
    断开不推、重连替换旧连接、上行仅 ping。

验证证据
  - 全量回归（离线容器 danyang-match-backend-test，mvn -o test）：
    328 tests / 0 failures / 0 errors / 3 skipped / BUILD SUCCESS（基线 308，+20）。
  - 负向验证：临时注释 assertMatchAlive + 两处 isMember，恰好 3 个用例变红，
    证明守卫有效；随后还原并 grep 确认无残留。

文档
  - 新增 Phase-09-V11-Recovery-Completion-Report-2026-09-16.md（完整报告）。
  - Phase-09-V11-Completion-Report.md 顶部加状态注记，推翻其 4 条 WebSocket 结论。
  - 计划/STATUS/JIRA_BACKLOG 同步 Phase 09 完成态与测试总数（328）。

Phase 10 未启动，按计划每阶段出报告后停止等待确认。

2026-09-16 16:42:48 +08:00