feat(gateway,runtime): 会话附件删除接口与图片输入模型兼容投影#713
Closed
wynxing wants to merge 1 commit into
Closed
Conversation
- 新增 DeleteSessionAsset 网关接口与运行时实现 - 新增 SessionAssetPort 接口抽象，解耦具体类型嵌入 - 图片输入模型兼容投影（projection） - ACL 测试键归一化与 nil source 保护 - Hook Matcher DSL 与命令处理器 - microcompact 删除与 input_preparer 迁移 - Web 端多模态会话资产支持 - TUI 会话资产删除事件处理
|
Codex usage limits have been reached for code reviews. Please check with the admins of this repo to increase the limits by adding credits. |
![fennoai[bot]](https://avatars.githubusercontent.com/in/1924301?s=60&v=4){kind=small}
| @@ -406,7 +696,7 @@ func (s *NetworkServer) withCORS(next http.Handler) http.Handler { | |||
| } | |||
|
|
|||
| writer.Header().Set("Access-Control-Allow-Methods", "GET, POST, OPTIONS") | |||
Contributor
There was a problem hiding this comment.
这里新增了浏览器端 deleteSessionAsset()，会发起 DELETE /api/session-assets/...，且通常携带 Authorization 和 X-NeoCode-Workspace-Hash，因此跨源浏览器连接会先发 CORS preflight。当前 Access-Control-Allow-Methods 仍只有 GET, POST, OPTIONS，preflight 会因为缺少 DELETE 被浏览器拦截，导致上传后清理/删除附件失败。建议把 DELETE 加入允许方法，并补一个 OPTIONS 覆盖测试。
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
1 participant
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.
概述
本 PR 是 #712 的干净替代版本，排除了纯空格/格式变更的脏提交。
变更内容
与 #712 的区别
unner.go,
unner_tool.go, \ ypes.go\）
关联
Closes #712