zjncs opened a new pull request, #5593: URL: https://github.com/apache/rocketmq-dashboard/pull/5593
## 问题 `web/src/services/messageService.ts` 的 `listDLQGroups` 与 `listDLQMessages` mock **不校验分页参数**： - **page 0/负值** 变成负 slice 索引（`from = Math.min((page-1)*pageSize, len)`）——`page=-1, pageSize=1` 从列表**末尾**取数=错误数据 - **pageSize 101+** 静默接受；`listDLQMessages` 对任意非法参数返回空页 后端对以上全部 400：`DLQService.java:35,41-48`（`MAX_PAGE_SIZE=100`、`"Invalid page or pageSize"`）与 `DLQController.java:90-94`（`@Min(1)` page、`@Min(1) @Max(100)` pageSize）。mock 测试因此在真实后端必拒的输入上假绿，mock 消费者看到负页的垃圾行。 同文件 `queryMessagePage`（57-77 行）的 mock 已精确镜像其后端校验——这两个分支被遗漏。与已合并 #5498（aclService 分页契约）同族。 ## 修复 两处 mock 均按后端消息拒绝越界值（`Invalid page or pageSize`）；合法路径行为不变。 ## 验证（三段） 1. **先红**：新增 5 用例（groups：page 0/负值/pageSize 101/0；messages：page 0/pageSize 101/0）在未修复代码上失败 2. **后绿 + 回归**：修复后 12/12（含既有 7 用例） 3. **突变**：mock 还原 → 5 用例红；恢复 → 绿 ## 碰撞核查 `web/src/services/messageService.ts` 无任何开放 PR 触碰（已认领的服务：aclService/clusterService/instanceService/opsService——188 open PR 文件清单核对）。 Closes #5592 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
