Validate the soft-delete suffix in /api/manage
还没有人认领这个 Issue。
评估
调研方向
从 csharp/src/HgResume.Api/Manage/RepoManageService.cs 中的 SoftDeleteRepo 开始，其中 SoftDeleteRepoRequest 的 Suffix（位于 Manage/ManageDtos.cs）在 Directory.Move 调用中被传递给 DeletedRepoName。请决定是提前以 400 拒绝该后缀，还是如 issue 所建议，检查解析后的路径是否仍位于已删除的文件夹之下。完成的标准是：包含 / 或 .. 的后缀被拒绝，并且 yyyy-MM-ddTHH-mm-ss__reset 格式仍然有效。
由索引模型根据 Issue 内容生成。
描述
POST /api/manage/repos/{code}/soft-delete passes SoftDeleteRepoRequest.Suffix straight into the destination path, with no validation beyond [Required, MinLength(1)]:
// RepoManageService.SoftDeleteRepo
Directory.Move(
PrefixRepoFilePath(code),
Path.Combine(deletedRepoPath, DeletedRepoName(code, deletedRepoSuffix))); // $"{code}__{suffix}"
A suffix that contains / or .. segments (e.g. x/../../../somewhere) moves the repo outside _____deleted_____. A suffix like a/b fails with a 500 because the intermediate directory doesn't exist.
The X-Manage-Secret check means only a trusted caller can do this, so the impact is low. It is still worth hardening, since the suffix is the only part of the path the caller controls freely.
Suggested fix: validate the suffix the same way as project codes, e.g. restrict it to [A-Za-z0-9_-] (which still allows the yyyy-MM-ddTHH-mm-ss__reset timestamps LexBox uses) and return 400 otherwise. Alternatively, check that the resolved full path is still under the deleted folder.
Source: csharp/src/HgResume.Api/Manage/RepoManageService.cs (SoftDeleteRepo), Manage/ManageDtos.cs (SoftDeleteRepoRequest).
- 主要语言
- C#
- 星标
- 0
- 派生
- 1
- PR 合并指标
- 30 天内没有已合并 PR
环境准备
- 提供 Dockerfile 或 Docker Compose 文件
- 没有 Pull Request 模板
- 没有贡献指南
从这里开始
- 先读完整个 Issue，再读项目的贡献指南。
- 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
- Fork 仓库，在一个分支上完成修改。
- 提交 Pull Request，并在描述里引用这个 Issue 编号。
sillsdev/hgresume 的其他 Issue
-
难度 3/5 半天 新手友好度 45/100
-
难度 3/5 半天 新手友好度 72/100
-
难度 1/5 1 小时以内 新手友好度 50/100
-
难度 5/5 一周以上 新手友好度 35/100
-
难度 4/5 3-5 天 新手友好度 35/100
查看 sillsdev/hgresume 的全部 Issue
相似的 Issue
-
难度 2/5 1-3 小时 新手友好度 72/100
Open-Systems-Pharmacology/OSPSuite.Core#3049 ·
维护者通常 1 天内回复
-
needs attention public-client untriaged
难度 2/5 1-3 小时 新手友好度 73/100
AzureAD/microsoft-authentication-library-for-dotnet#6242 ·
维护者通常 2 天内回复
-
难度 1/5 1-3 小时 新手友好度 90/100
criticalmanufacturing/cli#882 ·
维护者通常 1 天内回复
-
难度 2/5 1-3 小时 新手友好度 78/100
Dome-Works/DOME#129 ·
维护者通常 1 天内回复
-
bug sev:L
难度 2/5 1-3 小时 新手友好度 82/100
Systemorph/MeshWeaver#7480 ·
维护者通常 1 天内回复