Conversation
Owner
|
代码审查(基于 首帧现在调用完整 触发配置:
证据: 此前未知 建议在 以上为静态调用链推导并经独立复核,本次审查未重复运行测试。 |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
问题
合成分组的显式模型别名在 HTTP 请求可用,但 WebSocket 首帧仅按模型名称判断目标,导致合法路由被拒绝。
改动
验证
已验证:后端完整单元与集成测试、静态检查、安全扫描,前端 lint、类型检查、关键测试与依赖安全审计,前后端构建及部署脚本测试全部通过。独立复核通过。
部署脚本的系统专有命令在本地使用兼容适配;未使用真实上游账号进行外部请求验证。
重复检查:未发现等价实现;现有 #5013 明确不处理首帧路由。
Fixes #7368