docs: 文档整理,CLAUDE.md 瘦身一半,删掉重写期已完成的 22 份阶段产物

CLAUDE.md 从 490 行降到 228 行:只留日常要当场记住的约束,展开拆成五份专题
文档 —— docs/deploy.md(部署与备份恢复)、database.md(迁移执行器、基线、
drizzle-kit 的坑)、timezone.md(时区口径与那次成就订正)、contract.md
(出参不 parse 的四次故障)、ast-rules.md(AST 规则与 C++ 的调用形态)。

删掉的是阶段 0–5 那批一次性产物:4 份实施计划、10 份评审/核验/修复报告、
endpoint-inventory.md(110 端点是 2026-08 的快照,现在 363 条路由)、
docs/spikes/ 的 spike 与提取脚本(结论早已落进代码)。phase5 切换手册删之前
先把仍然有效的部分提炼进 docs/deploy.md:拓扑、deploy.sh、部署后验证清单、
NPM 那两个不能关的开关、pg_dumpall 恢复的两个坑、镜像体积;演练报告与回滚
两节随旧栈下线一并作废。

两份设计文档保留,补上状态行说明它们是「当初为什么这么定」而不是现状。

apps/web/CLAUDE.md 顺手订正过期内容:PUBLIC_OJ_URL / PUBLIC_WS_URL 两个变量
早已不存在(baseURL 写死 /api,dev 走 vite proxy、线上由 Caddy 同源伺服),
store 与 composable 清单补齐到与目录一致。

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
This commit is contained in:
2026-09-16 08:03:26 -06:00
parent 3559ae4d6f
commit a8408c0bb5
34 changed files with 593 additions and 7570 deletions

View File

@@ -1,7 +1,11 @@
# OJ2 设计文档:后端重写为 Bun + TypeScript
日期2026-08-06
状态:已确认,待实施
状态:**已实施、已上线**2026-08 切换完成,旧栈已不可逆下线)。本文保留为「当初为什么这么定」
的记录,**不是现状的说明** —— 现状看 `CLAUDE.md``docs/` 下的专题文档。
文中引用的 `docs/spikes/`(三个 spike 脚本)与 `docs/specs/schema.sql`(生产库结构 dump
已在 2026-09-16 删除结论都已落进代码脚本跑完就没用了schema dump 的内容进了迁移
`0000_crazy_gateway.sql`。要看它们去 git 历史。
## 1. 背景与动机
@@ -65,7 +69,7 @@
| 全站不用的分支 | OI 赛制(所有比赛均为 ACM |
| Python 生态锁定 | 仅 2 处:`jieba``flowchart/views/admin.py` 单文件)、`tree-sitter``ast_checker/`177 行) |
> 更正2026-08-06 阶段 0 重跑后):本表原写「端点合计 122oj 74 / admin 48、DEPRECATED 16、前端调用 78、疑似无人调用约 35%」,四项全错。前三项来自一版漏抓了 `tutorial/urls/tutorial.py` 与 `utils/urls.py` 的提取脚本(共漏 5 个端点,其中 4 个前端在用)与一版只数字面量、不含模板串的前端统计;「约 35%」是从 `(12278)/122` 推出来的,两个输入都错。现表为 `docs/spikes/` 三个脚本重跑的实测值,独立核验:`cd OnlineJudge && cat */urls/*.py utils/urls.py | grep -c "path("` → 127。
> 更正2026-08-06 阶段 0 重跑后):本表原写「端点合计 122oj 74 / admin 48、DEPRECATED 16、前端调用 78、疑似无人调用约 35%」,四项全错。前三项来自一版漏抓了 `tutorial/urls/tutorial.py` 与 `utils/urls.py` 的提取脚本(共漏 5 个端点,其中 4 个前端在用)与一版只数字面量、不含模板串的前端统计;「约 35%」是从 `(12278)/122` 推出来的,两个输入都错。现表为当时三个提取脚本重跑的实测值,独立核验:`cd OnlineJudge && cat */urls/*.py utils/urls.py | grep -c "path("` → 127。
> **减法空间只有 18%,不是三分之一。** 后续阶段按 18% 排期。
前端网络层集中度高,改动面小:
@@ -141,9 +145,9 @@ Projects/OJ/
## 7. 已验证的技术假设
三处高风险假设已在 Bun 1.3.11 上实测通过spike 代码 `docs/spikes/`。依赖清单与 lockfile 已随 spike 源码入库(`docs/spikes/package.json``bun.lock``cd docs/spikes && bun install` 后三个脚本均可直接重跑
三处高风险假设已在 Bun 1.3.11 上实测通过。当时的 spike 代码 `docs/spikes/`,已删除(见顶部说明),三条结论都已落进 `apps/api` 的实现
### 7.1 Django 密码哈希兼容`docs/spikes/pbkdf2-spike.ts`
### 7.1 Django 密码哈希兼容
用 Django 生成 `pbkdf2_sha256$1200000$...` 格式哈希Bun 侧用 `node:crypto``pbkdf2` 验证:
@@ -163,7 +167,7 @@ argon2id : true 耗时 88 ms
#### 7.1.1 `raw_password` 明文列保留(已决策)
生产库 `user` 表有一列 `raw_password character varying(20)``docs/specs/schema.sql:939`),存学生明文密码。
生产库 `user` 表有一列 `raw_password character varying(20)`),存学生明文密码。
**决策:保留。** 这是有意的运维需求——学生忘记密码是高频事件,教师需要能直接查到并告知,走"重置密码"流程在机房环境里成本过高。新后端照样维护这一列。
@@ -176,7 +180,7 @@ argon2id : true 耗时 88 ms
**若日后想在不改变教师查密码这一工作流的前提下收紧**(本次未采纳,仅备查):把 `raw_password` 改为用一把存在环境变量/密钥文件里、**不在数据库内**的密钥做可逆加密。教师查询走应用层解密,体验不变;而一份裸的数据库备份泄露时不再直接暴露明文。改动量约为一个加解密工具函数 + 一次存量数据迁移。
### 7.2 tree-sitter 迁移`docs/spikes/ast-spike.ts`
### 7.2 tree-sitter 迁移
复刻 `ast_checker/mappings/c.py` 的映射表,在 Bun 中用 `web-tree-sitter` 解析 C 代码:
@@ -204,7 +208,7 @@ C 与 Python 两套 grammar 均正常。`.wasm` 文件随 npm 包分发(`tree-
未选原生 NAPI 绑定:性能更高但需 node-gyp 现场编译Bun 支持稳定性较差。1.1ms 解析耗时在判题流程中可忽略(沙箱启动本身即数十毫秒),选 WASM 图部署简单。
### 7.3 `@node-rs/jieba` 替代 Python jieba`docs/spikes/jieba-spike.ts`
### 7.3 `@node-rs/jieba` 替代 Python jieba
对照 `flowchart/views/admin.py:65,191` 的两处用法——`jieba.add_word(w, freq=9999)` 加自定义词、`jieba.cut(text)` 切词——在 Bun 1.3.11 下验证 `@node-rs/jieba@2.0.1`NAPI 绑定):
@@ -249,9 +253,9 @@ jieba.loadDict(Buffer.from("两个整数 9999\n"))
盘点中发现的、明确不应带进新后端的实现:
- **`SessionRecordMiddleware``account/middleware.py:22-33`**:每个已登录请求都写一遍 sessionuser_agent / ip / last_activity遇到新 session key 还额外触发一次 `request.user.save()` —— 即每请求一次数据库写。这是"Django 太慢"的实际来源之一。新后端的会话信息留在 Redis不落库。
- **`User.session_keys`**:只写不读的死字段,随 `/api/sessions` 端点一并砍掉。(已由 `schema.sql:938` 确认该列存在。)
- **`User.session_keys`**:只写不读的死字段,随 `/api/sessions` 端点一并砍掉。
反过来,**必须复刻**的一项:`user.raw_password``schema.sql:939`保留,见 7.1.1。
反过来,**必须复刻**的一项:`user.raw_password` 保留,见 7.1.1。
## 9. 判题链路

View File

@@ -1,7 +1,8 @@
# OJ2 设计文档:课堂求助与协作编辑
日期2026-08-28
状态:已实施
状态:已实施。**第 1 节写的是重做之前的实现**y-webrtc 那套,`sync.ts` / `syncStatus.ts`
早已删除),留着是为了说明为什么要改;现状是第 3 节往后。
## 1. 背景

View File

@@ -1,190 +0,0 @@
# 端点清单(已裁决)
生成时间2026-08-06  裁决时间2026-08-06
合计 127 个端点 —— KEEP 110、CUT 17、REVIEW 0
**裁决结果:新后端需实现 110 个端点,砍掉 17 个(占 13%)。**
裁决说明:机器初判的 6 条 REVIEW 全部判为 KEEP。其中 5 条是下述提取盲点造成的假阴性
(前端确实在调用,只是提取脚本抓不到);`/api/judge_server_heartbeat/` 不经前端,是判题机
向后端注册心跳的接口,新架构判题沙箱镜像原样复用,必须保留。
> ⚠️ 本文件已完成人工裁决,**不要再运行 `docs/spikes/reconcile.ts`** —— 它会重新生成本文件,
> 把上面的裁决结果和末列理由全部冲掉。若确需重跑(例如后端 urls 有变动),先备份本文件。
> 已知盲点 1`ojnext/src/oj/api.ts` 第 45、73 行用变量动态传路径(形如 `http.get(endpoint)`),提取脚本的正则匹配不到这类调用。因此对应的后端端点会被本表判成“前端无调用”,但实际可能仍在使用 —— 例如 `/api/contest_submissions``getSubmissions` 里 `endpoint` 变量的另一分支)。
> 已知盲点 2`ojnext` 里有 4 处用原生 `fetch("/api/...")` 而非 `http.get/post(...)` 发起请求AI 流式响应场景:`src/oj/store/ai.ts`、`src/oj/problem/components/SubmissionResult.vue`、`src/oj/rank/list.vue`、`src/oj/class/pk.vue`),提取脚本只认 `get/post/put/delete(...)` 调用形式,完全抓不到 `fetch(...)`。本轮 REVIEW 里的 `/api/ai/analysis`、`/api/ai/hint`、`/api/ai/class_pk`、`/api/ai/class_single` 经人工核实均属此类,实际都在用。
> 已知盲点 3`ojnext/src/utils/download.ts` 是一个独立的 axios 实例(`baseURL: "/api/admin"`,与 `src/utils/http.ts` 那个共用实例无关),对外只暴露 `download(url)` 一个函数,内部走 `http.get(url)`。提取脚本既不认 `download(...)` 这种调用名,也抓不到内部那个变量 `url`,所以这条通道上的调用一律是假阴性。当前两个调用点(`src/admin/problem/components/Actions.vue:46`、`src/admin/problem/detail.vue:316`)都指向 `admin/test_case`,而该端点已因别处的字面量调用被判成 KEEP**本轮结论不受影响**。但日后新增的 `download(...)` 调用会静默变成假阴性 CUT裁决时留意。
> 盲点 1、2、3 都是“前端有调用=否”但实际有调用,人工裁决时不要仅凭本表这一列就判 CUTREVIEW 里唯一不属于此类的是 `/api/judge_server_heartbeat/`——它是判题机而非前端调用的接口,不受提取脚本盲点影响,是否保留需按后端间调用来判断。
> 反向对账(前端调用了、后端却查无此端点):**0 条**,前端全部调用路径都能在后端端点全集里找到对应。
| 裁决 | app | 侧 | 路径 | 视图 | 前端有调用 | 已标 DEPRECATED | 理由 |
|---|---|---|---|---|---|---|---|
| CUT | account | oj | `/api/change_password` | UserChangePasswordAPI.as_view | 否 | 是 | |
| CUT | account | oj | `/api/change_email` | UserChangeEmailAPI.as_view | 否 | 是 | |
| CUT | account | oj | `/api/check_username_or_email` | UsernameOrEmailCheck.as_view | 否 | 是 | |
| CUT | account | oj | `/api/sessions` | SessionManagementAPI.as_view | 否 | 是 | |
| CUT | account | oj | `/api/open_api_appkey` | OpenAPIAppkeyAPI.as_view | 否 | 是 | |
| CUT | account | oj | `/api/sso` | SSOAPI.as_view | 否 | 是 | |
| CUT | account | admin | `/api/admin/generate_user` | GenerateUserAPI.as_view | 否 | 是 | |
| CUT | conf | oj | `/api/languages` | LanguagesAPI.as_view | 否 | 是 | |
| CUT | contest | oj | `/api/contest/announcement` | ContestAnnouncementListAPI.as_view | 否 | 是 | |
| CUT | contest | admin | `/api/admin/contest/announcement` | ContestAnnouncementAPI.as_view | 否 | 是 | |
| CUT | contest | admin | `/api/admin/download_submissions` | DownloadContestSubmissions.as_view | 否 | 是 | |
| CUT | problemset | oj | `/api/problemset/<int:problem_set_id>/problems/<int:problem_id>` | ProblemSetProblemAPI.as_view | 否 | 是 | |
| CUT | problemset | oj | `/api/problemset/<int:problem_set_id>/progress` | ProblemSetProgressAPI.as_view | 否 | 是 | |
| CUT | problemset | oj | `/api/user/progress` | UserProgressAPI.as_view | 否 | 是 | |
| CUT | problemset | admin | `/api/admin/problemset/<int:problem_set_id>/sync` | ProblemSetSyncAPI.as_view | 否 | 是 | |
| CUT | submission | oj | `/api/submission_exists` | SubmissionExistsAPI.as_view | 否 | 是 | |
| CUT | utils | admin | `/api/admin/upload_file` | SimditorFileUploadAPIView.as_view | 否 | 是 | |
| KEEP | account | oj | `/api/login` | UserLoginAPI.as_view | 是 | 否 | |
| KEEP | account | oj | `/api/logout` | UserLogoutAPI.as_view | 是 | 否 | |
| KEEP | account | oj | `/api/register` | UserRegisterAPI.as_view | 是 | 否 | |
| KEEP | account | oj | `/api/profile` | UserProfileAPI.as_view | 是 | 否 | |
| KEEP | account | oj | `/api/profile/fresh_display_id` | ProfileProblemDisplayIDRefreshAPI.as_view | 是 | 否 | |
| KEEP | account | oj | `/api/metrics` | Metrics.as_view | 是 | 否 | |
| KEEP | account | oj | `/api/upload_avatar` | AvatarUploadAPI.as_view | 是 | 否 | |
| KEEP | account | oj | `/api/user_rank` | UserRankAPI.as_view | 是 | 否 | |
| KEEP | account | oj | `/api/user_activity_rank` | UserActivityRankAPI.as_view | 是 | 否 | |
| KEEP | account | oj | `/api/user_problem_rank` | UserProblemRankAPI.as_view | 是 | 否 | |
| KEEP | account | admin | `/api/admin/user` | UserAdminAPI.as_view | 是 | 否 | |
| KEEP | account | admin | `/api/admin/reset_password` | ResetUserPasswordAPI.as_view | 是 | 否 | |
| KEEP | achievement | oj | `/api/achievements` | AchievementListAPI.as_view | 是 | 否 | |
| KEEP | achievement | oj | `/api/achievements/summary` | AchievementSummaryAPI.as_view | 是 | 否 | |
| KEEP | achievement | oj | `/api/achievements/pending` | AchievementPendingAPI.as_view | 是 | 否 | |
| KEEP | achievement | admin | `/api/admin/achievement` | AchievementAdminAPI.as_view | 是 | 否 | |
| KEEP | achievement | admin | `/api/admin/achievement/metrics` | AchievementMetricAdminAPI.as_view | 是 | 否 | |
| KEEP | ai | oj | `/api/ai/detail` | AIDetailDataAPI.as_view | 是 | 否 | |
| KEEP | ai | oj | `/api/ai/duration` | AIDurationDataAPI.as_view | 是 | 否 | |
| KEEP | ai | oj | `/api/ai/heatmap` | AIHeatmapDataAPI.as_view | 是 | 否 | |
| KEEP | ai | oj | `/api/ai/login_summary` | AILoginSummaryAPI.as_view | 是 | 否 | |
| KEEP | ai | oj | `/api/ai/pinned` | AIPinnedReportAPI.as_view | 是 | 否 | |
| KEEP | ai | admin | `/api/admin/ai/reports` | AIAnalysisAdminAPI.as_view | 是 | 否 | |
| KEEP | announcement | oj | `/api/announcement` | AnnouncementAPI.as_view | 是 | 否 | |
| KEEP | announcement | admin | `/api/admin/announcement` | AnnouncementAdminAPI.as_view | 是 | 否 | |
| KEEP | class_pk | oj | `/api/class_rank` | ClassRankAPI.as_view | 是 | 否 | |
| KEEP | class_pk | oj | `/api/user_class_rank` | UserClassRankAPI.as_view | 是 | 否 | |
| KEEP | class_pk | oj | `/api/class_pk` | ClassPKAPI.as_view | 是 | 否 | |
| KEEP | conf | oj | `/api/website` | WebsiteConfigAPI.as_view | 是 | 否 | |
| KEEP | conf | oj | `/api/hitokoto` | HitokotoAPI.as_view | 是 | 否 | |
| KEEP | conf | oj | `/api/class_usernames` | ClassUsernamesAPI.as_view | 是 | 否 | |
| KEEP | conf | admin | `/api/admin/website` | WebsiteConfigAPI.as_view | 是 | 否 | |
| KEEP | conf | admin | `/api/admin/random_user` | RandomUsernameAPI.as_view | 是 | 否 | |
| KEEP | conf | admin | `/api/admin/judge_server` | JudgeServerAPI.as_view | 是 | 否 | |
| KEEP | conf | admin | `/api/admin/prune_test_case` | TestCasePruneAPI.as_view | 是 | 否 | |
| KEEP | conf | admin | `/api/admin/dashboard_info` | DashboardInfoAPI.as_view | 是 | 否 | |
| KEEP | contest | oj | `/api/contests` | ContestListAPI.as_view | 是 | 否 | |
| KEEP | contest | oj | `/api/contest` | ContestAPI.as_view | 是 | 否 | |
| KEEP | contest | oj | `/api/contest/password` | ContestPasswordVerifyAPI.as_view | 是 | 否 | |
| KEEP | contest | oj | `/api/contest/access` | ContestAccessAPI.as_view | 是 | 否 | |
| KEEP | contest | oj | `/api/contest_rank` | ContestRankAPI.as_view | 是 | 否 | |
| KEEP | contest | admin | `/api/admin/contest` | ContestAPI.as_view | 是 | 否 | |
| KEEP | contest | admin | `/api/admin/contest/clone` | ContestCloneAPI.as_view | 是 | 否 | |
| KEEP | contest | admin | `/api/admin/contest/acm_helper` | ACMContestHelper.as_view | 是 | 否 | |
| KEEP | flowchart | oj | `/api/flowchart/submission` | FlowchartSubmissionAPI.as_view | 是 | 否 | |
| KEEP | flowchart | oj | `/api/flowchart/submissions` | FlowchartSubmissionListAPI.as_view | 是 | 否 | |
| KEEP | flowchart | oj | `/api/flowchart/submission/retry` | FlowchartSubmissionRetryAPI.as_view | 是 | 否 | |
| KEEP | flowchart | oj | `/api/flowchart/submission/detail` | FlowchartSubmissionDetailAPI.as_view | 是 | 否 | |
| KEEP | flowchart | oj | `/api/flowchart/submission/current` | FlowchartSubmissionCurrentAPI.as_view | 是 | 否 | |
| KEEP | flowchart | admin | `/api/admin/flowchart/statistics` | FlowchartStatisticsAPI.as_view | 是 | 否 | |
| KEEP | message | oj | `/api/message` | MessageAPI.as_view | 是 | 否 | |
| KEEP | problem | oj | `/api/problem/tags` | ProblemTagAPI.as_view | 是 | 否 | |
| KEEP | problem | oj | `/api/problem` | ProblemAPI.as_view | 是 | 否 | |
| KEEP | problem | oj | `/api/problem/beat_count` | ProblemSolvedPeopleCount.as_view | 是 | 否 | |
| KEEP | problem | oj | `/api/problem/similar` | SimilarProblemAPI.as_view | 是 | 否 | |
| KEEP | problem | oj | `/api/problem/author` | ProblemAuthorAPI.as_view | 是 | 否 | |
| KEEP | problem | oj | `/api/problem/yearly_ac` | ProblemYearlyACRateAPI.as_view | 是 | 否 | |
| KEEP | problem | oj | `/api/pickone` | PickOneAPI.as_view | 是 | 否 | |
| KEEP | problem | oj | `/api/contest/problem` | ContestProblemAPI.as_view | 是 | 否 | |
| KEEP | problem | admin | `/api/admin/test_case` | TestCaseAPI.as_view | 是 | 否 | |
| KEEP | problem | admin | `/api/admin/sql_test_case_preview` | SQLTestCasePreviewAPI.as_view | 是 | 否 | |
| KEEP | problem | admin | `/api/admin/sql_test_case_scripts` | SQLTestCaseScriptsAPI.as_view | 是 | 否 | |
| KEEP | problem | admin | `/api/admin/sql_test_case_ai_gen` | SQLTestCaseAIGenAPI.as_view | 是 | 否 | |
| KEEP | problem | admin | `/api/admin/problem` | ProblemAPI.as_view | 是 | 否 | |
| KEEP | problem | admin | `/api/admin/problem/visible` | ProblemVisibleAPI.as_view | 是 | 否 | |
| KEEP | problem | admin | `/api/admin/problem/stuck` | StuckProblemsAPI.as_view | 是 | 否 | |
| KEEP | problem | admin | `/api/admin/problem/top_ac_trend` | TopACTrendAPI.as_view | 是 | 否 | |
| KEEP | problem | admin | `/api/admin/problem/flowchart` | ProblemFlowchartAIGen.as_view | 是 | 否 | |
| KEEP | problem | admin | `/api/admin/problem/tag` | TagAdminAPI.as_view | 是 | 否 | |
| KEEP | problem | admin | `/api/admin/problem/batch_tag` | BatchProblemTagAPI.as_view | 是 | 否 | |
| KEEP | problem | admin | `/api/admin/contest/problem` | ContestProblemAPI.as_view | 是 | 否 | |
| KEEP | problem | admin | `/api/admin/contest_problem/make_public` | MakeContestProblemPublicAPIView.as_view | 是 | 否 | |
| KEEP | problem | admin | `/api/admin/contest/add_problem_from_public` | AddContestProblemAPI.as_view | 是 | 否 | |
| KEEP | problemset | oj | `/api/problemset` | ProblemSetAPI.as_view | 是 | 否 | |
| KEEP | problemset | oj | `/api/problemset/<int:problem_set_id>` | ProblemSetDetailAPI.as_view | 是 | 否 | |
| KEEP | problemset | oj | `/api/problemset/<int:problem_set_id>/problems` | ProblemSetProblemAPI.as_view | 是 | 否 | |
| KEEP | problemset | oj | `/api/problemset/progress` | ProblemSetProgressAPI.as_view | 是 | 否 | |
| KEEP | problemset | oj | `/api/user/badges` | UserBadgeAPI.as_view | 是 | 否 | |
| KEEP | problemset | oj | `/api/problemset/<int:problem_set_id>/badges` | ProblemSetBadgeAPI.as_view | 是 | 否 | |
| KEEP | problemset | oj | `/api/problemset/<int:problem_set_id>/users_progress` | ProblemSetUserProgressAPI.as_view | 是 | 否 | |
| KEEP | problemset | admin | `/api/admin/problemset` | ProblemSetAdminAPI.as_view | 是 | 否 | |
| KEEP | problemset | admin | `/api/admin/problemset/visible` | ProblemSetVisibleAPI.as_view | 是 | 否 | |
| KEEP | problemset | admin | `/api/admin/problemset/status` | ProblemSetStatusAPI.as_view | 是 | 否 | |
| KEEP | problemset | admin | `/api/admin/problemset/<int:problem_set_id>` | ProblemSetDetailAdminAPI.as_view | 是 | 否 | |
| KEEP | problemset | admin | `/api/admin/problemset/<int:problem_set_id>/problems` | ProblemSetProblemAdminAPI.as_view | 是 | 否 | |
| KEEP | problemset | admin | `/api/admin/problemset/<int:problem_set_id>/problems/<int:problem_set_problem_id>` | ProblemSetProblemAdminAPI.as_view | 是 | 否 | |
| KEEP | problemset | admin | `/api/admin/problemset/<int:problem_set_id>/badges` | ProblemSetBadgeAdminAPI.as_view | 是 | 否 | |
| KEEP | problemset | admin | `/api/admin/problemset/<int:problem_set_id>/badges/<int:badge_id>` | ProblemSetBadgeAdminAPI.as_view | 是 | 否 | |
| KEEP | problemset | admin | `/api/admin/problemset/<int:problem_set_id>/progress` | ProblemSetProgressAdminAPI.as_view | 是 | 否 | |
| KEEP | problemset | admin | `/api/admin/problemset/<int:problem_set_id>/progress/<int:user_id>` | ProblemSetProgressAdminAPI.as_view | 是 | 否 | |
| KEEP | reaction | oj | `/api/reaction` | ReactionAPI.as_view | 是 | 否 | |
| KEEP | submission | oj | `/api/submission` | SubmissionAPI.as_view | 是 | 否 | |
| KEEP | submission | oj | `/api/submissions` | SubmissionListAPI.as_view | 是 | 否 | |
| KEEP | submission | oj | `/api/submissions/today_count` | SubmissionsTodayCount.as_view | 是 | 否 | |
| KEEP | submission | oj | `/api/format_code` | FormatCodeAPI.as_view | 是 | 否 | |
| KEEP | submission | admin | `/api/admin/submission/rejudge` | SubmissionRejudgeAPI.as_view | 是 | 否 | |
| KEEP | submission | admin | `/api/admin/submission/statistics` | SubmissionStatisticsAPI.as_view | 是 | 否 | |
| KEEP | tutorial | oj | `/api/tutorial` | TutorialAPI.as_view | 是 | 否 | |
| KEEP | tutorial | oj | `/api/tutorials` | TutorialTitlesAPI.as_view | 是 | 否 | |
| KEEP | tutorial | oj | `/api/exercises` | ExerciseAPI.as_view | 是 | 否 | |
| KEEP | tutorial | admin | `/api/admin/tutorial` | TutorialAdminAPI.as_view | 是 | 否 | |
| KEEP | tutorial | admin | `/api/admin/tutorial/visibility` | TutorialVisibilityAPI.as_view | 是 | 否 | |
| KEEP | tutorial | admin | `/api/admin/exercise` | ExerciseAdminAPI.as_view | 是 | 否 | |
| KEEP | utils | admin | `/api/admin/upload_image` | SimditorImageUploadAPIView.as_view | 是 | 否 | |
| KEEP | ai | oj | `/api/ai/analysis` | AIAnalysisAPI.as_view | 否 | 否 | 盲点 2走原生 `fetch``src/oj/store/ai.ts:107`),实际在用 |
| KEEP | ai | oj | `/api/ai/hint` | AIHintAPI.as_view | 否 | 否 | 盲点 2走原生 `fetch``src/oj/problem/components/SubmissionResult.vue:86`),实际在用 |
| KEEP | ai | oj | `/api/ai/class_pk` | ClassPKAnalysisAPI.as_view | 否 | 否 | 盲点 2走原生 `fetch``src/oj/class/pk.vue:179`),实际在用 |
| KEEP | ai | oj | `/api/ai/class_single` | SingleClassAnalysisAPI.as_view | 否 | 否 | 盲点 2走原生 `fetch``src/oj/rank/list.vue:98`),实际在用 |
| KEEP | conf | oj | `/api/judge_server_heartbeat/` | JudgeServerHeartbeatAPI.as_view | 否 | 否 | 非前端调用:判题机向后端注册心跳。新架构判题沙箱镜像原样复用,此接口必须保留 |
| KEEP | submission | oj | `/api/contest_submissions` | ContestSubmissionListAPI.as_view | 否 | 否 | 盲点 1`getSubmissions``endpoint` 变量的比赛分支(`src/oj/api.ts:73`),实际在用 |
---
## 交付核对2026-08-08阶段 5 之后)
把这张表里的 **110 条 KEEP 逐条对到新后端**,确认没有「当初判了要搬、后来忘了」的。
**结果110/110 全部有对应实现,零缺口。**
核对方法:把旧路径和新后端注册的 167 条路由都做词元化(去掉 `/api``admin`
参数段snake/kebab 拆开单复数归一后求交集。87 条自动匹配上,剩下 23 条
API 是重新设计过的,路径本来就对不上)逐条人工落实,见下表。
> 方法的局限:词元匹配只能提示「这两条像是同一个」,不能证明**行为**一致。
> 行为一致性靠的是阶段 3/4 的两轮独立评审和阶段 5 的实跑演练,不是这张表。
### 非显然的改名对照
`problemset``problem-sets` 这类一眼能猜到的没列。下面这些是**猜不到、
日后排查时会卡住人**的:
| 旧Django | 新Bun |
|---|---|
| `POST /api/register` | `POST /api/users` |
| `GET /api/logout` | `DELETE /api/auth/session` |
| `GET /api/hitokoto` | `GET /api/quotes/random` |
| `GET /api/pickone` | `GET /api/problems/random` |
| `GET /api/user_activity_rank` | `GET /api/rankings/activity` |
| `GET /api/profile/fresh_display_id` | `POST /api/me/problem-display-ids/refresh` |
| `GET /api/flowchart/submission/detail` | `GET /api/flowcharts/:id` |
| `POST /api/reaction` | `POST /api/problems/:id/reaction` |
| `PUT /api/admin/problemset/visible` | `PUT /api/admin/problem-sets/:id/visibility` |
| `GET /api/judge_server_heartbeat/` | `POST /api/judge-server/heartbeat` |
最后一条尤其要注意:**判题沙箱镜像是原样复用的**,它靠 compose 里的
`BACKEND_URL` 找后端,三套 compose 都已改成新路径。改动这条要同步改 compose
否则判题机会静默离线。

View File

@@ -1,87 +0,0 @@
# 阶段 0 / 1 / 2 出口标准核验
日期2026-08-07
方法:**逐条实跑,不采信文档声称**。阶段 1 后半段至阶段 3 的实现由外部 agentCodex完成
本次核验按外部代码对待,独立验证。
## 结论
| 阶段 | 出口标准条数 | 通过 | 结果 |
|---|---|---|---|
| 0 存量盘点 | 4 | 4 | ✅ |
| 1 骨架 | 5 | 5 | ✅ |
| 2 判题竖线 | 1端到端 | 1 | ✅ |
## 阶段 0
| 标准 | 实测 |
|---|---|
| `endpoint-inventory.md` 的 REVIEW 计数为 0 | `grep -c '^| REVIEW |'`**0** |
| 清单顶部记录最终数字 | 「合计 127 —— KEEP 110、CUT 17、REVIEW 0新后端需实现 110 个」 |
| jieba 结论写回设计文档 7.3 | `@node-rs/jieba` 在文档中出现 6 处7.3 节为实测结论 |
| `schema.sql` 入库 | `git ls-files` 命中 |
## 阶段 1
| 标准 | 实测 |
|---|---|
| `@oj2/contract` 为符号链接 | `contract -> ../../packages/contract`(另有 api、web 两个 workspace 链接) |
| `schema.ts` 27 张表且无框架表 | `grep -c 'pgTable('`**27**`django_*` / `auth_*` 匹配为空 |
| `/api/problems` 返回真实数据 | HTTP 200返回中文标题含 emoji如「🎮好耶是大冒险字段完整 |
| `apps/web` 构建成功 | `bun run build` 退出码 **0** |
| `/dev-problems` 显示 20 道题 | `GET /api/dev/problems` → 200题目数 **20** |
### 旧 Chrome 兼容约束(硬约束)专项核验
学校机房浏览器版本低,`mermaid-legacy` 等 fallback 依赖与 `legacy()` 插件不可动。实测:
- `apps/web/vite.config.ts``ojnext/vite.config.ts` 差异 **17 行,全部是新增代理**
`/api2``/ws2``/public/avatar`),未触碰 `legacy()` 插件与 Chrome 90 兼容配置
- 依赖 38 → 39**新增仅 `@oj2/contract`,删除 0 个**`mermaid-legacy` 仍在
约束守住。
## 阶段 2
出口标准:一名学生能登录、读公开题、提交代码,并通过 WebSocket 看到真实 JudgeServer 的判题结果。
实测流程(全部真实执行,非模拟):
1. **注册** `POST /api/users` → 201
2. **登录** `POST /api/auth/login` → 200`Set-Cookie: oj2_session=<32 字节随机令牌>`
3. **会话** `GET /api/me` → 200返回用户身份
4. **读公开题** `GET /api/problems/1004` → 200
5. **提交** `POST /api/submissions` → 201返回 `submissionId`
6. **判题** 真实 JudgeServer 执行,返回 `result: 0`ACCEPTED两个测试点均带
`memory` / `cpu_time` / `output_md5`
7. **WebSocket 推送** `ws://localhost:3000/ws/submissions`,订阅后收到三条:
```
{"type":"submission_update","result":6,"status":"pending"}
{"type":"submission_update","result":7,"status":"judging"}
{"type":"submission_update","result":0,"status":"finished","time_cost":0,"memory_cost":1470464,"score":0}
```
8. **统计回写** 题目 `myStatus` 变为 0已通过
### 附带核验:公开题详情无泄露
`GET /api/problems/1004` 的响应字段中**不含** `testCaseId``testCaseScore``answers`
`astRules` —— 学生侧看不到测试点与答案。
## 遗留待处理项(不影响上述判定)
1. **`GET /api/dev/problems``apps/web/src/oj/dev-problems.vue` 仍在。** 二者是阶段 1 的
临时链路验证物,阶段 1 计划正文写明「阶段 3 会删」。`apps/web/src/routes.ts` 里的
`dev-problems` 路由同样待删。
2. **占位用户 `devadmin` 无法登录。**`password` 列是字面量 `'unusable'`(非有效哈希),
`raw_password``'devonly'`。这是阶段 1 为满足 `problem.created_by_id` 外键而造的占位记录,
设计如此,不是缺陷。需要管理员账号做本地验证时得另建。
3. **判题容器健康状态依赖后端。** `oj2-judge` 的 healthcheck 需要能连上后端心跳端点;
API 未启动时 `docker compose ps` 会显示 unhealthy属预期行为非故障。
## 未覆盖
本次只核验出口标准,**未做代码质量与安全审查**。阶段 2、3 的实现均未经过任务级评审
(阶段 0、1 的评审曾逮到 `_id`/`id` 静默碰撞、端点漏抓 5 个、`key()` 误剥前缀等缺陷),
相关审查另行进行,见 `phase3-review-authz.md``phase3-review-leakage.md`

View File

@@ -1,226 +0,0 @@
# 阶段 3 覆盖率对账
日期2026-08-07首次对账2026-08-07 补记(阶段 3 收口)
基准:`docs/specs/endpoint-inventory.md` 的 110 条 KEEP 端点
对象:`apps/api/src/routes/*.ts` 的路由 handler
> ## 状态:阶段 3 出口标准已达成
>
> 出口标准(设计文档第 11 节)是「用户侧全部功能运行在新后端上」。达成判据:
> `apps/web` 里 `oj/` 与 `shared/` 两个目录**已无任何指向旧 Django 的运行时调用**
> 残留的 `utils/http` 引用全是 `import type { ApiResponse }` 这一个类型。
> 仍走旧后端的只剩 `admin/api.ts`85 处)与 `utils/download.ts`,入口都在后台管理界面,属阶段 4。
>
> 收口时补做的三件事:
> 1. 补齐 3 条被用户侧页面调用的 admin 端点(重判 / 提交统计 / 流程图统计),见下表;
> 2. 删除阶段 1 的临时验证物(`GET dev/problems`、`dev-problems.vue`、路由项、`problemSummarySchema`
> 3. 给 `utils/api2.ts` 补上 `login-required` 弹登录框、`permission-denied` 弹提示 ——
> 这两条 `utils/http.ts` 一直有api2 从建包起就漏了,导致此前已迁移的所有端点
> 在鉴权失败时都是「点了没反应」。新加的两个教师专属端点会放大这个问题,故一并补。
>
> 补记:两份评审里的 7 条 Minor 也已全部处理,逐条结论见 `phase3-fix-list.md` 文末。
## 结论
| | 旧端点 KEEP | 新后端已实现 | 缺口 |
|---|---|---|---|
| **oj 侧** | 65 | 65另有 4 条新增) | **0** |
| **admin 侧** | 45 | **3** | **42** |
| 合计 | 110 | 68 | 42 |
**oj 侧已全部覆盖。** admin 侧原计划整块推到阶段 4但其中 3 条被用户侧页面直接调用,
不做完阶段 3 的出口标准(用户侧全部功能跑在新后端上)就不成立,因此在本阶段一并补上:
| 旧端点 | 新路由 | 调用它的用户侧页面 |
|---|---|---|
| `GET admin/submission/rejudge` | `POST submissions/:id/rejudge` | `oj/submission/list.vue` 的重判按钮 |
| `GET admin/submission/statistics` | `GET submissions/statistics` | `StatisticsPanel.vue`(提交列表页 + 题目页) |
| `GET admin/flowchart/statistics` | `GET flowcharts/statistics` | `FlowchartStatisticsPanel.vue`(提交列表页) |
这三条虽然挂在旧后端的 admin 路由下,权限也确实是 `teacher_admin_required` /
`super_admin_required`,但入口在用户侧页面里 —— **「admin 路由」和「admin 页面」不是一回事**
按 URL 前缀切阶段会漏掉它们。剩下 42 条的入口都在后台管理界面,留给阶段 4。
> 对账方法说明:阶段 0 已定案 API 重新设计,新旧路径不同名(如旧 `/api/problem` → 新 `/api/problems`、
> 旧 `/api/pickone` → 新 `/api/problems/random`**无法按字符串自动匹配**。本表由人工按语义逐条比对,
> 判断依据是路由文件归属 + 路径语义 + HTTP 动词。个别条目的对应关系带主观判断,下表逐条列出以便复核。
## oj 侧逐条对照65 条)
| 旧 app | 旧端点 | 新路由 |
|---|---|---|
| account | `login` | `POST auth/login` |
| account | `logout` | `DELETE auth/session` |
| account | `register` | `POST users` |
| account | `profile` | `GET me` / `GET profiles/:username` / `PUT me/profile` |
| account | `profile/fresh_display_id` | `POST me/problem-display-ids/refresh` |
| account | `metrics` | `GET users/:id/metrics` |
| account | `upload_avatar` | `POST me/avatar` |
| account | `user_rank` | `GET rankings/users` |
| account | `user_activity_rank` | `GET rankings/activity` |
| account | `user_problem_rank` | `GET problems/:displayId/rank` |
| achievement | `achievements` | `GET achievements` |
| achievement | `achievements/summary` | `GET achievements/summary` |
| achievement | `achievements/pending` | `GET achievements/pending` |
| ai | `ai/detail` | `GET ai/detail` |
| ai | `ai/duration` | `GET ai/duration` |
| ai | `ai/heatmap` | `GET ai/heatmap` |
| ai | `ai/login_summary` | `GET ai/login-summary` |
| ai | `ai/pinned` | `GET ai/pinned` |
| ai | `ai/analysis` | `POST ai/analysis` |
| ai | `ai/hint` | `POST ai/hint` |
| ai | `ai/class_pk` | `POST ai/class-pk-analysis` |
| ai | `ai/class_single` | `POST ai/class-analysis` |
| announcement | `announcement` | `GET announcements` / `GET announcements/:id` |
| class_pk | `class_rank` | `GET rankings/classes` |
| class_pk | `user_class_rank` | `GET me/class-rank` |
| class_pk | `class_pk` | `POST classes/comparison` |
| conf | `website` | `GET site` |
| conf | `hitokoto` | `GET quotes/random` |
| conf | `class_usernames` | `GET classes/:className/usernames` |
| conf | `judge_server_heartbeat/` | `POST judge-server/heartbeat` |
| contest | `contests` | `GET contests` |
| contest | `contest` | `GET contests/:id` |
| contest | `contest/password` | `POST contests/:id/access` |
| contest | `contest/access` | `GET contests/:id/access` |
| contest | `contest_rank` | `GET contests/:id/rank` |
| flowchart | `flowchart/submission`POST | `POST flowcharts` |
| flowchart | `flowchart/submissions` | `GET flowcharts` |
| flowchart | `flowchart/submission/retry` | `POST flowcharts/:id/retry` |
| flowchart | `flowchart/submission/detail` | `GET flowcharts/:id` |
| flowchart | `flowchart/submission/current` | `GET problems/:id/flowchart/current` |
| message | `message` | `GET messages` / `POST messages` |
| problem | `problem/tags` | `GET problem-tags` |
| problem | `problem` | `GET problems/:displayId` |
| problem | `problem/beat_count` | `GET problems/:id/beat-count` |
| problem | `problem/similar` | `GET problems/:displayId/similar` |
| problem | `problem/author` | `GET problem-authors` |
| problem | `problem/yearly_ac` | `GET problems/:displayId/yearly-ac` |
| problem | `pickone` | `GET problems/random` |
| problem | `contest/problem` | `GET contests/:id/problems` + `GET contests/:id/problems/:displayId` |
| problemset | `problemset` | `GET problem-sets` |
| problemset | `problemset/<id>` | `GET problem-sets/:id` |
| problemset | `problemset/<id>/problems` | `GET problem-sets/:id/problems` |
| problemset | `problemset/progress` | `POST problem-set-progress` / `PUT problem-set-progress` |
| problemset | `user/badges` | `GET users/:username/badges` |
| problemset | `problemset/<id>/badges` | `GET problem-sets/:id/badges` |
| problemset | `problemset/<id>/users_progress` | `GET problem-sets/:id/user-progress` |
| reaction | `reaction` | `GET problems/:id/reaction` / `POST problems/:id/reaction` |
| submission | `submission` | `GET submissions/:id` |
| submission | `submissions` | `GET submissions` |
| submission | `submissions/today_count` | `GET submissions/today-count` |
| submission | `format_code` | `POST code/format` |
| submission | `contest_submissions` | `GET contests/:contestId/submissions` |
| tutorial | `tutorial` | `GET tutorials/:id` |
| tutorial | `tutorials` | `GET tutorials` |
| tutorial | `exercises` | `GET tutorials/:id/exercises` |
### 新增的 4 条(旧后端没有对应)
| 新路由 | 说明 |
|---|---|
| `POST submissions` | 旧后端提交走 `POST /api/submission`,与 `GET submission` 同路径不同动词,拆开后成独立条目 |
| `PUT submissions/:id` | **提交分享开关**(对齐旧 `SubmissionAPI.put` + `ShareSubmissionSerializer`)。判题结果写回走内部 worker不经 HTTP —— 早先这里写成「判题结果写回」,会让人误以为存在一个需要判题机凭据的写入端点 |
| `POST achievements/pending/read` | 成就已读标记 |
| `GET problems/:id/flowchart/history` | 流程图历史 |
| ~~`GET dev/problems`~~ | 阶段 1 的临时验证端点,**已删除**(连同 `dev-problems.vue`、路由与 `problemSummarySchema` |
## admin 侧缺口42 条,按 app
| app | 条数 |
|---|---|
| problem | 14 |
| problemset | 10 |
| conf | 5 |
| contest | 3 |
| tutorial | 3 |
| account | 2 |
| achievement | 2 |
| ai | 1 |
| announcement | 1 |
| utils | 1 |
`problem``problemset` 两块占了 24 条,超过 admin 缺口的一半 —— 排期时应作为主体。
`submission` 原 2 条、`flowchart` 原 1 条已在本阶段做完,见上方表格。)
## 待处理项
1. 本报告的对照关系带人工判断成分,若某条对应有异议,以实际业务行为为准。
2. `utils/download.ts` 仍指向旧后端的 `/api/admin`blob 下载),只被 admin 侧两个页面用,随阶段 4 一起切。
---
## 阶段 5 切换必做项(阶段 4 施工时发现,记在这里以免忘)
> **2026-08-08 更正:这条不是「切换必做项」,降级为「手工造库时的注意事项」。**
>
> 阶段 5 演练时实测了真实的 pg_dumpall 备份:里面带 30 条 `setval`
> 并且把生产快照里所有序列和 `max(id)` 逐个对过,**错位 0 个**。
> 下面这个现象只出现在我手工按显式 id 导入、又没补 setval 的本地库上,
> 对正常的备份/恢复不成立。切换当天不用管序列。详见
> [phase5-cutover-runbook.md](phase5-cutover-runbook.md)。
**导入数据后必须重置全部序列。** 本地库是按显式 id 从生产导入的,
`problem_tag_id_seq` 停在 6 而表里 max(id)=87于是第一次新建标签就撞
`duplicate key value violates unique constraint "problem_tag_pkey"`500
生产切换若沿用同一个库则不受影响(序列本来就是对的);但只要有任何一步是
「导出 → 导入到新库」,就必须补这一句:
```sql
do $$
declare r record; mx bigint;
begin
for r in
select split_part(pg_get_serial_sequence(quote_ident(t.table_name), c.column_name), '.', 2) as seqname,
t.table_name, c.column_name
from information_schema.tables t
join information_schema.columns c on c.table_name = t.table_name
join pg_sequences s on s.sequencename = split_part(pg_get_serial_sequence(quote_ident(t.table_name), c.column_name), '.', 2)
where t.table_schema = 'public'
loop
execute format('select coalesce(max(%I),0) from %I', r.column_name, r.table_name) into mx;
execute format('select setval(%L, greatest(%s, 1))', r.seqname, mx);
end loop;
end $$;
```
症状很隐蔽:读全部正常,只有**写**才炸,而且是导入后第一次写才炸。
---
## SQL 判题链路(阶段 2 补课2026-08-07
阶段 4 做题目管理时才发现:新后端**完全没有 SQL 判题**。旧后端有
`judge/sql_runner.py`378 行)+ `sql_dispatcher.py`113 行),走的是与沙箱完全
不同的路径(跑 SQLite 比结果集)。阶段 2 纵切时只打通了沙箱那条线,漏了这条。
### 防护为什么换了实现
旧实现靠 Python sqlite3 的三件套。`bun:sqlite` 一个都没有,实测:
| | 结论 |
|---|---|
| `setAuthorizer` / `setProgressHandler` / `setLimit` | 均无 |
| `PRAGMA max_page_count` | 有效 |
| `Worker.terminate()` 能否停掉跑飞的查询 | **不能** —— 递归 CTE 死循环卡死整个 worker只能从外面杀进程 |
| `node:sqlite` | 该 Bun 版本不可用 |
因此改成「WASM 引擎sql.js+ 独立子进程」,逐条替代:
| 旧防护 | 新做法 | 实测 |
|---|---|---|
| authorizer 禁 ATTACH | WASM 无宿主文件系统绑定,**结构上**够不到 | `attach '/etc/passwd'``unable to open database` |
| authorizer 白名单让查询题只读 | `PRAGMA query_only=1` | 查询题里 INSERT → 运行错误并说明 |
| progress_handler 墙钟超时 | 子进程外部 SIGKILL | 递归 CTE 死循环 → CPU 超时 |
| `setlimit(LIMIT_LENGTH)` | 子进程 `ulimit -d` | `hex(zeroblob(2e8))` → 内存超限 |
ATTACH 这条比旧实现**更强**:旧的靠 authorizer 拦,新的是够不到。
### 两个踩过的坑
1. **`ulimit` 必须用 `-d` 不能用 `-v`。** `-v` 限虚拟地址空间,而 JS 引擎预留巨量地址;
实测 `-v` 之下 Bun 退出时有概率 panicSIGILL结果早已写出但进程异常终止
父进程读到空串误判成超时 —— 6 次里坏 2 次,时好时坏。换 `-d`(实际提交内存,
Linux 4.7 起也覆盖匿名 mmap后 12/12 稳定。
2. 子进程写完结果**直接 SIGKILL 自己**,不走 `process.exit()` —— 后者仍有一段清理会撞限额。

View File

@@ -1,199 +0,0 @@
# 阶段 3 修复清单(两份评审合并)
日期2026-08-07
来源:`phase3-review-authz.md`(权限边界)+ `phase3-review-leakage.md`(数据泄露)
受审代码commit `8c00cdc`oj 侧 65 个端点
> ## 状态Critical / Important 已修复并复评通过7 条 Minor 已于 2026-08-07 全部处理(见文末)
>
> 修复提交:`b4b61af` / `8237909` / `b7adf29` / `f548aef`
> 复评报告:`phase3-review-rereview.md` —— 8 条F1-F6 + 收尾的 F4b、F5b**全部 ADDRESSED**
> 修复 diff 内无新引入破坏。
>
> 复评补上了控制方没验充分的一条:**F4b 的 `contestId`**。控制方当时用的提交本就不属于比赛,
> `contestId` 天然为 null运行时证据不成立。复评真造了一条比赛提交`contestId=7`、`ip` 有值),
> 确认数据库存的是真值、而 API 返回给提交者本人的是 `info:{}` / `ip:null` / `contestId:null`。
>
> 复评另核实:`sampleUser()` 是真正的默认关闭开关(覆盖全部 14 个下发点,含本清单未列的
> rankings 与题目列表/详情);`isRegularUser` 全仓确实只有一个调用点;`.env` 加载器的优先级
> 正确(真实环境变量 > cwd 的 .env > 仓库根 .env畸形行不崩、缺文件静默跳过限流参数
> `fill_rate=0.03` 与旧后端 `options/options.py` 逐值吻合,且拦截位置与
> `submission/views/oj.py` 一致(比赛权限校验之后、取题目之前)。
## 合并说明
两份评审独立进行、互不知情,却各自命中了同两条问题(`/profiles/:username` 匿名可读、
`realName` 无条件下发)。**独立复现提高了可信度**,此处合并为一条。
两份评审对同一问题的严重度判定不一致时,**取更严的一方**。理由:使用者是中职学生(未成年人),
姓名、邮箱、班级属于个人信息,泄露的后果不由技术标准衡量。
控制方(本文档作者)已对 F1、F2、F3 独立实跑复现,证据附在各条下。
---
## F1 —— 匿名可读任意用户完整档案 【Critical】
- **位置**`apps/api/src/routes/account.ts:101`(路由)、`apps/api/src/services/profile.ts:19`(字段)
- **两份评审均命中**authz C1 / leakage I-1
- **问题**`GET /profiles/:username` 只挂 `optionalAuth`handler 内无登录判断,匿名可读
`email``adminType``problemPermission``isDisabled``className``lastLogin`
- **旧后端行为**`OnlineJudge/account/views/oj.py``UserProfileAPI.get` 第一行即
`if not user.is_authenticated: return self.success()` —— 匿名直接返回空
- **放大效应**:用户名可经 `GET /rankings/users` 公开枚举,因此可无 cookie 批量收集
全校学生的邮箱与最后登录时间
**控制方实跑证据**
```
匿名 GET /api/profiles/e2etest -> 200
含 email: true | 含 adminType: true | 含 realName: true
{"user":{"username":"e2etest","email":"e2e@local.test","adminType":"Regular User",
"problemPermission":"None","lastLogin":"...","isDisabled":false,"className":"2301"}}
```
**修法**handler 开头判断未登录即返回空,对齐旧行为。
---
## F2 —— 学生真名无条件下发 【Critical】
- **位置**13 个下发点,其中 8 个匿名可达;`apps/api/src/routes/account.ts:168` 为典型
- **两份评审均命中**authz C2 / leakage I-2
- **问题**:旧后端将「是否返回真名」设计为 DRF `UsernameSerializer(need_real_name=False)`
**默认关闭**开关,全仓 11 处调用中仅比赛榜单一处显式打开。新后端未搬运这一层,真名随
用户对象无条件下发
- **唯一做对的地方**`apps/api/src/routes/contest.ts:209`
**控制方实跑证据**
```
匿名 GET /api/rankings/users -> 200
[{"user":{"id":4,"username":"e2etest","realName":"..."},...},
{"user":{"id":2,"username":"student","realName":"Phase 2 Student"},...}]
```
`"Phase 2 Student"` 为本机既有数据,非评审探针造出)
**修法**:在用户对象的序列化层加一个默认关闭的 `includeRealName` 开关,逐个下发点显式开启;
默认不给。**不要逐处删字段** —— 那样下次新增端点会重犯。
---
## F3 —— 匿名绕过提交可见性守卫,权限大于登录用户 【Critical】
- **位置**`apps/api/src/routes/submission.ts:211``apps/api/src/routes/helpers.ts:27`
- **来源**authz C3leakage 未覆盖)
- **问题**
```ts
export function isRegularUser(user: AuthUser | null | undefined) {
return user?.adminType === "Regular User" // isRegularUser(null) === false
}
// submission.ts:211
if (!(await getBooleanOption("submission_list_show_all", true)) && isRegularUser(user)) {
// 限制为只看自己的提交
}
```
匿名用户的 `isRegularUser(null)` 为 `false`,守卫整体短路,限制不生效
- **潜伏性**`submission_list_show_all` 默认为 `true`,该分支平时不执行,**开关一旦关闭立即暴露**。
而关闭这个开关的典型场景正是考试
**控制方实跑证据**(临时把开关置 false测后已还原
```
匿名 total = 10 ← 全部可见
登录学生 total = 0 ← 被正确限制
```
**修法**:守卫应为「非管理员即受限」,而非「是普通用户才受限」。匿名必须落入受限分支。
建议改用 `!isAdminRole(user)`(该函数对 null 返回 `false`,语义正确)。
---
## F4 —— 自己的提交详情返回判题内部信息与 IP 【Important】
- **位置**`apps/api/src/routes/submission.ts:187`
- **来源**leakage I-3
- **问题**:返回 `info`(含每个测试点的 `test_case` 编号与 `output_md5`)与 `ip`
- **旧后端行为**:这两个字段以 `is_admin_role()` 把关,而非「是不是自己的提交」
- **风险**:测试点编号与输出 md5 可用于反推测试数据规模与部分答案特征
**修法**`info` 与 `ip` 改为仅管理员可见,对齐旧后端。
---
## F5 —— 判题机 token 默认值硬编码进仓库 【Important】
- **位置**`docker/compose.dev.yml` 的 `${OJ2_JUDGE_TOKEN:-oj2-dev-token}` 及后端读取处
- **来源**authz Important
- **问题**token 校验本身实现正确(用了 `timingSafeEqual`),但缺省值 `"oj2-dev-token"`
写死在仓库里。旧后端在 env 缺失时用 `rand_str()` fail-safe宁可不可用也不用弱默认值
- **当前影响有限**`judge/run.ts` 用 env 里的 URL 而非数据库中的 `service_url`
**修法**去掉默认值env 缺失时启动失败或生成随机值,不要静默使用弱默认。
---
## F6 —— 提交接口缺少限流 【Important】
- **位置**`apps/api/src/routes/submission.ts` 的 `POST /submissions`
- **来源**authz Important
- **问题**:旧后端有 TokenBucket 限流,新后端未搬运
- **风险**:判题沙箱是有限资源,学生(或脚本)可连续提交打满队列
**修法**:按旧后端的限流参数重建。
---
## Minor共 7 条)—— 2026-08-07 已全部处理
在 admin 侧开工前清掉,避免同样的模式被复制 42 次。逐条结论:
| 编号 | 内容 | 处理 |
|---|---|---|
| authz M1 | `isAdminRole` 从白名单退化为黑名单 | **改回白名单**`ADMIN_ROLES` / `TEACHER_ROLES` 显式列举,对齐 `account/models.py:65-73`)。实测四种已知角色行为不变,而虚构的新角色「助教」现在默认**不是**管理员——黑名单写法下它会默认拿到管理员权限 |
| authz M2 | 比赛权限判断没有中间件兜底 | **新增 `requireContestAccess(checkType, paramName)` 中间件**`services/contest.ts`),把「取比赛 → 404 → 鉴权 → 401/403」收进路由注册行。手工调用点从 5 处降到 2 处:`GET /contests/:id/access` 是**报告**权限而非强制(不能 403`POST /submissions` 的比赛 id 来自请求体、中间件跑时 body 还没解析,两处都就地写了说明 |
| authz M3 | `blog` / `github` 从 URLField 降级为自由字符串 | **加回 URL 校验**(只放行 `http(s)://`,空串表示清空)。实测 `javascript:alert(1)` 与 `not a url` 均 400`https://example.com/x` 与空串 200 |
| authz M4 | `GET /dev/problems` 仍在线且无鉴权 | **已删除**(连同 `dev-problems.vue`、路由项、`problemSummarySchema`),实测 404 |
| 文档纠错 | `phase3-coverage.md` 把 `PUT submissions/:id` 写成「判题结果写回」 | **已改正**为「提交分享开关」,并注明判题结果写回走内部 worker 不经 HTTP |
| leakage M-1 | 站内信内嵌的 submission 多出 `info` / `ip` 两个空键 | **新增 `embeddedSubmissionSchema`**`omit` 掉 `info`/`ip`/`contestId`),不再复用 `submissionDetailSchema` 传空值——形状对上了,将来有人把空值改成真值也不会变成泄露。实测三个键均已消失 |
| leakage M-2 | `problem-sets/:id/user-progress` 返回学生 `realName` | **无需改动**F2 的 `sampleUser()` 默认关闭开关已经覆盖此处,现在返回 `null`,与旧后端 `UsernameSerializer()` 一致。报告提的「别把教师端功能删掉」的顾虑不成立——旧后端本来就不给 |
| leakage M-3 | `tutorials/:id/exercises` 下发练习答案 | **不改**:旧后端逐字相同(`ExerciseSerializer` 整个 `data` jsonb 出去,且无 `@login_required`)。这是教程练习「答案下发到浏览器、客户端比对」的既有设计,属遗留设计债,不是本次重写引入的回归。要改得连判题方式一起改,不在迁移范围内 |
### 顺带修掉的一个真回归(评审未覆盖)
`GET /api/messages` 内嵌的 submission 原本给的是 `problemId`(数字主键),而旧后端
`SubmissionSafeModelSerializer` 的 `problem` 是 `SlugRelatedField(slug_field="_id")`
即**展示用题号**。`oj/user/message.vue:20` 拿它拼 `/problem/<题号>` 链接,迁移后拼出的是
`/problem/undefined`,题号那一栏也是空的。已改为下发 `problem`(展示题号),实测返回 `"1004"`。
---
## 评审确认没有问题的部分
两份评审各自独立核实、结论一致的部分:
- **无任何敏感字段泄露**`raw_password`、`password`、`auth_token`、`open_api_appkey`、
`session_keys` 在 111 个请求的响应中零命中(含明文密码值的全文 grep
- **无泄题**`answers`、`ast_rules`、`test_case_id`、`test_case_score` 均未泄露,
已在 `answers` 含完整 C/Python 参考解的题目 1002 上专项验证
- **模板隐藏区正确剥离**
- **`contest.password` 未泄露**
- **11 处 `db.select()` 选全列的写法全部后接显式字段映射 + Zod strip无一泄露**
- **比赛权限是重建得最好的一块**`sig#timestamp` 密码、未开始拦截、`ProblemSafeSerializer`
脱敏字段集逐行对齐,五个端点实跑全部正确拦截
- 覆盖面111 个请求65 个 oj 路由中 61 个取得 2xx
---
## 附带需处理:本地样本数据受损
泄露评审的种子脚本覆盖了**题目 1001`problem.id=2`**的 `ast_rules`、`answers`、
`test_case_score`、`template`,原值已丢失(评审用兄弟题目的值做了近似填充,当前
`answers` 为 `[]`)。导入用的 `/tmp/problems.csv` 也已不存在。
影响本地样本数据的价值在于真实真实富文本、LaTeX、中文最能暴露序列化问题
1001 现已失真。
处理:需要时从生产重新导出该行:
```bash
docker exec oj-postgres psql -U onlinejudge -d onlinejudge -c \
"\copy (SELECT * FROM problem WHERE _id='1001') TO STDOUT WITH CSV HEADER" > p1001.csv
```
不紧急 —— 其余 19 道题未受影响。

View File

@@ -1,433 +0,0 @@
# 阶段 3 修复报告
日期2026-08-07
需求文档:`docs/specs/phase3-fix-list.md`(两份评审合并的 6 条 findings
受修代码:`apps/api`Hono + Drizzle基线 commit `9c04b00`
提交:
| 短 SHA | 标题 | 覆盖 |
|---|---|---|
| `b4b61af` | fix(阶段3): 匿名不可读用户档案,真名改为默认不下发 | F1、F2 |
| `8237909` | fix(阶段3): 提交可见性守卫补上匿名,详情脱敏,提交接口加限流 | F3、F4、F6 |
| `b7adf29` | fix(阶段3): 去掉判题机 token 的弱默认值 | F5 |
验证环境postgres `:5433`、redis `:6380`、judge `:8081`API 跑在 `:3000``bun --watch`,改完自动重载)。
测试账号 `e2etest`/`Test123456`Regular User`student`Regular User真名 `Phase 2 Student`)、
`devadmin`Super Admin
`cd apps/api && bunx tsc --noEmit`**0 错误**
14 条路径 × {匿名, 登录} 冒烟 → **无 5xx**
---
## F1 —— 匿名可读任意用户完整档案【Critical】
**改法**`apps/api/src/routes/account.ts:101` 的 handler 开头加登录判断,未登录返回空。
对齐旧后端 `OnlineJudge/account/views/oj.py:40``UserProfileAPI.get` 首行
`if not user.is_authenticated: return self.success()`
`services/profile.ts` **没动** —— 它已有 `showRealName` 形参,且调用方传的是
`c.get("user")?.id === target.id`(只有看自己的档案才给真名),这一点与旧后端
`UserProfileSerializer(profile, show_real_name=show_real_name)` 的语义一致。
**实跑对比**(为让证据可读,临时把 `e2etest``real_name` 设为 `张三-测试`,测后已还原为 `NULL`
修复前:
```
### F1 anon GET /api/profiles/e2etest
status 200 -> {"data":{"id":2,"user":{"id":4,"username":"e2etest","email":"e2e@local.test",
"adminType":"Regular User","problemPermission":"None","createTime":"2026-08-07 07:19:10.789+00",
"lastLogin":"2026-08-07 07:50:40.139+00","openApi":false,"isDisabled":false,"className":null},
"realName":null,"acmProblemsStatus":{"problems":{"5":{"_id":"1004","status":0}}},...
### F1 logged-in (self) GET /api/profiles/e2etest
status 200 email: e2e@local.test realName: 张三-测试
```
修复后:
```
### F1 anon GET /api/profiles/e2etest
status 200 -> {"data":null}
### F1 logged-in (self) GET /api/profiles/e2etest
status 200 email: e2e@local.test realName: 张三-测试
```
匿名拿到空,登录看自己的档案不受影响。
**前端影响:无。** `apps/web/src/shared/api.ts:28` 已经有
`if (response.data === null) return { error: null, data: null }` 的分支,天然兼容。
---
## F2 —— 学生真名无条件下发【Critical】
**改法(结构性,不是逐处删字段)**`apps/api/src/routes/helpers.ts` 新增序列化层:
```ts
export function sampleUser(
source: { id: number; username: string },
realName: string | null | undefined,
options: { includeRealName?: boolean } = {},
): SampleUser
```
`realName` 默认不下发,需要的地方显式传 `{ includeRealName: true }`。这是旧后端
`OnlineJudge/utils/api/_serializers.py``UsernameSerializer(need_real_name=False)`
同一套约定:全仓 11 处调用只有 `contest/serializers.py:84` 一处显式打开。
函数上写了注释,说明「所有下发用户对象的地方都必须走这个函数,不要再手写
`{ id, username, realName }`」——目的就是让后面 admin 侧 45 个端点铺开时不会重犯。
### 13 个下发点清单及处置
| # | 位置 | 端点 | 匿名可达 | 处置 |
|---|---|---|---|---|
| 1 | `routes/account.ts:168` | `GET /rankings/users` | 是 | `sampleUser()`,关 |
| 2 | `routes/problem.ts:86``listItem()` | `GET /problems``GET /problems/:displayId/similar` | 是 | `sampleUser()`,关 |
| 3 | `routes/problem.ts:333` | `GET /problems/:displayId` | 是 | 原本就写死 `null`;改为走 `sampleUser()` 统一入口 |
| 4 | `routes/contest.ts:34``creator()` | `GET /contests``GET /contests/:id` | 是 | `sampleUser()`,关 |
| 5 | `routes/contest.ts:131` | `GET /contests/:id/problems` | 否(比赛权限) | `sampleUser()`,关 |
| 6 | `routes/contest.ts:177` | `GET /contests/:id/problems/:displayId` | 否(比赛权限) | `sampleUser()`,关 |
| 7 | **`routes/contest.ts:209`** | `GET /contests/:id/rank` | 否 | **`{ includeRealName: admin }`,保留** —— 唯一打开的一处 |
| 8 | `routes/content.ts:43` | `GET /announcements` | 是 | `sampleUser()`,关 |
| 9 | `routes/content.ts:64` | `GET /announcements/:id` | 是 | `sampleUser()`,关 |
| 10 | `routes/content.ts:85` | `GET /messages` | 否(`requireAuth` | `sampleUser()`,关 |
| 11 | `routes/content.ts:196` | `GET /tutorials/:id` | 是 | `sampleUser()`,关 |
| 12 | `routes/problemset.ts:62``problemSetCreator()` | `GET /problemsets``GET /problemsets/:id` | 是 | `sampleUser()`,关 |
| 13 | `routes/problemset.ts:169` | `GET /problemsets/:id/problems` | 是 | `sampleUser()`,关 |
| 14 | `routes/problemset.ts:398` | 题单进度列表 | 否(教师及以上) | `sampleUser()`,关。旧后端 `problemset/serializers.py:248` 同样是默认关 |
清单比文档说的「13 个」多一条:文档大概把 `problem.ts:333`(本来就写死 `null`)没计入,
或者把 `problem.ts:86` 服务的两个端点算成一个。**14 处全部收口,无遗漏**
```
$ grep -rn "realName:" apps/api/src/routes/*.ts | grep -v sampleUser \
| grep -v "realName: schema.userProfile.realName" | grep -v "string | null"
account.ts:94: realName: null, ← 注册时写库的 insert values不是下发点
helpers.ts:23: realName: options.includeRealName === true ? (realName ?? null) : null,
```
`services/profile.ts:29``realName` 由 F1 的 `showRealName` 形参把关(只有看自己的档案才给),
与旧后端 `UserProfileSerializer` 一致,未改。
**实跑对比**(修复前用 `git stash` 把改动摘掉实测,非静态推断):
修复前:
```
--- announcements (anon) ---
[{"id":2,"username":"student","realName":"Phase 2 Student"}]
--- contests createdBy (anon) ---
[{"id":4,"username":"e2etest","realName":"张三-测试"}]
--- contest rank as CONTEST ADMIN (e2etest, creator) ---
[{"id":2,"username":"student","realName":"Phase 2 Student"}]
--- contest rank as NON-admin (student) ---
[{"id":2,"username":"student","realName":null}]
--- contest detail createdBy (anon) ---
{"id":4,"username":"e2etest","realName":"张三-测试"}
### 匿名 GET /api/rankings/users
[{"id":4,"username":"e2etest","realName":"张三-测试"},{"id":2,"username":"student","realName":"Phase 2 Student"}]
```
修复后:
```
--- announcements (anon) ---
[{"id":2,"username":"student","realName":null}]
--- contests createdBy (anon) ---
[{"id":4,"username":"e2etest","realName":null}]
--- contest rank as CONTEST ADMIN (e2etest, creator) ---
[{"id":2,"username":"student","realName":"Phase 2 Student"}] ← 仍然有,符合预期
--- contest rank as NON-admin (student) ---
[{"id":2,"username":"student","realName":null}]
--- contest detail createdBy (anon) ---
{"id":4,"username":"e2etest","realName":null}
### 匿名 GET /api/rankings/users
[{"id":2,"username":"student","realName":null},{"id":4,"username":"e2etest","realName":null}]
### 匿名 GET /api/problems (createdBy)
[{"id":1,"username":"devadmin","realName":null},{"id":1,"username":"devadmin","realName":null}]
```
比赛榜单对**比赛管理员**(这里是比赛创建者 `e2etest`)仍然下发真名,对非管理员仍然是 `null` —— 行为未变。
**前端影响**`apps/web` 只在 `oj/api.ts:86``shared/api.ts:37,47``realName`,都做了
`?? ""` 的兜底,字段变 `null` 不会炸。真名本来也不该在这些位置显示。
---
## F3 —— 匿名绕过提交可见性守卫【Critical】
**改法**`routes/submission.ts:220``isRegularUser(user)` 改为 `!isAdminRole(user)`
(非管理员即受限,匿名落入受限分支)。
**`isRegularUser` 的其他调用点:查过了,全仓只有这一处。**
```
$ grep -rn "isRegularUser" --include=*.ts apps/ packages/
apps/api/src/routes/submission.ts:31 import
apps/api/src/routes/submission.ts:211 (唯一调用点,即本 finding
apps/api/src/routes/helpers.ts:27 (定义)
```
既然零个其他调用点,**把 `isRegularUser` 整个删掉**,原地留了一条注释说明为什么不要再加回来。
留着一个对 null 返回 false 的「是普通用户才受限」判定,就是给下一个人准备的坑。
(这一步超出了「只改 findings」的字面范围但属于 F3 的根因,不是顺手重构。)
**实跑对比**(临时把 `options_sysoptions``submission_list_show_all` 置 false
**该行原本不存在,测后已 DELETE**
修复前:
```
### F3 GET /api/submissions with submission_list_show_all=false
anon total = 23 ← 全部可见
student total = 0 ← 被正确限制
```
修复后:
```
### F3 GET /api/submissions with submission_list_show_all=false
anon total = 0
student total = 0
```
还原核验:
```
$ bun db.ts delete → deleted rows: 1
$ bun db.ts show → [] (该 key 无行,回到原始状态)
```
---
## F4 —— 提交详情返回判题内部信息与 IP【Important】
**改法**`routes/submission.ts:195``const full = isAdminRole(user) || row.submission.userId === user.id`
改为 `const full = isAdminRole(user)`
依据旧后端 `OnlineJudge/submission/views/oj.py:100-104`
```python
if request.user.is_admin_role():
submission_data = ... SubmissionModelSerializer ... # fields = "__all__"
else:
submission_data = ... SubmissionSafeModelSerializer ... # exclude = ("info", "contest", "ip")
```
把关的是**角色**,不是「是不是自己的提交」。
**实跑对比**(用一条已判完、`info` 有内容的真实提交;`devadmin` 是 Super Admin
修复前(提交所有者 `e2etest`Regular User
```
### F4 own submission detail
id 0249d3606fab3e05f4926c45d00efdee | has info: {"err":null,"data":[{"error":0,"memory":7811072,
"output":null,"result":-1,"signal":0,"cpu_time":3,"exit_code":0,"real_time":8,
"test_case":"1","output_md5":"c4ca4238a0b923820dcc509a6f75849b"},{"error": ... | ip: null
```
修复后:
```
submission 0249d3606fab3e05f4926c45d00efdee
OWNER (Regular User) -> info: {} | ip: null | result: -1
ADMIN (Super Admin) -> info: {"err":null,"data":[{"error":0,"memory":7811072,"output":null,
"result":-1,"signal":0,"cpu_time":3,"exit_code":0,"real_time":8,"test_case":"1",
"output_md5":"c4ca4238a0b923820dcc509a6f75849b"},{"error":0,"memory":7749632, ... | ip: null
```
上面这条老提交的 `ip` 本来就是 `NULL`(本机请求没有 `X-Forwarded-For`
为了单独验证 `ip` 确实被挡住,另造了一条带 `X-Forwarded-For: 10.11.12.13` 的提交:
```
库里实际值: [{"id":"...","ip":"10.11.12.13","info_type":"object"}]
OWNER (Regular User) -> info: {} | ip: null
ADMIN (Super Admin) -> info: {} | ip: "10.11.12.13"
```
库里有值、学生看不到、管理员看得到 —— `ip` 是被脱敏而不是本来就空。
### 前端影响(这条确实波及前端)
`apps/web/src/oj/submission/detail.vue:149`
```html
<n-data-table
v-if="!hideList && submission.info && submission.info.data"
:columns="columns"
:data="submission.info.data"
/>
```
改完之后学生拿到 `info: {}``submission.info.data``undefined``v-if` 不成立,
**测试点结果表格不再渲染。不会报错,只是不显示。**
按要求仍按旧后端行为修。补充一点判断依据:**这不算功能回退**。旧后端 `SubmissionSafeModelSerializer`
本来就 `exclude=("info", ...)`,也就是说**生产环境的学生从来就没看到过这张表**——
`v-if` 的存在本身就是为这个场景写的。所以新后端这一版是「多给了」,现在改回去,
前端不需要适配,线上行为反而回到一致。
**另注意**:旧后端的 `SubmissionSafeModelSerializer` 还 exclude 了 `contest`
新后端仍无条件下发 `contestId`。这超出 F4 的字面范围(评审只点了 `info``ip`
本次**未改**,记在下方「疑虑」里。
---
## F5 —— 判题机 token 默认值硬编码【Important】
**改法**(三处):
1. `apps/api/src/config.ts`:抽出 `judgeServerToken()`env 缺失时 `randomBytes(32).toString("hex")`
`console.warn` 告警。对齐旧后端 `OnlineJudge/options/options.py:93`
```python
token = os.environ.get("JUDGE_SERVER_TOKEN")
return token if token else rand_str()
```
选「随机 fail-safe」而不是「启动失败」就是为了不把本地开发搞死 —— 服务照起,
只是判题机心跳被 403 挡掉,日志里有明显告警。
2. `docker/compose.dev.yml``${OJ2_JUDGE_TOKEN:-oj2-dev-token}` → `${OJ2_JUDGE_TOKEN:?...}`
未设置时 compose 直接报错退出。文件顶部加了本地怎么设的三行命令。
3. `.env.example``JUDGE_SERVER_TOKEN=` 留空 + 生成命令注释,说明
`JUDGE_SERVER_TOKEN`(后端读)与 `OJ2_JUDGE_TOKEN`(判题机容器读)名字不同但值必须一致。
**实跑对比**
后端 —— 修复前 `config.judgeServerToken` 恒为 `"oj2-dev-token"`;修复后:
```
--- no env (fail-safe random) ---
[config] JUDGE_SERVER_TOKEN 未设置,已生成一次性随机 token。判题机将无法通过鉴权
本地开发请在 .env 里设置 JUDGE_SERVER_TOKEN并让 docker/compose.dev.yml 的 OJ2_JUDGE_TOKEN 取同一个值。
token: 9acd77a3ea1bbd17... len=64
--- second run: different value (proves it is random, not a repo constant) ---
token: f2328363bcda0ad9...
--- with env set ---
token: real-token-from-env
```
两次运行值不同 → 确实是随机不是换了个仓库常量。env 存在时原样使用。
compose
```
--- compose without OJ2_JUDGE_TOKEN ---
error while interpolating services.judge.environment.TOKEN:
required variable OJ2_JUDGE_TOKEN is missing a value: 请先设置 OJ2_JUDGE_TOKEN见本文件顶部注释
--- compose with OJ2_JUDGE_TOKEN ---
TOKEN: abc123
```
(当前跑着的判题机容器是用旧的 `oj2-dev-token` 起的,本次没有重启它 ——
下次 `docker compose up` 之前需要按注释设一次 `OJ2_JUDGE_TOKEN` 与 `JUDGE_SERVER_TOKEN`。)
---
## F6 —— 提交接口缺少限流【Important】
**参数来自旧后端,不是拍脑袋定的**
- `OnlineJudge/utils/throttling.py` —— TokenBucket 算法
- `OnlineJudge/options/options.py:120` —— 默认参数
```python
throttling = {"ip": {"capacity": 100, "fill_rate": 0.1, "default_capacity": 50},
"user": {"capacity": 20, "fill_rate": 0.03, "default_capacity": 10}}
```
- `OnlineJudge/submission/views/oj.py:36-42` —— `SubmissionAPI.throttling` **只用 user 桶**
key 是 `str(request.user.id)``auth_method == "api_key"` 时直接跳过
**改法**:新增 `apps/api/src/services/throttling.ts`
- 同样只挂 user 桶、同样按 user id 做 key默认参数逐字照抄且和旧后端一样
**实际值以数据库 `throttling` 配置项为准**,缺失时才用默认值
- 落在 redis`throttling:user:<id>`TTL = `capacity/fill_rate + 60` 秒,每次调用刷新
- **改用 Lua 脚本做成原子操作**。旧实现自己在 docstring 里写明「对于单个 key 的操作不是线程安全的」,
而限流要挡的正是并发突发 —— 读改写有竞态等于没挡。算法与参数不变,只是把它做对
- 挂点位置与旧后端一致:比赛权限校验之后、取题目之前
- 超限返回 `429 too-many-submissions` + `Please wait N seconds`(旧后端文案
`"Please wait %d seconds" % int(wait)`)。为此 `http.ts` 的 `failure()` 状态码联合类型加了 `429`
**实跑对比**(同一账号连打 15 次 `POST /api/submissions`
修复前:
```
### F6 rapid POST /api/submissions x15
#1 201 #2 201 #3 201 #4 201 #5 201
#6 201 #7 201 #8 201 #9 201 #10 201
#11 201 #12 201 #13 201 #14 201 #15 201
```
15/15 全过,判题队列可以被一个脚本随便打满。
修复后:
```
### F6 rapid POST /api/submissions x15
#1 201 ... #9 201
#10 429:{"error":{"code":"too-many-submissions","message":"Please wait 23 seconds"}}
#11 429 ... #15 429
```
9 条通过后开始 429。桶初始 `default_capacity=10`,此前 F4 的验证提交已消耗 2 个、
期间回填约 1 个,落在 9 —— 与参数吻合。
---
## 改动文件清单
| 文件 | finding |
|---|---|
| `apps/api/src/routes/account.ts` | F1、F2 |
| `apps/api/src/routes/helpers.ts` | F2新增 `sampleUser`、F3删除 `isRegularUser` |
| `apps/api/src/routes/contest.ts` | F2 |
| `apps/api/src/routes/content.ts` | F2 |
| `apps/api/src/routes/problem.ts` | F2 |
| `apps/api/src/routes/problemset.ts` | F2 |
| `apps/api/src/routes/submission.ts` | F3、F4、F6 |
| `apps/api/src/services/throttling.ts` | F6新文件 |
| `apps/api/src/http.ts` | F6`failure()` 支持 429 |
| `apps/api/src/config.ts` | F5 |
| `docker/compose.dev.yml` | F5 |
| `.env.example` | F5 |
`apps/web` **一个字节没改**F1/F4 的前端影响见上文,均无需适配)。
---
## 纪律自查
- **旧仓库冻结**`OnlineJudge/` 与 `ojnext/` 的 `git status --short` 均为空,全程只读。
- **没写测试**:符合项目既定策略,验证一律走实跑 API。
- **没有范围蔓延**:唯一超出 findings 字面范围的是「删掉 `isRegularUser`」和
「`http.ts` 的 `failure()` 加 429」前者是 F3 的根因、后者是 F6 的必要条件,都在上面说明了。
- **数据库改动全部还原**
| 改动 | 还原 | 核验 |
|---|---|---|
| `e2etest.real_name` 设为 `张三-测试` | 改回 `NULL`(原值已知) | `[{"username":"student","real_name":"Phase 2 Student"},{"username":"e2etest","real_name":null}]` |
| `options_sysoptions` 插入 `submission_list_show_all=false`**该行原本不存在** | `DELETE` | `bun db.ts show` → `[]` |
| 造了 1 个比赛 + 1 条榜单行 + 1 条公告(原本 0/0/0 | 全部删除 | `contests: 0 \| ranks: 0 \| announcements: 0` |
| 限流/F4 测试产生 26 条提交 | 按快照删除,并把 `problem` 与 `user_profile` 的 `submission_number`/`accepted_number`/`statistic_info`/`acm_problems_status` 恢复到快照值 | `submissions now: 8 \| snapshot had: 8` |
| `devadmin` 密码临时改成探针口令(为了验证 F4 的管理员视角) | 用保存的原 hash 写回 | `restored devadmin password hash, identical: true \| raw_password: devonly` |
redis 里的 `throttling:*` 也清了(`deleted throttling keys: 1`),免得本地开发一上来就撞限流。
- **临时脚本全在 `/tmp` 的 scratchpad**,仓库里没留(`git status` 干净,只有 3 个提交)。
- **没有 drop/truncate 任何表。**
---
## 疑虑
1. **`contestId` 仍无条件下发**。旧后端的 `SubmissionSafeModelSerializer` 是
`exclude = ("info", "contest", "ip")` —— 三个字段,新后端只脱敏了两个。
评审 F4 只点了 `info` 与 `ip`,我按 finding 的字面范围修,没顺手动 `contestId`。
泄露面比另外两个小得多(只是一个比赛 id且比赛可见性另有把关但**严格说没对齐旧后端**。
建议下一轮一并处理,或明确判定新后端就是要下发它。
2. **限流的 429 状态码是新引入的语义**。旧后端所有错误都走 `self.error()` → HTTP 200 +
`{"error": ..., "data": null}` 信封;新后端用真实状态码。前端目前**没有**针对 429 的处理,
学生撞到限流时会走通用错误提示(能看到 `Please wait N seconds` 文案,但没有专门的 UI
不阻塞,但前端接线时值得单独确认一下。
3. **限流按 user id 计,匿名不涉及** —— 因为 `POST /submissions` 挂了 `requireAuth`。
旧后端的 ip 桶(`capacity 100, fill_rate 0.1`)在 `SubmissionAPI` 里同样没用上,
所以这里是对齐的,不是漏搬。但配置项里保留了 ip 桶的默认值,将来要挂匿名端点可以直接用。
4. **判题机容器还在用旧 token**。F5 改的是「以后」的行为,当前跑着的
`oj2-judge` 容器是之前用 `oj2-dev-token` 起的,本次没有重启它(重启会打断验证)。
下次 `docker compose -f docker/compose.dev.yml up` 之前必须先按注释设好
`OJ2_JUDGE_TOKEN` 与 `JUDGE_SERVER_TOKEN`(两者取同一个值),否则 compose 会直接报错。
5. **F2 的下发点我数出 14 处,文档说 13 处**。差异见上文表格的说明(大概率是
`problem.ts:333` 本来就写死 `null` 没计入)。已用 grep 反证收口完整,
但如果评审方手里有一份逐条清单,值得对一遍确认不是我漏看了某个反方向的差异。

View File

@@ -1,338 +0,0 @@
# 阶段 3 权限与鉴权边界评审
日期2026-08-07
受审对象:`apps/api`Hono + Bunoj 侧 65 条端点
参照基准:`../OnlineJudge`Django + DRF只读
工作清单:`docs/specs/phase3-coverage.md` 的「oj 侧逐条对照」表
验证方式:逐条读源码 + 对本机 `http://localhost:3000` 实跑(测试账号 `e2etest`,普通学生)
---
## 1. 结论摘要
**3 条 Critical2 条 Important4 条 Minor。**
整体上权限骨架是**忠实重建**的:比赛的 `check_contest_permission`含密码保护、未开始拦截、contest admin 豁免、ProblemSafeSerializer 的字段脱敏)、提交的 `check_user_permission`、流程图的所有者判断、角色判定函数(`is_admin_role` / `is_teacher_or_above` / `is_contest_admin`)都与旧后端逐行等价,实跑验证也全部对上。**我事先怀疑的四处里有三处是误判**——`GET /contests/:id``GET /classes/:className/usernames``GET /users/:id/metrics` 在旧后端本来就是无鉴权的,新后端没有放宽;判题机心跳也确实做了 token 校验,只是校验用的密钥有问题。
真正的问题集中在**"序列化层的隐式权限"**:旧后端把权限判断藏在 DRF 序列化器的默认参数里(`UsernameSerializer(need_real_name=False)``UserProfileAPI` 开头的 `is_authenticated` 短路),新后端重写时只搬了装饰器、没搬序列化器里的这层,于是**真实姓名和邮箱被无差别下发给未登录访客**。学生是中职生(未成年人),真名 + 用户名 + 班级可以直接拼出花名册,这是本次最严重的问题。
另有一处逻辑翻转:`submission_list_show_all=false` 时,**未登录用户能拿到全站提交列表,而登录的学生拿到空列表**——匿名比登录更有权限。
---
## 2. 逐条对照表(只列有差异的行)
未列出的条目 = 新旧权限要求一致,见第 4 节。
| # | 旧端点 | 旧权限要求 | 新路由 | 新权限实现 | 判定 |
|---|---|---|---|---|---|
| 1 | `account/profile`GET | 视图首行 `if not user.is_authenticated: return success()` —— 等价 `@login_required` | `GET /profiles/:username` | `optionalAuth`,无任何 user 判断 | **更松Critical** |
| 2 | `account/user_rank` | 无装饰器,但 `RankInfoSerializer → UsernameSerializer(need_real_name=False)` 恒返回 `real_name: null` | `GET /rankings/users` | 无中间件,`realName: profile.realName` 无条件下发 | **更松Critical** |
| 3 | `submission/submissions` | 无装饰器;`not show_all and user.is_regular_user()` —— 匿名走到这行会 `AttributeError`Django `AnonymousUser` 无该方法),必定拿不到数据 | `GET /submissions` | `optionalAuth``isRegularUser(null) === false` → 守卫不生效,直接返回全量 | **更松Critical** |
| 4 | `announcement/announcement` | `AnnouncementSerializer.created_by = UsernameSerializer()``real_name: null` | `GET /announcements``GET /announcements/:id` | `createdBy.realName` 无条件下发content.ts:43、64 | **更松** |
| 5 | `contest/contests``contest/contest` | `ContestSerializer.created_by = UsernameSerializer()``null` | `GET /contests``GET /contests/:id` | `creator()` 返回 realNamecontest.ts:31-34 → :47 | **更松** |
| 6 | `problem/problem`(列表与详情) | `ProblemListSerializer.created_by = UsernameSerializer()``null` | `GET /problems``GET /problems/:displayId/similar` | `listItem()` 下发 realNameproblem.ts:86 | **更松** |
| 7 | `problem/contest/problem` | 同上 | `GET /contests/:id/problems``.../:displayId` | contest.ts:131、:177 下发 realName | **更松** |
| 8 | `problemset/problemset``/<id>``/<id>/problems` | `UsernameSerializer()``null` | `GET /problem-sets*` | problemset.ts:62、:169 下发 realName | **更松** |
| 9 | `problemset/<id>/users_progress` | `@teacher_admin_required` + `ProblemSetProgressSerializer.user = UsernameSerializer()`**即使教师也拿不到 real_name** | `GET /problem-sets/:id/user-progress` | `requireAuth` + `isTeacherOrAbove`(角色判断一致),但 problemset.ts:398 下发 realName | 角色一致,字段**更松** |
| 10 | `message/message`GET | `@login_required` + `sender = UsernameSerializer()``null` | `GET /messages` | `requireAuth`一致content.ts:85 下发 `sender.realName` | 角色一致,字段**更松** |
| 11 | `tutorial/tutorial` | 无装饰器;`created_by = UserSerializer()`**含 email**,无 real_name | `GET /tutorials/:id` | 无中间件;`createdBy.realName` 下发,不含 email | realName **更松** / email **更严** |
| 12 | `conf/judge_server_heartbeat/` | 无装饰器,校验 `sha256(SysOptions.judge_server_token)`env 缺失时回落到 `rand_str()`随机、fail-safe | `POST /judge-server/heartbeat` | 校验 `sha256(config.judgeServerToken)`env 缺失时回落到硬编码常量 `"oj2-dev-token"`fail-open | **更松Important** |
| 13 | `submission/submission`POST | `@login_required` + `TokenBucket` 用户级限流(`SysOptions.throttling` | `POST /submissions` | `requireAuth`**无限流** | **更松Important** |
| 14 | `conf/class_usernames` | 无装饰器,`classroom` 参数无格式校验 | `GET /classes/:className/usernames` | 无中间件,但加了 `^\d{3,4}$` 校验 | **更严** |
| 15 | 全部需登录端点 | 装饰器内 `if request.user.is_disabled: return error` | 所有路由 | `getUserByToken` 发现 `isDisabled` 直接删 session 并返回 null | **更严**(禁用即时踢线) |
---
## 3. Findings
### Critical
---
#### C1 — `GET /api/profiles/:username` 未登录即可读取任意用户完整档案(含邮箱)
**位置**`apps/api/src/routes/account.ts:101-108``optionalAuth`
**服务层**`apps/api/src/services/profile.ts:6-41`
**旧后端**`OnlineJudge/account/views/oj.py:38-41`
```python
async def get(self, request, **kwargs):
user = request.user
if not user.is_authenticated:
return self.success() # ← 匿名直接返回空,等价 @login_required
```
新后端只用了 `optionalAuth`handler 内部**没有任何** `c.get("user")` 判断,`getUserProfileById` 的第二参数只控制 `realName` 是否下发,其余字段(含 `email`)无条件返回。
**实跑证据**(无 cookie
```
$ curl -s http://localhost:3000/api/profiles/e2etest
{"data":{"id":2,"user":{"id":4,"username":"e2etest","email":"e2e@local.test",
"adminType":"Regular User","problemPermission":"None",
"createTime":"2026-08-07 07:19:10.789+00","lastLogin":"2026-08-07 07:25:35.98+00",
"openApi":false,"isDisabled":false,"className":null},
"realName":null,"acmProblemsStatus":{"problems":{"5":{"_id":"1004","status":0}}},
"avatar":"/public/avatar/default.png","blog":null,"mood":"probe",
"github":null,"school":null,"major":null,"language":null,
"acceptedNumber":1,"submissionNumber":2}}
```
**复现**`curl http://localhost:3000/api/profiles/<任意用户名>`,不带 Cookie。
**影响**:泄露邮箱、账号角色(`adminType` 可用于定位管理员账号做定向爆破)、最后登录时间、班级、全部做题记录。配合同样无鉴权的 `GET /classes/:className/usernames`(可枚举整班用户名,旧后端同样开放),可无凭据批量拉取全校师生的邮箱与角色。
---
#### C2 — 真实姓名(`realName`)被无条件序列化,未登录即可读取学生真名
**最严重实例**`apps/api/src/routes/account.ts:168``GET /rankings/users`,无任何中间件)
```ts
user: { id: user.id, username: user.username, realName: profile.realName },
```
**旧后端**`OnlineJudge/account/serializers.py:145-150` + `OnlineJudge/utils/api/_serializers.py:9-14`
```python
class RankInfoSerializer(serializers.ModelSerializer):
user = UsernameSerializer() # ← need_real_name 默认 False
class UsernameSerializer(serializers.Serializer):
def get_real_name(self, obj):
return obj.userprofile.real_name if self.need_real_name else None
```
旧后端把"是否下发真名"做成了序列化器的**默认关闭开关**,全仓库 11 处 `UsernameSerializer(...)` 调用里只有 1 处显式打开(`contest/serializers.py:84`,且条件是 `is_contest_admin`)。新后端没有对应机制,凡是 join 到 `user_profile` 的地方就直接把 `realName` 塞进响应。
**实跑证据**(先给 `e2etest` 写入真名,再无 cookie 请求,最后已还原为 NULL
```
$ curl -s "http://localhost:3000/api/rankings/users?limit=5"
{"data":{"results":[
{"id":2,"user":{"id":4,"username":"e2etest","realName":"张三(审计测试)"},
"acceptedNumber":1,"submissionNumber":2,"mood":"probe"},
{"id":1,"user":{"id":2,"username":"student","realName":"Phase 2 Student"},
"acceptedNumber":1,"submissionNumber":5,"mood":null}, ...]}}
```
`student` 这条是**本机既有数据**,没有经过我任何修改,其 `real_name = "Phase 2 Student"` 直接被匿名请求读到。
**同一根因的全部下发点**(旧后端对应字段一律为 `null`
| 文件:行 | 路由 | 是否匿名可达 |
|---|---|---|
| `routes/account.ts:168` | `GET /rankings/users` | **是**(学生真名,危害最大) |
| `routes/content.ts:43` | `GET /announcements` | 是 |
| `routes/content.ts:64` | `GET /announcements/:id` | 是 |
| `routes/content.ts:196` | `GET /tutorials/:id` | 是 |
| `routes/contest.ts:34``creator()`,经 :47 | `GET /contests``GET /contests/:id` | 是 |
| `routes/problem.ts:86``listItem()` | `GET /problems``GET /problems/:displayId/similar` | 是 |
| `routes/problemset.ts:62``problemSetCreator()` | `GET /problem-sets``GET /problem-sets/:id` | 是 |
| `routes/problemset.ts:169` | `GET /problem-sets/:id/problems` | 是 |
| `routes/contest.ts:131` | `GET /contests/:id/problems` | 需登录 + 过密码 |
| `routes/contest.ts:177` | `GET /contests/:id/problems/:displayId` | 需登录 + 过密码 |
| `routes/content.ts:85` | `GET /messages``sender.realName` | 需登录 |
| `routes/problemset.ts:398` | `GET /problem-sets/:id/user-progress` | 需教师 |
注:`routes/problem.ts:333``GET /problems/:displayId`)和 `routes/contest.ts:209`(比赛榜单,`admin ? realName : null`)是**正确**的两处,说明这套逻辑并非无人知晓,只是没有被统一执行。
**建议**:在 `sampleUserSchema` 层面把 `realName` 变成必须显式打开的字段(例如收敛成一个 `serializeUserRef(user, { realName: boolean })` 帮助函数),而不是在 13 个调用点分别记得置 null。
---
#### C3 — `submission_list_show_all=false` 时,未登录用户能读全站提交列表,登录学生反而读不到
**位置**`apps/api/src/routes/submission.ts:211`
```ts
if (!(await getBooleanOption("submission_list_show_all", true)) && isRegularUser(user)) {
return success(c, submissionListSchema.parse({ results: [], total: 0 }))
}
```
`isRegularUser` 定义于 `routes/helpers.ts:27-29``user?.adminType === "Regular User"`。**匿名用户 `user === null` → 返回 `false` → 守卫短路,全量下发。**
**旧后端**`OnlineJudge/submission/views/oj.py:149-151`
```python
show_all = await SysOptions.aget("submission_list_show_all")
if not show_all and request.user.is_regular_user():
return self.success({"results": [], "total": 0})
```
Django 的 `AnonymousUser` 没有 `is_regular_user` 方法(`account/models.py:53` 定义在自定义 `User` 上),匿名请求走到这行会抛 `AttributeError` → 500。旧后端在这个配置下**绝不可能**把列表交出去。
**实跑证据**(临时插入 `options_sysoptions` 行,测完已 DELETE 还原):
```
site.submissionListShowAll = false
ANON /submissions?limit=2 -> total=7 n=2 ← 未登录,拿到全部 7 条
REGULAR logged-in /submissions?limit=2 -> total=0 n=0 ← 登录学生,拿到 0 条
```
匿名响应含每条提交的 `username`、题号、语言、结果、耗时。
**复现**:把 `options_sysoptions``submission_list_show_all` 置为 `false`,然后不带 Cookie 请求 `GET /api/submissions?limit=10`
**影响**:该开关的唯一用途就是"考试期间不让学生互相看提交"。现在学生只要开一个隐私窗口(或直接删 Cookie就能绕过开关等于失效。
---
### Important
---
#### I1 — 判题机共享密钥的默认值硬编码在仓库里fail-open
**位置**`apps/api/src/config.ts:8`
```ts
judgeServerToken: process.env.JUDGE_SERVER_TOKEN ?? "oj2-dev-token",
```
`.env.example:4` 同样写着 `JUDGE_SERVER_TOKEN=oj2-dev-token`
**旧后端**`OnlineJudge/options/options.py:92-94`
```python
def default_token():
token = os.environ.get("JUDGE_SERVER_TOKEN")
return token if token else rand_str() # ← 没配就随机fail-safe
```
生产 compose`OnlineJudge/docker-compose.yml:44,65`)显式注入 `_Mam^^1rvC86Qko2d0`。也就是说旧后端**漏配环境变量的后果是判题机连不上(立刻暴露)**,新后端**漏配的后果是任何人都能通过心跳校验(静默)**。
**实跑证据**
```
no-token -> 403 {"error":{"code":"invalid-judge-token",...}} ← 校验存在,符合旧行为
default-dev-token -> 200 {"error":null,"data":null} ← 用仓库里的常量即通过
```
(本机确实跑在默认 token 下。测试插入的 `hostname='pwned-by-audit'` 行已删除。)
**当前实际影响有限**`judge/run.ts:69` 派发判题用的是 `config.judgeServerUrl`(环境变量),**不读** DB 里的 `service_url`,所以伪造心跳无法把判题流量劫走,只能污染 `judge_server` 表 / 管理端仪表盘。但这是运气好,不是设计使然——`service_url` 字段被写进了库judge-server.ts:63一旦将来 admin 侧或调度器改成读它,同一个洞就升级为 Critical。
**建议**`JUDGE_SERVER_TOKEN` 缺失时启动即 fail或生成随机值并打日志不要留可猜的默认值。
---
#### I2 — `POST /submissions` 丢失了提交限流
**位置**`apps/api/src/routes/submission.ts:52`(只有 `requireAuth`
**旧后端**`OnlineJudge/submission/views/oj.py:35-42``:68-70`
```python
def throttling(self, request):
user_bucket = TokenBucket(key=str(request.user.id), redis_conn=cache,
**SysOptions.throttling["user"])
can_consume, wait = user_bucket.consume()
if not can_consume:
return "Please wait %d seconds" % (int(wait))
```
旧配置 `{"capacity": 20, "fill_rate": 0.03}` ≈ 每用户约 2 提交/分钟。新后端没有任何等价物,任一登录学生可以无限速向 `judgeQueue` 灌任务。这不是读权限问题,但它是旧后端里唯一挡住"一个学生打爆判题沙箱"的机制,重写时整体丢失了,归入本报告以免遗漏。
---
### Minor
**M1 — `isAdminRole` 从白名单退化为黑名单**
`routes/helpers.ts:31-33``Boolean(user && user.adminType !== "Regular User")`
旧:`account/models.py:65-70` 显式列举 `[STUDENT_ADMIN, TEACHER_ADMIN, SUPER_ADMIN]`
当前四种角色下两者等价,但将来新增任何角色(如"助教""家长")都会**默认获得 admin 权限**,包括 `canViewSubmission` 里的"看所有人代码"。建议改回白名单。
**M2 — 比赛权限判断没有中间件兜底**
`canAccessContest``contest.ts:116/144/193``submission.ts:63/253` 共 5 处手工调用。旧后端用 `@check_contest_permission` 装饰器,漏挂会很显眼;新后端漏调一次就是静默放行,且 `GET /contests/:id/problems` 这类路由挂的是 `optionalAuth`(本身不拦人),从中间件列表上完全看不出它受保护。目前 5 处都调对了,属可维护性风险。
**M3 — `blog` / `github` 从 URLField 降级为自由字符串**
`packages/contract/src/account.ts:15,17``z.string().max(256)`;旧 `account/serializers.py:125,127``serializers.URLField`。现在 `PUT /me/profile` 可以写入 `javascript:alert(1)` 之类的值,是否可利用取决于前端如何渲染这两个字段。
**M4 — `GET /dev/problems` 仍在线且无鉴权**
`routes/problem.ts:231`。内容本身是公开题目摘要,无实际泄露,但它是阶段 1 的临时端点,`phase3-coverage.md` 第 120 行已标注应删除,至今还挂在生产路由树上。
**(文档纠错,非安全问题)** `phase3-coverage.md:96``PUT submissions/:id` 描述为"判题结果写回"。实际实现(`submission.ts:301-316`)是**提交分享开关**,对应旧 `SubmissionAPI.put` + `ShareSubmissionSerializer`。判题结果写回走的是内部 worker不经 HTTP。这条描述会让人误以为存在一个需要判题机凭据的写入端点建议改正。
---
## 4. 核实过、确认没有问题的项
以下是逐条读过源码(多数还实跑过)、确认**新后端权限要求与旧后端等价或更严**的部分。
**四个预先怀疑点里的三个是误判:**
- `GET /contests/:id`contest.ts:78——旧 `ContestAPI.get``contest/views/oj.py:38-51`)本来就**没有任何装饰器**,公开可读。新后端行为一致,且 `serializeContest` 不下发 `password`(旧 `ContestSerializer``exclude`)。**真正受保护的比赛内容(题目、榜单、提交)全部走 `canAccessContest`,实跑验证如下:**
```
ANON /contests/5 200 ← 与旧后端一致
ANON /contests/5/problems 401 login-required
ANON /contests/5/rank 401 login-required
ANON /contests/5/submissions 401 login-required
ANON /contests/5/access 401 login-required
AUTHnopw /contests/5/problems 403 wrong-password ← 登录但没过密码,正确拦截
AUTHnopw /contests/5/rank 403 wrong-password
```
(测试用的密码保护比赛 id=5 已删除。)
- `GET /classes/:className/usernames`site.ts:36——旧 `ClassUsernamesAPI``conf/views.py:237-243`)无装饰器,公开。新后端一致,且**多加了** `^\d{3,4}$` 格式校验。属旧后端遗留的开放面,不是本次重写引入的(但确实是 C1 的放大器,值得单独排期收口)。
- `GET /users/:id/metrics`account.ts:140——旧 `Metrics``account/views/oj.py:66-83`)无装饰器,按 `?userid=` 任取,也不限制只查自己。新后端一致。同上,属遗留开放面。
- `POST /judge-server/heartbeat`——token 校验**存在**且用了 `timingSafeEqual`,比旧后端的裸字符串比较更好;问题只在默认值(见 I1
**比赛权限(最复杂的一块,逐条对齐):**
- `canAccessContest``services/contest.ts:45-63`)与 `check_contest_permission._check_access``account/decorators.py:122-136`)逐行等价:未登录 → login-requiredcontest admin 豁免;密码保护校验 session 内存的密码;`check_type != "details"` 时拦截未开始的比赛。
- `checkContestPassword``services/contest.ts:28-37`)完整复刻了 `sig#timestamp` 形式(`decorators.py:87-110`),含 sha256 前 8 位与过期判断。
- `contestDetailsAllowed``services/contest.ts:24-26`= `Contest.problem_details_permission``contest/models.py:42-43`)。
- 比赛未结束时对非管理员脱敏的字段集完全一致:`difficulty`、`submissionNumber`、`acceptedNumber`、`statisticInfo` 被置空 —— 正是旧 `ProblemSafeSerializer``problem/serializers.py:211-228`exclude 的那几个(`answers` 新后端从不下发)。
- 比赛榜单 `realName` 正确地由 `admin ? realName : null` 控制contest.ts:209对齐 `ACMContestRankSerializer(is_contest_admin=...)`。
- `POST /submissions` 的比赛分支完整保留了"比赛已结束禁止提交"和 `allowedIpRanges` 白名单submission.ts:65-68 vs `submission/views/oj.py:47-53``ipAllowed`services/contest.ts:71-85的 CIDR 计算正确,空列表放行、无 IP 拒绝。
- `GET/POST /contests/:id/access` 均要求登录且比赛必须设了密码,与旧 `ContestAccessAPI` / `ContestPasswordVerifyAPI`(均 `@login_required`)一致。
**提交与流程图的所有者判断:**
- `canViewSubmission`submission.ts:166-178与 `Submission.check_user_permission``submission/models.py:47-56`)等价,且新增了 `if (!user) return false` —— 更严。
- `check_share=False` 语义(用于分享开关和 `canUnshare`)正确传递。
- `GET /submissions/:id` 用 `requireAuth`,无权限时返回 404 而非 403不泄露存在性 —— 比旧的 "No permission for this submission" 更严。
- `PUT /submissions/:id`(分享开关)保留了"比赛进行中不得分享"。
- 流程图 `canView`flowchart.ts:24-26= `FlowchartSubmission.check_user_permission``flowchart/models.py:59-63`);列表的 `myself` / `username` / 普通用户默认只看自己 三段逻辑与 `flowchart/views/oj.py:78-83` 一致;`current` / `history` 强制 `userId = 自己`。
**角色与身份:**
- `isRegularUser` / `isAdminRole` / `isTeacherOrAbove` / `isSuperAdmin`helpers.ts:27-41与 `account/models.py:53-73` 语义一致(`isAdminRole` 的黑名单写法见 M1
- `isContestAdmin`services/contest.ts:20-22= `User.is_contest_admin``account/models.py:78-79`)。
- 禁用账号:`getUserByToken`auth/session.ts:105-108发现 `isDisabled` 立即删除 session 并返回 null比旧后端"每个装饰器里各判一次"更彻底。
- Session token 用 `randomBytes(32)` + RedisCookie `httpOnly` + `SameSite=Lax`,登出正确删 Redis key 与 Cookie。
**逐条确认权限一致的端点:**
- `POST /auth/login`、`DELETE /auth/session`、`POST /users`(注册)、`GET /me` —— 均与旧后端一致(`GET /me` 匿名返回 null对应旧 `UserProfileAPI` 匿名返回空)。注册同样受 `allow_register` 控制;`rawPassword` 明文留存是旧 `User.set_password``account/models.py:81-83`)就有的行为,非新增。
- `PUT /me/profile` 可写字段集与旧 `EditUserProfileSerializer` **完全相同**8 个字段,无 `acceptedNumber` 等统计字段),无字段注入面。
- `POST /me/avatar`、`POST /me/problem-display-ids/refresh`、`GET /problems/:displayId/rank`、`GET /me/class-rank` —— `requireAuth`,且只操作 `c.get("user")!.id`,无越权参数。
- 成就四条achievement.ts:39/75/103/111—— 均 `requireAuth``?username=` 允许查他人,**与旧 `_resolve_user``achievement/views/oj.py:10-15``?name=`)完全一致**,不是新放宽;`pending` / `pending/read` 强制本人。隐藏成就的掩码逻辑也保留了(不下发 metric/threshold/progress
- AI 九条 —— 全部 `requireAuth``targetUser`ai.ts:45-59正确复刻了"仅 teacher_or_above 才能用 `?username=` 查他人"`ai/views/oj.py:239-244``/ai/class-pk-analysis` 保留 `isTeacherOrAbove`(对齐 `@teacher_admin_required``/ai/hint` 强制提交必须属于本人,且系统提示词明确禁止透露参考答案。
- `GET /messages` 只查 `recipientId = 自己``POST /messages` handler 内 `isSuperAdmin` 判断,对齐 `@super_admin_required`。
- 表情 `GET/POST /problems/:id/reaction` —— `requireAuth`,且 POST 要求该题有 AC 记录才能表态。
- 题单:`/problem-sets*` 全部过滤 `visible=true AND status != 'draft'``POST/PUT /problem-set-progress` 只改自己的进度,且 PUT 校验 submission 归属本人 + 已 AC + 题目确在题单内;`/problem-sets/:id/user-progress` 角色判断与 `@teacher_admin_required` 一致。
- 公开内容(`GET /announcements*`、`/tutorials*`、`/problem-tags`、`/problems/random`、`/problem-authors`、`/problems/:displayId/yearly-ac`、`/problem-sets/:id/badges`、`/site`、`/quotes/random`、`/submissions/today-count`、`/rankings/activity`、`/rankings/classes`、`/classes/comparison`)—— 旧后端对应视图**同样无装饰器**,一致。`/problem-authors?all=1` 能看到不可见题目的作者名,旧 `ProblemAuthorAPI``problem/views/oj.py:248`)行为相同。
- 公告 / 教程 / 题单 / 题目查询一律带 `visible` / `is_public` / `status != draft` 过滤,无隐藏内容泄露。
- WebSocket `/ws/submissions`index.ts:71-83、websocket.ts—— 升级前强制校验 session订阅 topic 按 `userId` 隔离;`subscribe` 消息查库时带 `eq(userId, ws.data.userId)`,无法订阅他人提交;推送前二次校验账号未禁用。**无越权面。**
- 头像静态服务index.ts:53-68对路径做了 basename 校验,无目录穿越。
**Hono 路由注册顺序**已核:`/problems/random`、`/problems/:id/beat-count`、`/submissions/today-count` 等具体路径均注册在同前缀的 `:param` 路由之前,不存在鉴权路由被无鉴权通配路由抢先匹配的情况。
---
## 5. 测试期间对本机数据库的改动(均已还原)
| 操作 | 还原情况 |
|---|---|
| `user_profile.real_name` 设为 `'张三(审计测试)'`user_id=4、`'李老师(审计)'`author | 已置回 NULL / 原值 |
| `options_sysoptions` 插入 `submission_list_show_all=false` | 已 DELETE该键原本不存在 |
| `judge_server` 插入 `hostname='pwned-by-audit'` | 已 DELETE |
| `contest` 插入密码保护比赛id=5 | 已 DELETE验证过 0 行 |
未执行任何 `DROP` / `TRUNCATE`;未修改 `OnlineJudge/`、`ojnext/` 或 OJ2 的任何源码;未改动 git 状态。

View File

@@ -1,583 +0,0 @@
# 阶段 3 评审:响应数据泄露
日期2026-08-07
受审对象:`apps/api/src`Hono + Drizzleoj 侧 65 条端点)
参照基准:`OnlineJudge/<app>/serializers.py` 的 DRF serializer 白名单(只读,未改动)
方法:静态扫描 + 实跑取真实响应逐字段比对
---
## 1. 结论摘要
**Critical0 条。Important3 条。Minor3 条。**
最重要的一条是 **`GET /api/profiles/:username` 允许匿名访问并返回 `user.email`**
`routes/account.ts:101` + `services/profile.ts:19`)。旧后端 `UserProfileAPI.get`
第一件事就是 `if not user.is_authenticated: return self.success()`,匿名拿不到任何东西;
新后端挂的是 `optionalAuth`,不带任何 cookie 就能按用户名遍历全校学生邮箱。
好消息(**已逐条实证,不是推断**
- **`user.raw_password`(明文密码列)从未出现在任何 oj 侧响应里。** 全量响应体里
grep `Test123456` / `student123` / `devonly` / `Probe123456` —— **0 命中**
- `password` / `auth_token` / `open_api_appkey` / `session_keys` 这四个字段名
**在全部响应里一次都没出现过**(对 111 次请求的响应做了递归字段名收集后统一 grep
- `problem.answers`(库里存的是**完整参考解代码**C + Python`test_case_id`
`test_case_score``ast_rules` **全部没有出现在题目相关响应里**
- `problem.template` 只返回 `//TEMPLATE BEGIN/END` 区间,`//PREPEND` / `//APPEND`
隐藏区被正确剥掉(与旧 `parse_problem_template` 行为一致)。
- `contest.password` **没有泄露**`GET /api/contests` 那句 `db.select()`(选全部列)
后面接的是显式字段映射password 只被用来算 `contestType`
- 他人代码:`GET /api/submissions/:id` 的权限判定与旧 `check_user_permission` 等价,
未 share 且题目未开 `share_submission` 时返回 404。
也就是说:**旧后端 serializer 挡住的"泄题类"字段,新后端一条都没漏。**
本次发现的问题集中在 **PII邮箱、真实姓名****判题细节(`info` / `ip`** 两类。
---
## 2. 扫描方法与覆盖面
### 2.1 静态扫描
`apps/api/src` 下检索四种"选全部列"写法:
| 写法 | 命中数 | 结论 |
|---|---|---|
| `db.select()` 后不跟对象字面量 | 11 处 | 见第 4 节,**全部无泄露** |
| `db.query.xxx.findMany/findFirst``columns:` | 0 处 | 新后端完全没用 relational query API |
| `...row` / `...user` 整行展开进响应 | 1 处 | `problem.ts:228`,展开的是聚合查询结果(`total`/`accepted`),非表行 |
| `c.json(row)` 直接丢数据库行 | 0 处 | 全部响应走 `success(c, ...)` + Zod `.parse()` |
额外检查 `packages/contract`**没有任何 `.passthrough()` / `z.looseObject` / `.catchall()`**
Zod v4 object 默认 strip 未知键,因此"多选了列"本身不会自动变成泄露 ——
只有被显式写进 `.parse({...})` 入参的字段才会出去。这是新后端的第二道防线,实测有效。
`z.record(z.string(), z.unknown())` 有 16 处(`statisticInfo``exercise.data`
`flowchartData``sqlConfig``acmProblemsStatus``submissionInfo`
这些是 jsonb 原样透传,是唯一绕过 Zod 裁剪的通道。逐个核对过,见第 4/5 节。
### 2.2 动态扫描
API 跑在 `http://localhost:3000`。用三个身份 + 匿名共四种视角:
| 身份 | 说明 |
|---|---|
| `e2etest`user 4 | 普通学生,主视角 |
| `leakprobe2`user 5 | 现场用 `POST /api/users` 注册的第二个学生,验"看他人数据" |
| `student`user 2 | 已有学生,作为"被看的人"(有提交、有 `real_name` |
| 匿名(不带 cookie | 验未登录可见面 |
因为库里 `contest` / `problemset` / `tutorial` / `exercise` / `announcement` /
`message` / `achievement` / `flowchart_submission` / `acm_contest_rank` 都是**空表**
先造了带标记值的测试数据再打:
- 带密码的比赛(`password = 'SUPERSECRET-CONTEST-PW'`+ 一道比赛题
- 题单 + 题单题(`hint = 'HINT-SECRET'`+ 徽章 + 进度
- 教程 + 练习(`data.answer` / `data.explanation = 'SECRET-ANSWER-EXPLANATION'`
- 隐藏成就(`name` / `description` 带标记)
- 他人的两条提交:一条 `shared = true`、一条 `shared = false``code` 带标记
- 题目 `template` 塞进 `//PREPEND BEGIN\nSECRET-PREPEND\n//PREPEND END` 等隐藏区
- 题目 `ast_rules` / `answers` 塞标记值
脚本对每个响应**递归收集全部字段路径**(含数组元素、最深 9 层),
再对字段名做敏感词匹配,同时对响应原文做标记值 grep这一条能抓到"字段名没问题但值是别人的"的情况)。
**覆盖111 次请求65 条 oj 路由中 61 条拿到了 2xx 响应。**
未取得 2xx 的 4 条及原因:
| 路由 | 状态 | 原因 |
|---|---|---|
| `POST /api/judge-server/heartbeat` | 403 | 需要 judge server token非 oj 学生面 |
| `POST /api/messages` | 403 | 仅 super admin 可调(与旧后端一致) |
| `POST /api/flowcharts` | 400 | 需要真实流程图 payload`GET` 系列已覆盖同一序列化路径 |
| `POST /api/code/format` | 500 | 本机没装 `ruff` / `clang-format`;响应体无数据字段 |
原始响应留档:
`/tmp/claude-1000/-home-xuyue-Projects-OJ/ab9e12e4-.../scratchpad/{probe2,probe3,probe4}.txt`
### 2.3 环境还原
评审过程中造的数据**已全部清理**,改过的既有行已还原(见第 6 节的一条例外,需要你确认)。
临时脚本已从仓库删除,`git status` 干净。**未修改 OJ2 任何业务代码,未动 OnlineJudge / ojnext。**
---
## 3. Findings
### Important
---
#### I-1. `GET /api/profiles/:username` 匿名可访问,且返回 `user.email`
**位置**`apps/api/src/routes/account.ts:101``apps/api/src/services/profile.ts:19`
```ts
// account.ts:101
accountRoutes.get("/profiles/:username", optionalAuth, async (c) => {
```
```ts
// services/profile.ts:19
email: row.user.email,
```
**旧后端**`account/views/oj.py:36``UserProfileAPI.get`
```python
async def get(self, request, **kwargs):
user = request.user
if not user.is_authenticated:
return self.success() # ← 匿名直接返回空
```
**实际响应****完全不带 cookie**
```
$ GET /api/profiles/student (no cookie)
[200]
{"data":{"id":1,"user":{"id":2,"username":"student",
"email":"student@example.test", ← 学生邮箱,匿名可见
"adminType":"Regular User","problemPermission":"None",
"createTime":"2026-08-07 04:27:24.688+00",
"lastLogin":"2026-08-07 07:29:46.337+00", ← 上次登录时间,匿名可见
"openApi":false,"isDisabled":false,"className":null},
"realName":null,"acmProblemsStatus":{...},"avatar":"/public/avatar/default.png",
"mood":null,...,"acceptedNumber":1,"submissionNumber":5}}
```
**影响**:用户名在本站是公开的(`GET /api/rankings/users``GET /api/classes/:className/usernames`
都匿名可拿全量用户名),所以这等于**任何人都能离线遍历出全校学生的邮箱 + 上次登录时间**。
中职学生的邮箱多为学号/姓名拼音派生,属实名可关联数据。
**注意区分**:登录用户看他人 email 是**旧后端也有的行为**(旧 `UserSerializer.Meta.fields`
里就有 `email``UserProfileSerializer` 用的就是它)。**回归点只在"匿名也能看"**。
修的时候只需把 `optionalAuth` 换成 `requireAuth` 即可对齐旧行为;
如果想顺手收紧"登录用户看他人 email",那是超出对齐范围的改进,需另行决定。
---
#### I-2. `realName`(学生真实姓名)在 8 处 `createdBy` / `user` 内嵌里未做门控
**位置**(每处都是把 `schema.userProfile.realName` 原样塞进响应):
| 文件:行 | 端点 | 谁的真名 |
|---|---|---|
| `routes/account.ts:168` | `GET /api/rankings/users` | **学生本人**(匿名可见) |
| `routes/content.ts:85` | `GET /api/messages` | 消息发送者 |
| `routes/problemset.ts:398` | `GET /api/problem-sets/:id/user-progress` | **学生本人**(教师端) |
| `routes/problem.ts:86` | `GET /api/problems``GET /api/problems/:displayId` | 出题人 |
| `routes/content.ts:43` | `GET /api/announcements` | 公告作者 |
| `routes/content.ts:64` | `GET /api/announcements/:id` | 公告作者 |
| `routes/content.ts:196` | `GET /api/tutorials/:id` | 教程作者 |
| `routes/problemset.ts:169` | `GET /api/problem-sets/:id/problems` | 出题人 |
| `routes/contest.ts:131``contest.ts:177` | `GET /api/contests/:id/problems(/:displayId)` | 出题人 |
**旧后端**`utils/api/_serializers.py:4`
```python
class UsernameSerializer(serializers.Serializer):
id = serializers.IntegerField()
username = serializers.CharField()
real_name = serializers.SerializerMethodField()
def __init__(self, *args, **kwargs):
self.need_real_name = kwargs.pop("need_real_name", False) # ← 默认 False
super().__init__(*args, **kwargs)
def get_real_name(self, obj):
return obj.userprofile.real_name if self.need_real_name else None
```
旧后端**只有比赛管理员**这一个调用点传 `need_real_name=True`
`contest/serializers.py``ACMContestRankSerializer.get_user`)。
排行榜走的 `RankInfoSerializer` 用的是裸 `UsernameSerializer()``real_name` 恒为 `null`
**实际响应****完全不带 cookie**;测试前给 user 4 设了 `real_name = 'REALNAME-SECRET-E2E'`
```
$ GET /api/rankings/users?limit=20 (no cookie)
[200]
{"data":{"results":[
{"id":2,"user":{"id":4,"username":"e2etest",
"realName":"REALNAME-SECRET-E2E"}, ← 真名,匿名可见
"acceptedNumber":1,"submissionNumber":2,"mood":"probe"},
{"id":1,"user":{"id":2,"username":"student",
"realName":"Phase 2 Student"}, ← 真名,匿名可见
"acceptedNumber":1,"submissionNumber":5,"mood":null},
...],"total":5}}
```
```
$ GET /api/messages (cookie: e2etest)
[200]
{"data":{"results":[{"id":1,
"sender":{"id":2,"username":"student","realName":"Phase 2 Student"}, ← 旧后端此处为 null
...}]}}
```
**影响**`GET /api/rankings/users` 这一条最严重 —— **匿名 + 一个 GET 就能拿到
"用户名 ↔ 真实姓名"的全量映射表**,把原本半匿名的排行榜变成了实名榜。
中职学生属未成年人,这是实打实的 PII 外泄面扩大。
**做对了的反例**(说明这不是"整体没设计",而是漏了):
`routes/contest.ts:209` 是**唯一**做了门控的地方,与旧后端语义一致:
```ts
user: { id: user.id, username: user.username, realName: admin ? realName : null },
```
`routes/problem.ts:333``routes/account.ts:94``routes/contest.ts:34`
`routes/problemset.ts:62` 硬编码 `realName: null`,也是对的。
所以修复方式很清楚:把上表 8 处对齐成 `contest.ts:209` 那种写法。
---
#### I-3. 本人提交详情返回 `info`(逐测试点判题明细)与 `ip`,旧后端对所有非管理员都隐藏
**位置**`apps/api/src/routes/submission.ts:187``:195``:199`
```ts
// submission.ts:187
const full = isAdminRole(user) || row.submission.userId === user.id
// ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ 旧后端没有这个分支
return submissionDetailSchema.parse({
...
info: full ? row.submission.info : {}, // :195
ip: full ? row.submission.ip : null, // :199
```
**旧后端**`submission/views/oj.py:96`
```python
if request.user.is_admin_role():
submission_data = await self.async_serialize_data(SubmissionModelSerializer, submission)
else:
submission_data = await self.async_serialize_data(SubmissionSafeModelSerializer, submission)
```
`SubmissionSafeModelSerializer``submission/serializers.py`
```python
class Meta:
model = Submission
exclude = ("info", "contest", "ip")
```
判定条件是 **`is_admin_role()`**,不是"是不是自己的提交"。
所以旧后端下,学生看**自己的**提交也拿不到 `info``ip`
**实际响应**`student` 看自己的提交):
```
$ GET /api/submissions/LEAKPROBE-SUB-PRIVATE (cookie: student本人)
[200]
{"data":{"id":"LEAKPROBE-SUB-PRIVATE","userId":2,"username":"student",
"code":"...","result":0,
"info":{"err":null,"data":[
{"error":0,"memory":7991296,"output":null,"result":0,"signal":0,
"cpu_time":3,"exit_code":0,"real_time":6,
"test_case":"1", ← 测试点编号
"output_md5":"caf1a3dfb505ffed0d024130f58c5cfa"}, ← 输出指纹
{..."test_case":"2","output_md5":"248e844336797ec98478f85e7626de4a"},
{..."test_case":"3","output_md5":"ab233b682ec355648e7891e66c54191b"},
{..."test_case":"4","output_md5":"cfee398643cbc3dc5eefc89334cacdc1"},
{..."test_case":"5","output_md5":"bc6dc48b743dc5d013b1abaebd2faed2"}]},
"ip":"10.9.8.7", ← 旧后端不返回
...}}
```
**影响**:学生能看到本题**测试点总数、每个测试点的编号、单点耗时/内存/退出码,
以及自己输出的 md5**。这不等于直接泄题(`output` 字段是空的,且 md5 是学生自己程序的输出,
不是标准答案),但它把判题内部结构暴露给了学生:
反复提交 + 比对 `output_md5` 可以**逐测试点二分定位哪一个点挂了**
这正是旧后端把 `info` 关掉想避免的事。
**分级说明**:按你给的口径,"测试点/答案泄露给学生 → Critical"。
我定 Important 而非 Critical理由是泄露的是**测试点元信息 + 学生自己输出的指纹**
不含标准答案、不含测试点输入输出内容,也不含 `test_case_id`(拿不到测试数据目录)。
但它确实越过了旧白名单,且方向是"学生本可以不知道的判题内部"
所以我建议按 Critical 的优先级排期修,只是不按 Critical 定级。
**已做对的部分**:看**他人**提交时 `full` 为 false`info` 返回 `{}``ip` 返回 `null` ——
实测确认(见下),这部分与旧后端一致。
```
$ GET /api/submissions/LEAKPROBE-SUB-SHARED (cookie: leakprobe2他人)
[200]
{"data":{...,"userId":2,"username":"student",
"code":"SECRET-CODE-OF-OTHER-USER-SHARED", ← shared=true旧后端同样返回属预期
"info":{},"ip":null, ← 正确置空
"shared":true,...}}
```
---
### Minor
---
#### M-1. `GET /api/messages` 的 `submission` 内嵌多出 `info` / `ip` 两个字段名(值恒为空)
**位置**`apps/api/src/routes/content.ts:96``:100`
```ts
info: {}, // content.ts:96
ip: null, // content.ts:100
```
`MessageSerializer.submission` 用的是 `SubmissionSafeModelSerializer`
`info` / `contest` / `ip` 三个键**根本不出现**。新后端硬编码成空值后仍把键留在响应里。
**实际响应**
```
$ GET /api/messages (cookie: e2etest)
{"data":{"results":[{"id":1,"sender":{...},"message":"LEAKPROBE msg",
"submission":{"id":"2e16...","userId":2,"username":"student",
"code":"n = int(input())\nprint(...)",
"info":{}, ← 键存在,值为空
"ip":null, ← 键存在,值为空
...}}],"total":1}}
```
**无数据泄露**(值恒为空,写死在代码里,不读数据库)。仅为白名单形状不一致。
之所以还是列出来,是因为它和 I-3 共用 `submissionDetailSchema` ——
如果将来有人"顺手"把这里改成传真实值,就会变成真泄露。
`submission.code` 出现在这里是**旧后端也有的**:旧 `SubmissionSafeModelSerializer`
`exclude` 不含 `code`。消息由 super admin 发出,指向收件人自己的提交,属预期。)
---
#### M-2. `GET /api/problem-sets/:id/user-progress` 返回学生 `realName`
**位置**`apps/api/src/routes/problemset.ts:398`
`ProblemSetProgressSerializer.user = UsernameSerializer()``problemset/serializers.py:248`
`real_name` 恒为 `null`。新后端返回真实值。
单列为 Minor 而不并入 I-2是因为**该端点已由 `isTeacherOrAbove(user)` 门控**
`problemset.ts``if (!isTeacherOrAbove(user)) return failure(c, 403, ...)`
实测普通学生调用返回 403受众只有教师。给教师看学生真名大概率是**产品上想要的**。
所以:**先确认这是不是有意为之**,如果是,就别跟着 I-2 一起改,
否则会把教师端的功能删掉。
---
#### M-3. `GET /api/tutorials/:id/exercises` 返回练习答案 —— 与旧后端一致,**不是回归**
**位置**`apps/api/src/routes/content.ts:207`
```ts
const rows = await db.select().from(schema.exercise).where(...)
return success(c, rows.map((row) => exerciseSchema.parse({
id: row.id, type: row.type, data: objectValue(row.data), order: row.order,
})))
```
`data` 是 jsonb 原样透传(`contract/src/content.ts:72`
`z.record(z.string(), z.unknown())`,不裁剪内容)。
**实际响应****匿名也能拿**
```
$ GET /api/tutorials/1/exercises (no cookie)
[200]
{"data":[{"id":1,"type":"mcq","data":{
"answer":1, ← 正确答案
"options":["1","2"],"question":"1+1?",
"explanation":"SECRET-ANSWER-EXPLANATION"}, ← 答案解析
"order":1}]}
```
**旧后端完全一样**`tutorial/serializers.py`
```python
class ExerciseSerializer(serializers.ModelSerializer):
class Meta:
model = Exercise
fields = ["id", "type", "data", "order"] # data 整个 jsonb 出去
```
且旧 `ExerciseAPI``tutorial/views/oj.py`)也没有 `@login_required`
**结论:行为一致,不算本次重写引入的问题。** 之所以写进报告,是因为这是教程练习的
**前端判题设计**(答案下发到浏览器、客户端比对),学生 F12 就能看到答案。
迁移不需要动它,但如果哪天想改判题方式,这是已知的口子 —— 属于遗留设计债,不是回归。
---
## 4. 静态检查结果11 处"选全部列"逐条核实
`db.select()`(不跟对象字面量,等价 `SELECT *`)全部命中如下。
**结论11 处全部无泄露** —— 每一处的查询结果都只是中间变量,
真正进响应的是后面手写的显式字段映射 + Zod `.parse()`
| # | 文件:行 | 选了哪张表 | 表里有敏感列吗 | 是否泄露 | 依据 |
|---|---|---|---|---|---|
| 1 | `routes/contest.ts:70` | `contest` | **有 `password`** | **否** | 结果传给 `serializeContest()``contest.ts:37`),只挑 11 个字段;`password` 仅用于 `contestType: contest.password ? "Password Protected" : "Public"``:49`)。实测 `GET /api/contests` 响应无 `password` 键 |
| 2 | `routes/content.ts:207` | `exercise` | `data` 含答案 | 否(但见 M-3 | 只取 `id/type/data/order`,与旧 `ExerciseSerializer` 字段完全一致 |
| 3 | `routes/problemset.ts:85` | `problemset_progress` | 无 | 否 | 结果只用于取 `completedProblemsCount` / 算 `progressSummary()` |
| 4 | `routes/problemset.ts:86` | `problemset_badge` | 无 | 否 | 结果传给 `badgeData()``problemset.ts:66`),显式挑 8 个字段 |
| 5 | `routes/problemset.ts:123` | `problemset` | 无 | 否 | 传给 `serializeProblemSet()`;旧 `ProblemSetListSerializer` 也含 `visible`,属对齐 |
| 6 | `routes/problemset.ts:133` | `problemset` | 无 | 否 | 同上 |
| 7 | `routes/problemset.ts:361` | `problemset_badge` | 无 | 否 | 传给 `badgeData()` |
| 8 | `routes/achievement.ts:43` | `achievement` | 隐藏成就的 `name`/`description`/`threshold` | **否** | `achievement.ts` 里按 `masked = achievement.hidden && !record``name` 换成 `"???"``description` 换成 `"达成条件保密"``metric`/`operator`/`threshold`/`progress` 全置 `null` |
| 9 | `routes/achievement.ts:44` | `user_achievement` | 无 | 否 | 只用来建 `Map` 判断是否已解锁 |
| 10 | `services/contest.ts:40` | `contest` | **有 `password`** | **否** | `findVisibleContest()` 是内部 helper返回值不直接进响应调用方`contest.ts:79/85/97/114`)都再走 `serializeContest()` |
| 11 | `services/achievements.ts:27` | `achievement` | 同 #8 | 否 | 后台解锁判定用,不进任何 HTTP 响应 |
**#8 的实证**(隐藏成就掩码生效):
```
$ GET /api/achievements (cookie: e2etest)
{"data":{"username":"e2etest","achievements":[
{"id":1,"name":"LEAKPROBE ach","description":"d",...,"unlocked":true,...},
{"id":2,"name":"???","description":"达成条件保密", ← 掩码生效
"icon":"noto:red-question-mark","rarity":"gold","hidden":true,
"metric":null,"operator":null,"threshold":null, ← 达成条件全部置空
"unlocked":false,"progress":null,"unlockRate":0}]}}
```
(造数据时该成就的真实 `name``LEAKPROBE hidden SECRETNAME`
`description``SECRET-CONDITION`,两个标记值在响应里 grep 均为 0 命中。)
**#1 / #10 的实证**(比赛密码不泄露):
```
$ GET /api/contests?limit=50 (no cookie)
{"data":{"results":[
{"id":3,"title":"LEAKPROBE contest","description":"desc","tag":"probe",
"startTime":"...","endTime":"...","createTime":"...","lastUpdateTime":"...",
"createdBy":{"id":1,"username":"devadmin","realName":null},
"status":"0","contestType":"Password Protected"}, ← 只暴露"有没有密码"
{"id":4,...,"contestType":"Public"}],"total":2}}
```
真实密码 `SUPERSECRET-CONTEST-PW``GET /api/contests``GET /api/contests/:id`
`GET /api/contests/:id/access``POST /api/contests/:id/access`(成功与失败两种)
五个响应里 grep 均为 **0 命中**。未通过密码校验时 `/problems``/rank``/submissions`
全部 403通过后才放行 —— 与旧后端 `@check_contest_permission` 行为一致。
**唯一的整行展开**`routes/problem.ts:228`
```ts
return success(c, rows.map((row) => yearlyAcSchema.parse({ ...row, acRate: ... })))
```
`rows` 来自 `db.select({ year: ..., total: count(), accepted: ... })` 的显式聚合投影,
不是表行,`...row` 只有 3 个数值键。`yearlyAcSchema` 又会 strip 掉多余键。**无泄露。**
---
## 5. 核实过、确认没问题的端点
以下端点的**每一个字段**都比对过旧 serializer 白名单,无越界、无敏感值。
**题目类**(旧基准:`ProblemListSerializer` / `ProblemSerializer` / `ProblemSafeSerializer`
- `GET /api/problems``GET /api/problems/:displayId`
—— 无 `answers` / `astRules` / `testCaseId` / `testCaseScore``hasAstRules` 只给布尔值。
实测拿真实题目 `1002`(其 `answers` 列存着完整 C + Python 参考解)验证:
响应里 grep `#include<stdio.h>` / `scanf(` / `printf(` / `//PREPEND` / `//APPEND`
**全部 0 命中**`template` 只有 `//TEMPLATE` 区间的内容。
顺带一提:旧 `ProblemSerializer.Meta.exclude` **没有**排掉 `ast_rules`
也就是旧后端其实会把 AST 规则发给学生 —— **新后端这里比旧的更严,是改进,不是问题**
- `GET /api/problems/random``/similar``/yearly-ac``/rank``/:id/beat-count``/:id/reaction`
- `GET /api/problem-tags``GET /api/problem-authors`
- `GET /api/contests/:id/problems``GET /api/contests/:id/problems/:displayId`
—— 比赛进行中正确按 `ProblemSafeSerializer` 语义把 `difficulty``""`
`submissionNumber` / `acceptedNumber``0``statisticInfo``{}`
**提交类**(旧基准:`SubmissionListSerializer`
- `GET /api/submissions``GET /api/submissions/today-count`
—— 列表无 `code` / `info` / `ip`,与旧 `exclude = ("info", "contest", "code", "ip")` 一致。
- `GET /api/submissions/:id` 的**权限判定**:未 share 且题目未开 `share_submission` → 404
匿名 → 401开了 `share_submission``shared=true` → 放行。与旧 `check_user_permission` 等价。
- `GET /api/contests/:contestId/submissions`
**比赛类**`GET /api/contests``/:id``/:id/access``/:id/rank``POST /:id/access`
`/:id/rank``realName` 已正确门控,见 I-2 反例)
**题单类**`GET /api/problem-sets``/:id``/:id/problems``/:id/badges`
`/:id/user-progress`(见 M-2`GET /api/users/:username/badges`
`POST|PUT /api/problem-set-progress`
**成就类**`GET /api/achievements``/summary``/pending``POST /achievements/pending/read`
**班级/排行类**`GET /api/rankings/activity``/rankings/classes``GET /api/me/class-rank`
`GET /api/classes/:className/usernames``POST /api/classes/comparison`
—— 只返回用户名和聚合统计,无 `realName` / `email`。匿名可访问,
但旧 `ClassRankAPI` / `ClassPKAPI` 同样没有 `@login_required`**属对齐**。
**AI 类**`GET /api/ai/detail``/duration``/heatmap``/login-summary``/pinned`
—— 另外确认:带 `?username=student` 请求 `/ai/detail`,响应里的 `"user"` 仍是 `"e2etest"`
说明 `targetUser()` 对普通学生忽略了该参数,**没有越权看他人 AI 报告**。
**内容类**`GET /api/announcements``/:id``GET /api/tutorials``/:id`
`GET /api/messages`(见 I-2 / M-1
**流程图类**`GET /api/flowcharts``/:id``GET /api/problems/:id/flowchart/current``/history`
—— 他人流程图返回 404`leakprobe2` 实测)。
**账号/站点类**
- `POST /api/auth/login``{"data":{"ok":true}}`(不回显用户信息)
- `POST /api/users`(注册)→ `{"data":{"ok":true}}`
- `DELETE /api/auth/session``POST /api/me/avatar``POST /api/me/problem-display-ids/refresh`
- `GET /api/me` → 返回自己的 `email` / `realName`**正常**(旧后端 `show_real_name=True`
- `PUT /api/me/profile` → 同 `GET /api/me`**正常**(旧 `ProfileAPI.put` 也是 `show_real_name=True`
- `GET /api/site` → 只有 `websiteBaseUrl` / `websiteName` / `websiteNameShortcut` /
`websiteFooter` / `allowRegister` / `submissionListShowAll` / `classList` / `enableMaxkb`
无 SMTP、无 judge token
- `GET /api/quotes/random``GET /api/users/:id/metrics`
### 顺带提一句:新后端比旧后端裁剪更严的地方
按你的要求,这些不算 finding但列出来供你确认前端会不会缺字段
1. **`problem.ast_rules`**:旧 `ProblemSerializer` 没排掉它,学生能拿到 AST 规则原文;
新后端只给 `hasAstRules: boolean`。**新的更安全**,但如果 ojnext 有地方读
`ast_rules` 的具体内容(比如提示"必须用 for 循环"),需要补个专门的字段。
2. **`GET /api/tutorials/:id``createdBy`**:旧 `TutorialSerializer` 用的是
`UserSerializer`**会带 `email`**;新后端只给 `{id, username, realName}`
**新的更安全**,前端应该用不到教程作者的邮箱。
---
## 6. 需要你确认的一件事(环境影响)
评审造数据时,脚本的 `select * from problem order by id limit 1` 选中了
**题目 id=2`_id` 1001"🐟三天打鱼两天晒网"**,并覆写了它的
`ast_rules``answers``test_case_score``template` 四列,用来验证这些字段会不会外泄。
**原值没有备份,已丢失。**
清理时按同类题目的形状还原成了:
```
template = {} (与覆写前实测值一致,这个是准的)
ast_rules = null (id 3/4/5 等题目均为 null)
answers = []
test_case_score = [{"score":20,"input_name":"1.in","output_name":"1.out"}, ... 共 5 项]
(照抄 id=4/5 的形状;该题 test_case_id 目录下无 info 文件可查真实点数)
```
其余改动**已完整还原**`problem 5.share_submission``false`
user 2/4 的 `class_name``null`、user 4 的 `real_name``null`
所有 `submission.ip``null`。造的行(比赛/题单/教程/练习/公告/成就/消息/
流程图/比赛排名/两条 LEAKPROBE 提交/三个 leakprobe 账号)**已全部删除**
最终库状态20 道题、8 条提交、3 个用户devadmin / student / e2etest、其余业务表为空。
`answers``test_case_score` 影响的是**评测与题解**,不影响本次结论。
但**题目 1001 是学生做的第一题**,如果这台机器上的数据之后还要用,
建议从生产重新导一次该题,或至少确认 1001 的测试点数确实是 5。
---
## 7. 修复优先级建议
| 优先级 | Finding | 一句话改法 |
|---|---|---|
| 1 | I-1 | `routes/account.ts:101``optionalAuth``requireAuth` |
| 2 | I-2 | 上表 8 处按 `contest.ts:209` 的写法补门控;`rankings/users` 最急 |
| 3 | I-3 | `submission.ts:187``full` 去掉 `\|\| row.submission.userId === user.id`,只留 `isAdminRole(user)` |
| 4 | M-1 | 给 `submissionDetailSchema` 拆一个不含 `info` / `ip` 的 safe 变体,`content.ts` 的 messages 用它 |
| 5 | M-2 | 先确认是不是产品有意为之,是就不动 |
| — | M-3 | 与旧后端一致,本次迁移不动 |

View File

@@ -1,153 +0,0 @@
# 阶段 3 修复复评
日期2026-08-07
复评范围commit `9c04b00..f548aef`4 个提交),对照 `docs/specs/phase3-fix-list.md` 的 6 条 findings + 2 条收尾修复F4b、F5b
方法:读 diff + 读源码 + 独立实跑(不复用实施者的证据,除非明确说明)。所有测试数据均已清理并核对还原。
---
## 逐条核验
### F1 —— 匿名可读任意用户完整档案【Critical】→ **ADDRESSED**
`apps/api/src/routes/account.ts:101``optionalAuth` 之后立即 `if (!c.get("user")) return success(c, null)`,早于任何数据库查询。
独立实跑:
```
匿名 GET /api/profiles/e2etest -> 200 {"data":null}
```
`services/profile.ts``showRealName` 参数链路未动,登录看自己档案不受影响(未重复验证,逻辑未改,风险低)。
前端 `apps/web/src/shared/api.ts:28` 已有 `data === null` 分支,兼容确认(读码确认,未跑前端)。
### F2 —— 学生真名无条件下发【Critical】→ **ADDRESSED**(默认关闭开关,非逐处删字段)
见下方「重点判断 1」结论14 个下发点全部经过统一的 `sampleUser()`,默认 `realName: null`,唯一显式打开的是 `contest.ts:209`(比赛榜单,且区分 admin/非 admin
独立实跑(与 diff 里的证据不同的端点,避免只是复读实施者的话):
```
匿名 GET /api/rankings/users?limit=2 -> [{"username":"student","realName":null},{"username":"e2etest","realName":null}]
匿名 GET /api/problems?limit=2 createdBy -> {"id":1,"username":"devadmin","realName":null} (x2)
匿名 GET /api/problems/1002 createdBy -> {"id":1,"username":"devadmin","realName":null}
```
`grep -rn "realName" apps/api/src --include=*.ts` 复查:除 `sampleUser` 内部实现、`db.select()` 取列、`account.ts:94`(注册写库常量 null`services/profile.ts:29`F1 已核实的独立开关)外,没有任何路由手写 `{ ..., realName }` 对象绕过 `sampleUser()`
### F3 —— 匿名绕过提交可见性守卫【Critical】→ **ADDRESSED**
`submission.ts:227``!(await getBooleanOption(...)) && !isAdminRole(user)``isAdminRole(null)` 返回 `false``!false = true`,匿名正确落入受限分支。
独立实跑(临时把 `submission_list_show_all` 写为 `false`,测后 DELETE 该行,确认恢复为 `[]`
```
匿名 GET /api/submissions -> total = 0
登录学生 GET /api/submissions -> total = 0
```
两者一致地受限F3 描述的「匿名权限大于登录用户」的错位已消除。
### F4 —— 提交详情返回 info/ip【Important】→ **ADDRESSED**
`submission.ts:199``const full = isAdminRole(user)`,不再以 `row.submission.userId === user.id` 放行。`/submissions/:id``requireAuth`,匿名根本到不了这里,不存在新的匿名向量。
### F4b —— contestId 未脱敏 → **ADDRESSED已用真实比赛提交独立验证**(原报告只有代码审查,本次补上实跑)
`submission.ts:214``contestId: full ? row.submission.contestId : null`
独立实跑(造了一场临时比赛 + 一条挂在该比赛下的真实提交,验证后已删除,见下方清理记录):
```
DB 实际存储:{"contestId":7,"ip":"9.9.9.9"}
提交所有者 (Regular User) GET /api/submissions/<id> ->
{"info":{},"ip":null,"contestId":null, ...} ← 三个字段全部脱敏
```
`contestId` 确实是被脱敏成 `null`,而不是碰巧本来就是 `null`(数据库里明确存的是 `7`)。控制方文档里点名"没验充分"的这一条已经补齐。
### F5 —— 判题机 token 弱默认值【Important】→ **ADDRESSED**
`config.ts``judgeServerToken()`env 存在则用 env不存在则 `randomBytes(32)` + `console.warn``docker/compose.dev.yml``${OJ2_JUDGE_TOKEN:?...}` 缺失时 compose 直接报错退出(未重新实跑 compose读码 + 报告证据一致,逻辑简单,风险低)。
### F5b —— 仓库根 .env 读不到 → **ADDRESSED加载器安全**
见下方「重点判断 3」独立做了三组隔离测试shell env 优先级、cwd .env 优先级、缺文件不崩、畸形行不崩),并额外做了一次端到端真实提交(问题 1002Python3确认判题机用当前 `.env` 里的 token 正常认证、提交离开 PENDING/JUDGING拿到 result=-2非 PENDING/JUDGING证明判题机-后端握手成功)。测试提交与计数器已回滚,见清理记录。
### F6 —— 提交接口缺限流【Important】→ **ADDRESSED参数与旧后端逐字对齐独立验证**
见下方「重点判断 4」。独立绕过 HTTP 层直接调用 `consumeToken()` 12 次(避免污染 submission 计数):
```
#1~#10 allowed:true
#11 allowed:false, wait≈33.32s
#12 allowed:false, wait≈33.32s
```
`wait = (1 - 0) / 0.03 ≈ 33.33`,与旧后端 `fill_rate=0.03` 精确吻合,`default_capacity=10` 也吻合(第 11 次才被挡)。
---
## 四个重点判断
### 1. F2 是否真的做成了「默认关闭的开关」?
**是。** `apps/api/src/routes/helpers.ts``sampleUser()` 是唯一入口,`options.includeRealName === true` 才下发真名,默认 `false`。核对了全仓 14 个下发点(比 findings 文档的 13 处多一个,多出的是 `problem.ts:333`,原先就写死 `null`,现在统一走 `sampleUser` 入口,不影响结论):
| 下发点 | 状态 |
|---|---|
| rankings/users、announcements(x2)、messages、tutorials/:id、problems 列表(x2)、problem-sets(x2)、problemset progress、contest creator(x2)、contest problems(x2) | 全部 `sampleUser(user, realName)`,默认关 |
| `contest.ts:209`(比赛榜单) | 唯一 `{ includeRealName: admin }`,对齐旧后端 `contest/serializers.py:84` |
`grep -rn "realName:" apps/api/src/routes/*.ts` 复查无遗漏(本次独立复跑,非照抄报告)。**不是逐处删字段**——`sampleUser()` 是统一的序列化函数,下次新增端点如果照抄现有写法(调用 `sampleUser`)默认就是关的,不会重犯。唯一的隐患是「有人手写字面量绕过 `sampleUser`diff 里已经全部清干净,长期靠代码评审维持(无法用类型系统强制,值得记一条范围外观察)。
### 2. `isRegularUser` 是否真的只有一处调用?删除后是否有其它同类空值陷阱?
**只有一处,属实。** `grep -rn "isRegularUser" --include=*.ts .`(排除 node_modules覆盖整个仓库而非只有 apps/、packages/)只命中 `helpers.ts` 里的警示注释,无任何遗留调用点。
**搜了其它同类模式**(对 `null` 用户取 `adminType` 做权限判断),命中 `apps/api/src/routes/flowchart.ts:98`
```ts
if (c.req.query("myself") === "1" || (!username && user.adminType === "Regular User")) ...
```
这处**不是**同类陷阱:该路由挂在 `flowchartRoutes.get("/flowcharts", requireAuth, ...)``user``c.get("user")!` 取得,`requireAuth` 保证非空匿名到不了这行。这是「Regular User 专属限制」的合法写法(限制普通用户只看自己的,不限制教师/管理员),语义与 F3 的场景不同——F3 的路由是 `optionalAuth`,匿名 `user` 可能为 `null`。**未在 diff 内,非本次改动引入,仅作范围外观察记录**,不计入 findings。
其余 `adminType` 使用点(`classroom.ts``account.ts``contest.ts``inArray`/`sql` 过滤,`services/contest.ts``isContestAdmin`)均为「构造 SQL 过滤条件」或「非 null 保证下的角色判断」,没有第二个「匿名反而权限更大」的实例。
### 3. F5b 的 `.env` 加载器是否安全?
**安全,三点分别独立验证:**
- **真实 env 优先于仓库根 `.env`**`JUDGE_SERVER_TOKEN=from-shell-env bun -e '...'` 得到 `from-shell-env`,不是 `.env` 里的值。
- **cwd 下的 `.env`Bun 自动加载)优先于仓库根 `.env`**`bun --env-file=.env.test-cwd` 模拟 cwd 优先加载后,`loadRepoRootEnv()` 因为 `process.env[key] !== undefined` 而跳过,结果仍是 cwd 的值。
- **解析不会被畸形行搞崩**:构造了空值(`=novalue`)、无等号行、前导空格键、单/双引号、值里带等号、`export FOO=bar` 语法、行内 `#` 注释、CRLF 结尾等混合样本喂给等价解析逻辑,全部正常跳过或按字面处理,无异常抛出。(`export FOO=bar` 会被解析成键名 `"export FOO"`,即该变量实际上不会被正确加载——是一个小的解析局限,不是崩溃,本仓库的 `.env`/`.env.example` 都不用这种写法,记为范围外观察。)
- **根目录无 `.env` 时静默跳过**`readFileSync``ENOENT``catch {}` 吞掉,不影响启动——这是生产场景(真实环境变量注入)的必经路径,逻辑上有覆盖(未在容器里额外验证,风险低,纯 try/catch 结构)。
- **端到端**:仓库根 `.env` 当前有真实 `JUDGE_SERVER_TOKEN`,直接提交一条真实代码(问题 1002/Python3判题机在数秒内返回非 PENDING/JUDGING 结果,证明 token 握手成功、判题闭环工作,不是只停留在配置读取层面。
### 4. 限流参数是否真的对齐旧后端?
**是,非拍脑袋。** 对照 `OnlineJudge/options/options.py:120`
```python
throttling = {"ip": {"capacity": 100, "fill_rate": 0.1, "default_capacity": 50},
"user": {"capacity": 20, "fill_rate": 0.03, "default_capacity": 10}}
```
`apps/api/src/services/throttling.ts``throttlingDefaults` 逐字段相同。
**挂点位置**对照 `OnlineJudge/submission/views/oj.py:68``SubmissionAPI.post``throttling()``check_contest_permission`(比赛权限校验)之后、取 `Problem` 之前调用——`apps/api/src/routes/submission.ts` 的挂点(比赛权限校验后、`db.select(schema.problem)` 前)位置一致。
**独立数值验证**:绕开 HTTP 直接调用 `consumeToken()` 12 次,第 11 次起被拒,`wait≈33.32s`,与 `(1 token 缺口) / (fill_rate=0.03) ≈ 33.33s` 吻合,`default_capacity=10` 与观察到的"第 11 次才拒绝"一致。参数和算法都对得上,不是抄了个数字但算法跑偏。
Lua 脚本把旧实现「非线程安全」的读改写做成了原子操作,这是双方都认可的合理增强,不是风险点。)
---
## 修复 diff 内新引入的破坏
**无。**
- `bunx tsc --noEmit` 独立重跑0 错误。
- 独立冒烟测试 `/api/contests``/api/problems``/api/problem-sets``/api/rankings/users``/api/announcements``/api/submissions``/api/problems/1002` 等端点,均 200`sampleUser()` 各调用点(含 `row ?? { id, username: "" }` 的 null 分支、`{ id: row.creatorId, username: row.creatorUsername }` 的裸对象分支)未见运行时异常。
- `apps/web` 零改动diff stat 确认F1/F4 对前端的潜在影响(`data:null` 分支、`info.data` 表格不渲染)均有既存代码兜底或本就是回归到旧后端行为,读码确认不炸。
- F4b 的 `contestId` 收口用真实比赛提交实测通过,未见回归。
## 范围外观察(仅记录,不阻塞)
1. `apps/api/src/routes/flowchart.ts:98` 有一处外观相似的 `user.adminType === "Regular User"` 判断,但路由挂 `requireAuth`,不构成 F3 类陷阱。建议后续如果这条路由改成 `optionalAuth`,需要一并检查。
2. `.env` 解析器不支持 `export KEY=value` 语法(会把 `export KEY` 当整个键名),本仓库当前 `.env`/`.env.example` 不用这种写法,暂无影响。
3. F2 的「默认关闭」防线目前只靠约定(大家都调用 `sampleUser()`),没有类型系统强制。长期看这是 admin 侧 45 个端点铺开前值得补一道 lint/测试的地方,但不属于本次 findings。
4. 限流的 429 是新引入的 HTTP 语义(旧后端全走 200+error 信封),前端目前没有针对 429 的专门处理(报告里已自述,非新发现)。
## 结论
**All findings addressed: Yes**
F1 / F2 / F3 / F4 / F4b / F5 / F5b / F6 —— 8 项全部 ADDRESSED均有独立实跑或读码验证支持diff 范围内未发现新引入的破坏。

View File

@@ -1,496 +0,0 @@
# 阶段 4 后台接口权限边界评审
评审对象:`apps/api/src/routes/admin/*.ts`10 个文件、86 个 handler+ 3 个挂在 `/admin` 之外但对应旧后端 admin 视图的端点。
参照物:`OnlineJudge/` 各 app 的 `views/admin.py`,装饰器定义在 `account/decorators.py`
评审范围:**只看角色守卫与归属校验**,不评审风格 / 性能 / 可维护性。
- 评审人:独立安全评审
- 日期2026-08-07
- 实跑环境:`http://localhost:3000`postgres 5433 / redis 6380
- 实跑账号:`student`(id=2)、`e2etest`(id=4),测试期间临时提权,**结束后已全部还原为 Regular User / None**;造的 6 道题、3 场比赛、2 个题单、1 个奖章、1 个标签已全部删除,`problem` 表回到 20 行
---
> **修复记录2026-08-07本文档之后**
>
> C1、C2、I1、I2、I3、I4 **六条全部已修**,并用两个教师账号(互相不可见的题单/比赛)
> 实跑复验通过:跨题单删奖章 → 404 且对方奖章还在;跨租户 make-public → 404
> from-public 拖别人的赛题 → 400 `not-a-public-problem`;克隆比赛回传 `password: null`
> 分析接口移到 `/admin/problem-analytics/*` 后不再被 `/problems/:id` 遮蔽;
> upload-image 守卫收回 `requireAdmin`。
>
> Minor 四条未修M1 已按「有意偏离」保留M2/M3/M4 留待阶段 5。
> **更正2026-08-08本报告有一处对旧后端的判断是错的。**
>
> 文末对照表原写「`DashboardInfoAPI` / `RandomUsernameAPI` 旧后端任何人可读,
> 含班级用户名枚举」。**不成立** —— 这两条都挂在 `/api/admin/` 下,而
> `account/middleware.py:36` 的 `AdminRoleRequiredMiddleware` 在中间件层就要求
> 登录且 `is_admin_role()`。「无装饰器」是真的,「任何人可读」不是。
>
> 差别很大:真实缺口只是「任何管理员可读,而非仅超管」。查旧后端的权限时,
> 别只看装饰器,那个中间件是所有 `/api/admin/` 的地板。
## 结论速览
| 级别 | 数量 | 条目 |
|---|---|---|
| Critical | 2 | C1 跨题单删奖章会连带删掉别人的 user_badgeC2 make-public 无归属校验且回传完整题面 |
| Important | 4 | I1 两个 requireTeacher 端点被 `/problems/:id` 路由遮蔽I2 from-public 不校验源题归属I3 克隆比赛不校验归属且回传明文密码I4 upload-image 守卫比旧后端严过头,教师/学生管理员写题面时会 403 |
| Minor | 4 | M1 visibility 的归属规则与 canEdit 不一致M2 from-public 的错误码构成比赛存在性预言机M3 三个 admin 端点的守卫写在 handler 里而非注册行M4 禁用账号落到 401 而不是「账号已禁用」 |
守卫覆盖:**86 个后台 handler 全部挂了守卫,没有一个漏挂**。角色档位与旧后端装饰器逐条比对后全部一致(见文末对照表)。问题全部集中在**对象级归属校验**这一层。
---
## Critical
### C1 — 跨题单删奖章:奖章保住了,但别人的 user_badge 被真删了
**文件**`apps/api/src/routes/admin/problemset.ts:365-378`
```ts
const deleted = await db.transaction(async (tx) => {
await tx.delete(schema.userBadge).where(eq(schema.userBadge.badgeId, badgeId)) // ← 先删,没带题单条件
return tx.delete(schema.problemsetBadge).where(and(
eq(schema.problemsetBadge.id, badgeId),
eq(schema.problemsetBadge.problemsetId, row.id), // ← 后校验
)).returning({ id: schema.problemsetBadge.id })
})
if (deleted.length === 0) return failure(c, 404, "badge-not-found", "奖章不存在")
```
父子归属校验放在了子表清理**之后**,而且 `if (deleted.length === 0)` 的 404 是在事务**提交之后**才返回的 —— 回调正常结束,事务照常 COMMIT。于是「奖章没删成」和「奖章的获得记录已经删光了」同时成立。
这正是任务里点名要查的场景:**A 题单的 id + B 题单的奖章 id**。
**旧后端对应行为**`problemset/views/admin.py:287-301`,先 `ProblemSetBadge.objects.get(id=badge_id, problemset=problem_set)`,取不到直接 `return self.error("奖章不存在")``badge.delete()` 根本不会执行user_badge 一行不动。
**实跑验证**
前置:`student`(A) 与 `e2etest`(B) 均为 Teacher Admin/Own。A 建题单 2B 建题单 3 并在其下建奖章 2。手工插入两行 user_badge
```
docker exec oj2-postgres psql ... -tAc "insert into user_badge (user_id,badge_id,earned_time) values (4,2,now()),(2,2,now()); select id,user_id,badge_id from user_badge"
INSERT 0 2
2|4|2
3|2|2
```
A只拥有题单 2打 B 的奖章 2
```
DELETE /api/admin/problem-sets/2/badges/2 (cookie = A)
→ 404 {"error":{"code":"badge-not-found","message":"奖章不存在"}}
```
看起来被挡住了。查库:
```
select count(*) from user_badge; → 0
select id,problemset_id,name from problemset_badge; → 2|3|REVIEW-B-BADGE
```
奖章还在,**两行 user_badge 全没了**。
对照:同一路由的 PUT 是安全的(校验和写在同一条 `and()` 里):
```
PUT /api/admin/problem-sets/2/badges/2 body={name:"HACKED",...} (cookie = A)
→ 404 badge-not-found
GET /api/admin/problem-sets/3/badges (cookie = B)
→ 200 [{"id":2,"name":"REVIEW-B-BADGE","conditionValue":1,...}] # 未被篡改
```
**风险场景**:教师 A 在自己题单页面点删除奖章时,前端只要拼错 badgeId或 A 手工构造请求),就能把教师 B 班上全体学生已获得的奖章记录一次性抹掉,而 A 自己看到的是「奖章不存在」——**破坏是静默的,没有任何一方会收到提示**。奖章记录没有别处备份,`recalculateBadge` 只在改奖章时触发B 不改奖章就永远恢复不了 earnedTime。
**修法**:把 `userBadge` 的清理条件收进子查询,或把归属校验提到事务最前面并在不匹配时 `throw` 触发回滚。
---
### C2 — `make-public` 完全不校验归属,且把整份题面回传给越权者
**文件**`apps/api/src/routes/admin/problem.ts:491-533`
```ts
adminProblemRoutes.post("/problems/:id/make-public", requireProblemPermission, async (c) => {
...
const [problem] = await db.select().from(schema.problem).where(eq(schema.problem.id, id)).limit(1)
if (!problem) return failure(c, 404, "problem-not-found", "Problem does not exist")
if (!problem.contestId || problem.isPublic) { ... }
// ↑ 这里之后直接开事务,没有 canEdit / ownedBy
...
return success(c, await serialize(created), 201) // ← 回传完整题面
})
```
同一文件里 `GET /problems/:id`(275)、`PUT`(323)、`DELETE`(371)、`/test-cases`(602)、`/sql-scripts`(628) 全都过了 `canEdit`,唯独这一条没有。
**旧后端对应行为**`problem/views/admin.py:456-483``MakeContestProblemPublicAPIView` 同样没有 `ensure_created_by` —— **这个洞是从旧后端继承来的**。但旧后端第 483 行是 `return self.success()`**不回传任何数据**;转出来的公开题 `created_by` 仍是原作者且 `visible=False`,越权者拿不到内容。新后端第 532 行 `return success(c, await serialize(created), 201)` 把 description、samples、testCaseId、**answers标准答案** 一起吐回去,**泄露面严格大于旧后端**。
**实跑验证**
A 建比赛 3A 所有)并在其中建比赛题 38。B 直接读该题:
```
GET /api/admin/problems/38 (cookie = B)
→ 404 {"error":{"code":"problem-not-found","message":"Problem does not exist"}} # 归属校验生效
```
B 换成 make-public
```
POST /api/admin/problems/38/make-public body={"displayId":"STOLEN1"} (cookie = B)
→ 201 {"data":{"id":39,"_id":"STOLEN1","title":"REVIEW A","description":"<p>d</p>",
"inputDescription":"i","outputDescription":"o","samples":[{"input":"1","output":"1"}],
"testCaseId":"reviewdummy00000000000000000000",
"testCaseScore":[{"score":100,"input_name":"1.in","output_name":"1.out"}],
"hint":"","languages":["Python3"],"template":...
```
同一个人、同一道题,读接口 404、写接口 201 并把题面全给了。副作用还有两条A 的比赛题被改成 `isPublic=true`A 之后再想正常转公开会被 409 "Already be a public problem" 挡住),以及库里多出一道 A 名下的公开题。
**风险场景**:比赛开始前,任意持有 problem_permission 的管理员(含 Student Admin遍历 problem id 就能把别人未开赛比赛的题面连同标准答案全部拖走。中职学校场景下这就是考前泄题。
**修法**:在 496 行之后补 `if (!(await canEdit(user, problem)))` → 404顺带把 `problem.contestId` 分支的成功响应改成不回传题面(或至少 omit `answers`)。
---
## Important
### I1 — `/problems/stuck` 与 `/problems/ac-trend` 被 `/problems/:id` 遮蔽,两个 requireTeacher 端点实际不可达
**文件**`apps/api/src/routes/admin/tag.ts:190``tag.ts:215`(被 `apps/api/src/routes/admin/problem.ts:275` 遮蔽);成因在 `apps/api/src/routes/admin/index.ts:30-32``adminProblemRoutes` 先于 `adminTagRoutes` 注册)
Hono 按注册顺序匹配,`GET /problems/:id``:id` 会吃掉 `stuck` / `ac-trend` 这两个字面段。于是:
- 生效的守卫是 `requireProblemPermission`problem.ts:275**不是** tag.ts 上写的 `requireTeacher`
- handler 也是 problem.ts 的,`queryInteger("stuck", 0, {min:1})` 回落到 0查不到题 → 404
- tag.ts:190 / 215 的两个 handler **一次都没被执行过**
**旧后端对应行为**`problem/views/admin.py:638-640``StuckProblemsAPI.get` `@teacher_admin_required`)与 `675-677``TopACTrendAPI.get` `@teacher_admin_required`),两条独立 URL正常可达。
**实跑验证**(用 problem_permission 做判别,因为两个守卫的档位不同)
`student` = Teacher Admin + `problem_permission='None'`。按 tag.ts 写的 `requireTeacher` 应当放行;按 problem.ts 的 `requireProblemPermission` 应当 403
```
GET /api/admin/problems/stuck → 403 {"error":{"code":"permission-denied","message":"权限不足"}}
GET /api/admin/problems/ac-trend → 403 {"error":{"code":"permission-denied","message":"权限不足"}}
GET /api/admin/contests → 200 {"data":{"results":[],"total":0}} # requireTeacher 确实放行
```
判定:跑的是 `requireProblemPermission`,即 problem.ts:275。
把权限补回 `Own`、再提到 Super Admin两条依旧是 problem.ts 的错误体:
```
GET /api/admin/problems/stuck (Teacher/Own) → 404 {"code":"problem-not-found","message":"Problem does not exist"}
GET /api/admin/problems/ac-trend (Teacher/Own) → 404 {"code":"problem-not-found","message":"Problem does not exist"}
GET /api/admin/problems/stuck (Super Admin) → 404 {"code":"problem-not-found","message":"Problem does not exist"}
```
**风险场景**:当前只是「两个教师功能死了 + 挂的守卫不是实际生效的守卫」。真正的隐患在于**注册行上写的守卫和实际执行的守卫不一致**——`index.ts:15-21` 的注释明确说「不在总入口兜一层,就是为了让守卫从注册行上看得出来」,这条遮蔽正好打破了那个前提。将来若有人放宽 `GET /problems/:id`,会连带把两个 teacher-only 的统计接口一起放宽,而 tag.ts 上的 `requireTeacher` 看着还是对的。
**修法**:把 tag.ts 的两条静态路由挪到 problem.ts 的 `/problems/:id` 之前注册(调整 index.ts 顺序,或把它们改成 `/problem-stats/stuck` 这类不冲突的路径)。
---
### I2 — `from-public` 只校验目标比赛归属,不校验源题归属,可把别人的比赛题拖进自己的比赛
**文件**`apps/api/src/routes/admin/problem.ts:535-580`
```ts
const [contest] = await db.select().from(schema.contest).where(eq(schema.contest.id, contestId)).limit(1)
const [problem] = await db.select().from(schema.problem)
.where(eq(schema.problem.id, parsed.data.problemId)).limit(1) // ← 任意 id无过滤
if (!contest || !problem) return failure(c, 404, "not-found", ...)
if (user.adminType !== "Super Admin" && contest.createdById !== user.id) { // ← 只校验比赛
return failure(c, 404, "contest-not-found", "Contest does not exist")
}
```
源题既不校验 `createdById`,也不校验 `visible`,更不限制 `contestId is null`(尽管路由叫 from-**public**)。拷贝出来的题落在调用者自己的比赛里,之后 `canEdit` 走「比赛题看比赛创建者」分支 → 调用者对它有完全读写权。
**旧后端对应行为**`problem/views/admin.py:486-511``AddContestProblemAPI.post` —— **一个装饰器都没有,也没有任何 ensure_created_by**,比新后端还宽(连管理员身份都不要求)。新后端补上了 `requireProblemPermission` + 比赛归属,是净收紧;但对象级的源题归属仍然缺失。
**实跑验证**
A 的公开题 37A 所有、A 的比赛题 38在 A 的比赛 3 里。B 建自己的比赛 5
```
POST /api/admin/contests/5/problems/from-public {"problemId":37,"displayId":"X1"} (cookie = B)
→ 201 {"data":{"id":40,"_id":"X1","title":"REVIEW REVIEWA1","description":"<p>d</p>",...}}
POST /api/admin/contests/5/problems/from-public {"problemId":38,"displayId":"X2"} (cookie = B)
→ 201 {"data":{"id":41,"_id":"X2","title":"REVIEW A","description":"<p>d</p>",
"testCaseId":"reviewdummy00000000000000000000",...}}
GET /api/admin/problems/41 (cookie = B)
→ 200 {"data":{"id":41,"_id":"X2","title":"REVIEW A",...}} # 复制品 B 完全可读可改
```
问题 38 是 A 的**比赛题**B 直读它是 404见 C2 实跑),经这条路径拷一份就拿到了。
**风险场景**:与 C2 同源 —— 考前拖走别人比赛的题面与标准答案,只是入口换成了「往自己比赛里加题」。
**修法**:源题至少要求 `contestId is null`(名副其实的 public理想情况再叠一层 `canEdit(user, problem) || problem.visible`
---
### I3 — 克隆比赛不校验归属,且响应里回传原比赛的明文密码
**文件**`apps/api/src/routes/admin/contest.ts:172-233``password: original.contest.password` 在 192 行,`serialize` 下发 `password` 在 49 行)
```ts
adminContestRoutes.post("/contests/:id/clone", requireTeacher, async (c) => {
const [original] = await selectContest(id)
// 克隆不要求 ownedBy旧后端这里也没有 ensure_created_by教师可以拿别人的比赛做模板。
// 克隆出来的归调用者所有、且默认不可见,所以不构成越权修改。
if (!original) return failure(c, 404, "contest-not-found", "Contest does not exist")
```
注释对「不构成越权**修改**」的判断是对的,但漏了越权**读取**:克隆会把原比赛的全部题目连同 `password` 一起复制进调用者名下的新比赛,调用者随后对这些题有完整读写权。
**旧后端对应行为**`contest/views/admin.py:265-301``ContestCloneAPI.post``@teacher_admin_required`,确实没有 `ensure_created_by`,也照样 `password=original.password`。**行为等价**;但旧的 `ContestAdminSerializer` 同样下发 password所以泄露面一致属于原样继承的洞。
**实跑验证**
A 建密码保护赛 3password=`secret123`visible=true。B 直读被拦:
```
GET /api/admin/contests/3 (cookie = B) → 404 contest-not-found
PUT /api/admin/contests/3 (cookie = B) → 404 contest-not-found
GET /api/admin/contests/3/problems (cookie = B) → 404 contest-not-found
GET /api/admin/contests/3/acm-helper (cookie=B) → 404 contest-not-found
```
B 克隆:
```
POST /api/admin/contests/3/clone (cookie = B)
→ 201 {"data":{"id":4,"title":"REVIEW-CONTEST-A","description":"d","tag":"t",
"startTime":"2026-08-07 23:19:29.475+00","endTime":"2026-08-08 00:19:29.475+00",
...,"password":"secret123"
```
四条读接口全 404克隆一次全拿到外加明文密码。
**风险场景**:任一 Teacher Admin 可在开赛前克隆同事的比赛,读全部题面,并拿到密码直接以学生身份进原赛。
**修法**:至少不要把 `password` 复制到克隆件、也不要在克隆响应里回传;更彻底的做法是给 clone 加 `ownedBy`(会改变旧行为,需要产品确认「拿别人比赛当模板」是不是真需求)。
---
### I4 — `POST /upload-image` 收成 Super Admin教师/学生管理员写题面时会 403
**文件**`apps/api/src/routes/admin/conf.ts:206`
```ts
adminConfRoutes.post("/upload-image", requireSuperAdmin, async (c) => {
```
**旧后端对应行为**`utils/views.py:13-16``SimditorImageUploadAPIView.post` —— **完全没有装饰器**`utils/urls.py:6` 挂在 `api/admin/upload_image`),任何人包括匿名都能传图。新后端收成 Super Admin 是必要的修补,但收过头了。
前端 `ojnext/src/admin/api.ts:159-163``uploadImage``MarkdownEditor.vue` / `TextEditor.vue` 调用,而这两个编辑器出现在:
- `src/admin/problem/detail.vue` —— 题目编辑页,`requireProblemPermission` 档位Student Admin 就能进
- `src/admin/contest/detail.vue` —— 比赛编辑页,`requireTeacher` 档位
- `src/admin/announcement/detail.vue``src/admin/tutorial/detail.vue` —— 这两个确实是 Super Admin
**实跑验证**`e2etest` = Teacher Admin/Own
```
POST /api/admin/upload-image (cookie = Teacher Admin)
→ 403 {"error":{"code":"permission-denied","message":"权限不足"}}
```
**风险场景**:不是越权,是**可用性回归**——教师往比赛描述里插图、学生管理员往题面里插图,都会静默失败。这类问题在权限评审里要一起提,因为「修法」是调守卫档位。
**修法**:改成 `requireAdmin`(三种管理员均可)。上传目录是服务端生成文件名、限了后缀和 10MB风险可控。
---
## Minor
### M1 — `PUT /problems/:id/visibility` 的归属规则与 `canEdit` 不一致,比赛题上会反转
**文件**`apps/api/src/routes/admin/tag.ts:173-186`
```ts
if (!canManageAllProblems(user) && problem.createdById !== user.id) {
return failure(c, 404, "problem-not-found", "题目不存在")
}
```
用的是**公开题**的规则(看 `problem.createdById`),而 `problem.ts:44-52``canEdit` 对比赛题走的是「看比赛创建者」。结果在比赛题上两套规则给出相反答案。
**旧后端对应行为**`problem/views/admin.py:600-610``ProblemVisibleAPI.put` **完全没有归属校验**(而且 605-607 的 `self.error` 漏写 return题不存在会 500。新后端补了校验是净收紧这里记的是新后端内部两套规则不自洽。
**实跑验证**
题 41 位于 B 的比赛 5 里,但 `created_by_id = 2`Afrom-public 复制时继承的):
```
PUT /api/admin/problems/41/visibility (cookie = B比赛所有者) → 404 {"code":"problem-not-found","message":"题目不存在"}
PUT /api/admin/problems/41/visibility (cookie = A非比赛所有者) → 200 {"data":{"visible":false}}
GET /api/admin/problems/41 (cookie = A) → 404 # canEdit 说 A 无权
GET /api/admin/problems/41 (cookie = B) → 200 # canEdit 说 B 有权
```
同一道题,`canEdit` 判 B 能改、A 不能;`visibility` 判 A 能改、B 不能。A 能把 B 比赛里的题隐藏掉B 自己反而改不回来。
**修法**`visibility` 改用 `canEdit`
### M2 — `from-public` 的错误码构成比赛存在性预言机
**文件**`apps/api/src/routes/admin/problem.ts:544` vs `546-548`
比赛不存在返回 `not-found`,比赛存在但不属于你返回 `contest-not-found`。攻击者带一个已知有效的 `problemId`,就能靠错误码区分「这个 contestId 不存在」和「存在但是别人的」。其余所有跨租户路径都统一成了同一个码已实跑确认contest 系列全是 `contest-not-found`、problemset 系列全是 `problem-set-not-found`、problem 系列全是 `problem-not-found`),只有这一处破例。
**旧后端对应行为**`problem/views/admin.py:493-494` 统一返回 `"Contest or Problem does not exist"`,不区分。
(仅静态判断 + 上文 I2 实跑观察到的错误码差异,未专门构造预言机实验。)
### M3 — 三个 admin 端点的守卫写在 handler 体内,不在注册行
**文件**`apps/api/src/routes/submission.ts:221-224``submission.ts:337-340``apps/api/src/routes/flowchart.ts:142-145`
```ts
submissionRoutes.get("/submissions/statistics", requireAuth, async (c) => {
if (!isTeacherOrAbove(c.get("user"))) return failure(c, 403, "permission-denied", ...)
submissionRoutes.post("/submissions/:id/rejudge", requireAuth, async (c) => {
if (!isSuperAdmin(c.get("user"))) return failure(c, 403, "permission-denied", ...)
flowchartRoutes.get("/flowcharts/statistics", requireAuth, async (c) => {
if (!isTeacherOrAbove(c.get("user"))) return failure(c, 403, "permission-denied", ...)
```
**档位本身正确**,与 `submission/views/admin.py:14``@super_admin_required`)、`submission/views/admin.py:31``@teacher_admin_required`)、`flowchart/views/admin.py:69``@teacher_admin_required`)逐条对应。问题只是写法违背了 `routes/admin/index.ts:15-21` 立下的约定(「守卫要从注册行上看得出来」),下一个人加同类端点时容易漏掉那个 if。建议改用 `requireTeacher` / `requireSuperAdmin` 中间件。
(仅静态判断,未实跑 —— 这三条不在阶段 4 的 admin 目录里。)
### M4 — 禁用账号落到 401 `login-required`,与旧后端的「账号已禁用」不同
**文件**`apps/api/src/auth/middleware.ts:32-34`(注释)、`41`
`getSessionUser` 对禁用用户返回 null → 401 `login-required`。旧 `account/decorators.py:36-37` 是先过权限检查再报 `permission-denied` + 「账号已禁用」。
方向上更安全(提前一步拦),但前端拦截器对 `login-required` 是弹登录框:一个会话中途被禁用的用户会陷入「弹登录框 → 登录 → 又被弹」的循环,而不是看到「账号已禁用」。属于可用性 / 可诊断性问题,不是越权。
(仅静态判断。)
---
## 有意偏离的独立判断
| 偏离 | 判断 | 依据 |
|---|---|---|
| 删除不存在的记录返回 404 而非静默成功 | **合理** | 旧 `announcement/views/admin.py:57-61``tutorial/views/admin.py:62-66``achievement/views/admin.py:72-78` 都是 `filter().delete()` 静默成功,后台点了删除却没删掉是真的会误导人。这些资源全在 Super Admin 档位404 不构成枚举风险。**唯一例外是奖章删除**——那里的 404 和真实副作用不一致,见 C1问题不在 404 本身而在事务顺序。 |
| 题单后台列表不按 visible 过滤 | **合理,且是必要修复** | 旧 `problemset/views/admin.py:35` 写死 `filter(visible=True)`,同时 `ProblemSetVisibleAPI`(356-372) 又提供切换可见性——设成不可见后题单从后台列表消失,界面上再也改不回来。新写法仍保留 `createdById` 过滤(`problemset.ts:80`),越权面没有变化。已实跑确认 A 看不到 B 的题单(`GET /problem-sets` 返回 `{"results":[],"total":0}`)。 |
| ACM 核查校验 rank 归属 | **合理,且修掉了旧后端一个真洞** | 旧 `contest/views/admin.py:205` 只按 `pk=data["rank_id"]` 取,任一 Teacher Admin 带任意 rank_id 就能改别的比赛的核查标记。新 `contest.ts:298-302``and(id, contestId)` 收口,父子归属正确。(静态判断 + `GET /contests/3/acm-helper` 的跨租户 404 实跑PUT 分支因需要真实 ACM 排名数据未实跑。) |
| 题目删除统一成一条路由 | **合理** | 旧的两条(`ProblemAPI.delete` 332 行 `contest_id__isnull=True` + `ContestProblemAPI.delete` 443 行 `contest_id__isnull=False`)只是按 contestId 分流,比赛是从题目推导出来的,前端根本不需要传 contestId。新 `problem.ts:371-379``canEdit` 已经把两种归属规则都覆盖了:公开题看题目创建者、比赛题看比赛创建者,与旧的 `ensure_created_by(problem)` / `ensure_created_by(problem.contest)` 一一对应。附带把「有提交记录不能删」从只对比赛题生效(旧 447 行)推广到公开题,是收紧不是放松。 |
---
## 核实过、确认没问题的部分
这一节界定覆盖面,与上面的问题清单同等重要。
**角色守卫档位86/86 逐条比对,全部一致,无一漏挂)**
| 新后端路由组 | 守卫 | 旧后端装饰器 | 结论 |
|---|---|---|---|
| `/users*`6 条) | requireSuperAdmin | `account/views/admin.py:43,76,125,159,244` `@super_admin_required` | 一致 |
| `/announcements*`5 条) | requireSuperAdmin | `announcement/views/admin.py:13,23,40,57` | 一致 |
| `/tutorials*` `/exercises*`10 条) | requireSuperAdmin | `tutorial/views/admin.py:17,27,44,62,70,90,106,119,127` | 一致 |
| `/achievements*` `/achievement-metrics`6 条) | requireSuperAdmin | `achievement/views/admin.py:10,21,43,72,82` | 一致 |
| `/website` `/judge-servers*` `/orphan-test-cases*` `/dashboard` `/random-usernames`9 条) | requireSuperAdmin | `conf/views.py:49,66,76,84,148,162``DashboardInfoAPI`(187) 与 `RandomUsernameAPI`(216) 旧后端**无装饰器** | **收紧**(后两条旧后端任何管理员可读,不限超管) |
| `/contests*`5 条) | requireTeacher | `contest/views/admin.py:33,53,78,267` `@teacher_admin_required` | 一致 |
| `/contests/:id/acm-helper`2 条) | requireTeacher | `contest/views/admin.py:167,200` | 一致 |
| `/problem-sets*`15 条) | requireTeacher | `problemset/views/admin.py` 全部 `@teacher_admin_required` | 一致 |
| `/ai/reports*`3 条) | requireTeacher | `ai/views/admin.py:9,31` `@teacher_admin_required` | 一致 |
| `/problems/stuck` `/problems/ac-trend` | requireTeacher写了但被遮蔽 | `problem/views/admin.py:639,676` | 档位对,见 I1 |
| `/problems*` `/contests/:id/problems*` `/test-cases` `/sql-test-cases/*`14 条) | requireProblemPermission | `problem/views/admin.py:237,259,294,326,458,757,798``TestCaseAPI`(143)、`ContestProblemAPI`(343) 旧后端**无装饰器**、`AddContestProblemAPI`(486) 亦无 | 一致或**收紧** |
| `/problem-tags*` `/problems/batch-tag` `/problems/:id/visibility` `/problems/flowchart`5 条) | requireProblemPermission | `problem/views/admin.py:515,524,554,570,601,614` | 一致 |
**中间件语义**`auth/middleware.ts:48-67``ADMIN_ROLES` / `TEACHER_ROLES` 白名单与 `account/models.py:65-73``is_admin_role()` / `is_teacher_or_above()` 逐项一致;`requireProblemPermission` = 管理员 ∧ `problemPermission !== "None"`,与 `decorators.py:78-84` 一致。已实跑确认 Teacher Admin + `problem_permission='None'``/problems/stuck` 上得到 403`/contests` 上得到 200。
**跨租户读写隔离(实跑,全部 404、不泄露存在性**
```
(cookie = B, 全部指向 A 的资源)
GET /api/admin/problem-sets/2 → 404 problem-set-not-found
GET /api/admin/problem-sets/2/badges → 404 problem-set-not-found
DELETE /api/admin/problem-sets/3/progress/4 → 404 problem-set-not-found (A 打 B)
PUT /api/admin/problem-sets/2/problems/1 → 404 problem-not-in-set (跨题单 itemId)
GET /api/admin/contests/3 → 404 contest-not-found
PUT /api/admin/contests/3 → 404 contest-not-found
GET /api/admin/contests/3/problems → 404 contest-not-found
GET /api/admin/contests/3/acm-helper → 404 contest-not-found
GET /api/admin/problems/37 → 404 problem-not-found
DELETE /api/admin/problems/37 → 404 problem-not-found
GET /api/admin/problems/37/test-cases → 404 problem-not-found
PUT /api/admin/problems/37/visibility → 404 problem-not-found
GET /api/admin/problems → 200 {"results":[],"total":0} # 列表按 createdById 过滤
POST /api/admin/problems/batch-tag {problemIds:[40]} → 404 no-problems # 非本人题目被过滤掉
```
错误消息统一是「不存在」,没有出现「无权限」。**归属校验的正向行为是对的,问题只出在 C1/C2/I2 三条例外路径上。**
**超管旁路(实跑)**`student` 提为 Super Admin 后,`GET /problem-sets/3``GET /contests/5``GET /problems/41` 全部 200`ownedBy` / `canEdit` 的超管分支正确。
**其他确认无误的点**
- `canEdit``problem.ts:44-52`)对「比赛题看比赛创建者、公开题看题目创建者」的区分实现正确,且比赛题分支**不**给 `problemPermission=All` 旁路 —— 与旧 `ensure_created_by` 对非 Problem 实例只认 `created_by`/超管的语义一致
- `PUT /problem-sets/:id/problems/:itemId``DELETE /problem-sets/:id/problems/:itemId``PUT /problem-sets/:id/badges/:badgeId``DELETE /problem-sets/:id/progress/:userId` 的父子归属都写在同一条 `and()` 里,无 TOCTOU已实跑确认**只有 badges 的 DELETE 例外,见 C1**
- `DELETE /problem-tags/:id``tag.ts:92-102`)的事务里先删 `problemTags` 再删 `problemTag`,但删的是同一个 id标签不存在时前一句是空操作**不存在 C1 那种连带破坏**
- `DELETE /users``account.ts:237-257`)禁止删除自己,已实跑确认返回 400 `cannot-delete-self`
- `PUT /users/:id``normalizePermission``account.ts:49-53`)与 `account/views/admin.py:98-105` 的归一逻辑一致,降级超管会同步清掉 All
- 真名下发受控:`sampleUser``routes/helpers.ts:15-25`)默认 `realName: null`,只有 `acm-helper``contest.ts:272`)显式下发,在 requireTeacher + 归属校验之后
> **2026-08-31 订正**:本条原先还写了「题单进度(`problemset.ts:426`)」,与代码不符 ——
> 学生端那条 `GET /problem-sets/:id/user-progress` 走的是 `sampleUser(progressUser, realName)`
> 没传 `includeRealName``realName` 恒为 `null`SQL 里那次 `leftJoin userProfile` 是白查的)。
> 真正显式下发真名的是**后台**那条 `GET /admin/problem-sets/:id/progress`,它手写
> `adminProblemSetProgressSchema`,在 requireTeacher + `loadOwned` 归属校验之后,结论仍成立。
>
> 另外当时漏了一条:学生端那条 `user-progress` 只有 `requireTeacher`、**没有归属校验**
> 任何 Teacher Admin 都能读到别人建的题单的学生名单与进度(不含真名)。已补上归属校验,
> 口径与后台的 `loadOwned` 一致,越权报 404。
- `GET /ai/reports/:id``ai.ts:70-83`)不下发 `data` / `systemPrompt` / `userPrompt`
- `GET /judge-servers``conf.ts:90-100`)下发 judge token但在 requireSuperAdmin 之后,与旧 `conf/views.py:66-74` 一致
- `DELETE /orphan-test-cases``conf.ts:147-160`)对指定 id 也先确认是孤儿,比旧 `conf/views.py:162-171` 严 —— 合理收紧
---
## 覆盖率
| 项 | 数 |
|---|---|
| `routes/admin/*.ts` 中的 handler | 86 |
| 逐条静态比对旧后端装饰器的 handler | 86100% |
| `/admin` 之外但对应旧 admin 视图的 handler | 3submission statistics / rejudge、flowchart statistics |
| **合计评审端点** | **89** |
| 实跑验证的端点 | **46**52% |
| 实跑覆盖的角色组合 | Regular User(隐含匿名 401)、Student Admin+Own、Teacher Admin+None、Teacher Admin+Own、Super Admin+All |
| 实跑的跨租户攻击路径 | 18 条14 条被正确拦截、4 条成功越权) |
未实跑的 43 条主要是:只在 Super Admin 档位、无归属概念的 CRUDtutorials/exercises/achievements/announcements 的写操作),以及需要真实测试点文件或真实 ACM 排名数据的端点(`/test-cases` 上传、`/sql-scripts``/sql-test-cases/*``acm-helper` 的 PUT、`/problems/flowchart` 需要 AI 服务)。这些的守卫档位已逐条静态核对,均与旧后端一致。
## 环境还原确认
```
select 'problem',count(*) from problem → 20 (与评审开始时一致)
select 'contest',count(*) from contest → 0
select 'problemset',count(*) from problemset → 0
select 'badge',count(*) from problemset_badge → 0
select 'user_badge',count(*) from user_badge → 0
select count(*) from problem_tag where id=95 → 0 (测试标签已删)
id|username|admin_type |problem_permission
1 |devadmin|Super Admin |All
2 |student |Regular User |None
4 |e2etest |Regular User |None
```
`e2etest` 的密码在测试中被 reset-password 接口改掉,已通过 `PUT /users/4` 改回 `Test123456` 并重新登录验证成功;`user_profile.real_name` 也已还原为 NULL。未执行任何 drop / truncate。

View File

@@ -1,150 +0,0 @@
# Phase 4 安全评审 — SQL 判题引擎沙箱逃逸与资源耗尽
评审对象:`apps/api/src/judge/sql/`engine.ts / child.ts / index.ts+ 判题接线 `judge/run.ts:judgeSqlSubmission` + 测试点处理 `services/test-case.ts`
参照旧实现:`OnlineJudge/judge/sql_runner.py` / `sql_dispatcher.py`Python sqlite3 + authorizer / progress_handler / setlimit
**评审方式**:全部实跑。快速迭代喂 `child.ts` JSON 作业;关键结论走完整提交链路(建 SQL 题 → 提交 → 判题结果)与父进程 `index.ts` 逻辑。测试环境已还原(题目/提交/测试点目录已删,`student` 已改回 Regular User / None无残留进程
> **修复记录2026-08-07本文档之后**
>
> - **I-1 已修**`runStudent` 加逐语句守卫,学生 SQL 里的 PRAGMA 一律拒(两种题型都拒,
> 对齐旧实现 authorizer 的 `_DENIED_ALWAYS`)。关键字判定用 `sqlite3_normalized_sql`
> 注释/大小写/空白由 SQLite 自己抹平。另加兜底:每条语句前重放 `query_only` 与
> `max_page_count`,即便判定漏了也关不掉只读。**M-1 的 `max_page_count` 可调大部分一并修掉。**
> - **I-2 已修**:子进程用 stderr 报阶段(`prepare` / `student` / `display`),父进程边读边换
> 兜底时限 —— 一进学生 SQL 就收到「题目时限 + 2s」。1s 限的题跑飞语句实测 **26s → 3.06s**。
> 顺带修正归因:卡在受信脚本(出题人的初始化/标准答案)现在报 SYSTEM_ERROR不再甩给学生 TLE。
> **补充2026-08-08Minor 收尾)**
>
> - **M-1 已全部修完。** 第二半(`max_page_count` 学生可调大)在 I-1 拦 PRAGMA 时就一并
> 修掉了。第一半改成引擎侧按字节记账(`ByteBudget`):单值超题目内存限、或结果集
> 累计超限,都按 MLE 拒。旧实现的 `setlimit(SQLITE_LIMIT_LENGTH)` 复刻不了 ——
> sql.js 的 wasm 没导出 `sqlite3_limit`。实测**同一句 SQL 在 4MB 的题上被拦、
> 在 64MB 的题上放行**,题目的 `memoryLimit` 现在对学生是真约束。
> (测试尺寸特意选在 8MB 而不是 100MBwasm 堆触顶时的兜底报的是同一句错误信息,
> 用大值测根本分不清是谁拦的 —— 第一版测试就是这么误判的。)
> - **M-3 已由阶段 5 的分阶段超时顺带解决**:预览从 25s 降到 **13005ms** 实测。
> - **M-2 不修**(见下)。
> **M-2 的处理:不修,理由写在这里**
>
> 要做「强制两个测试点的期望结果不同」,得在建题时把每个测试点都跑一遍标准答案再比对。
> 技术上可行,但它会把一类**合法出题**也挡掉(比如刻意用两组不同数据验证同一个边界、
> 结果恰好相同),而代价是老师建题时被一条看不懂的错误拦住。评审也确认这与旧实现行为
> 一致、非回归。
>
> 更合适的做法是「提示」而不是「拒绝」,但那要动契约和后台前端。留给以后,
> 现在记在这里,免得下次评审又把它当成新发现。
> - `engine.ts` 头部的防护对照表已按实测重写,两处削弱写在正文里。
> 另记stock sql.js 的 wasm **没有导出** `sqlite3_progress_handler` / `sqlite3_interrupt` /
> `sqlite3_set_authorizer` / `sqlite3_limit`(已核对导出表),要用得自己编 wasm。
## 结论速览
| # | 威胁 | 结果 | 级别 |
|---|---|---|---|
| 1 | 文件系统逃逸ATTACH 等) | **打不穿**。WASM 无宿主 FS 绑定,结构性隔离,比旧实现更强 | — |
| 2 | 超时挂住判题进程 | 单条长语句引擎拦不到,只靠父进程 25s 后 SIGKILL1s 限的题实测 26s 才 TLE25x 放大 + 判题池仅 2 并发 → 可拒绝服务 | **Important** |
| 3 | 内存耗尽 | 单值/瞬时内存被固定 512MB `ulimit -d` 兜住(已验证生效),但与题目 `memoryLimit` 脱钩,且 `max_page_count` 学生可自行调大 | **Minor** |
| 4 | 查询题只读绕过 | **破防**`PRAGMA query_only=0` 一句即可恢复写权限,实测查询题用 DML 拿到 Accepted | **Important** |
| 5 | 硬编码常量作弊 | 多测试点防线成立(前提:测试点数据确实不同,代码只强制"≥2 个"不强制"数据不同" | **Minor** |
| 6 | 出题人侧预览/生成 | 与学生同隔离,无宿主 FS比旧实现更强教师可让预览请求挂 25s | **Minor** |
对照表逐条核对结果(实现者声明 vs 实测):
| 旧防护 | 新做法 | 核对结果 |
|---|---|---|
| authorizer 禁 ATTACH | WASM 无宿主 FS结构上够不到 | ✅ **成立且更强** |
| authorizer 白名单让查询题只读 | `PRAGMA query_only=1` | ❌ **不成立**:学生可 `PRAGMA query_only=0` 反手关掉 |
| progress_handler 墙钟超时 | 子进程外部 SIGKILL | ⚠️ **部分成立**:只在父进程路径生效且慢 25x单条长语句引擎内拦不到 |
| `setlimit(LIMIT_LENGTH)` | 子进程 `ulimit -d` | ⚠️ **部分成立**`-d` 确实作用到最终进程,但固定 512MB、与题目内存限脱钩 |
---
## Important
### I-1 查询题只读可被 `PRAGMA query_only=0` 一句绕过
- **位置**`apps/api/src/judge/sql/engine.ts:199-200``if (mode === "query") db.run("PRAGMA query_only=1")`),学生 SQL 随后在 `engine.ts:202 → executeStatements` 无任何语句级过滤地执行。
- **根因**:新实现放弃了 authorizer学生 SQL 的 PRAGMA 完全不设防。`query_only` 只是一个可写开关,学生自己就能把它关回去。旧实现把 `SQLITE_PRAGMA` 放进 `_DENIED_ALWAYS`,学生连一条 PRAGMA 都跑不了(`sql_runner.py` `_run_student` authorizer
- **攻击手法**:查询题提交 `PRAGMA query_only=0; <任意 DML/DDL>; <正确 SELECT>`
- **实测完整提交链路query 题 id=43标准答案 `SELECT a FROM t ORDER BY a`**
攻击提交:
```sql
PRAGMA query_only=0; UPDATE t SET a=a; SELECT a FROM t ORDER BY a;
```
→ `RESULT 0`**ACCEPTED**)—— UPDATE 成功执行,只读约束失效。
对照提交(不带 PRAGMA
```sql
UPDATE t SET a=a; SELECT a FROM t ORDER BY a;
```
→ `RESULT 4`RUNTIME_ERROR`err_info: 本题为查询题,禁止修改数据或表结构…`
child.ts 层复现同样区分:带 PRAGMA 的 `DELETE/INSERT` 走到 WA写入生效不带的报 readonly。
- **旧实现表现**`PRAGMA query_only=0` 会被 authorizer 直接 DENY`not authorized`),学生无法改动只读态;且白名单模式下任何非 SELECT 授权码都拒绝。
- **实际危害边界**:每个测试点都是独立的内存库、无宿主 FS、跨测试点不持久因此**不构成沙箱逃逸,也不构成作弊**学生仍需对所有测试点产出正确结果集DML 帮不上)。危害限于:查询题的"只读"教学约束被破坏,学生可用增删改而非查询"混"过题目。但它直接证伪了对照表第 2 行的安全声明,且利用成本为零,故列 Important。
- **修复方向**:学生 SQL 执行前用 `iterateStatements` 预扫,遇到 PRAGMA或至少 `query_only` / `max_page_count` / `writable_schema` 等敏感 PRAGMA即拒或在只读模式下用不可翻转的手段如整库以 immutable/只读方式打开)替代可写开关。
### I-2 单条长语句超时只能靠父进程 25s 后 SIGKILL放大 25x 且可拖垮判题池
- **位置**`engine.ts:116-149``executeStatements` 的 deadline 检查在 **`engine.ts:126`**,只在语句之间触发);`index.ts:45``setTimeout(kill, budgetMs + HARD_TIMEOUT_SLACK_MS)``index.ts:80``runSqlCase` budget = `max(timeLimitMs*5, 10_000)``index.ts:25``HARD_TIMEOUT_SLACK_MS = 15_000`)。
- **根因**:引擎的墙钟检查只发生在 `iterateStatements` 的每条语句开头。**一条长时间运行的语句(递归 CTE 死循环、大 CROSS JOIN 喂聚合)在单次 `step()` 内部执行,`step()` 期间不检查 deadline**,引擎无法中断。旧实现的 `progress_handler` 每 1000 条 VM 指令回调一次,在语句执行**过程中**就能按墙钟中断,约 `timeLimitMs`1s即杀。
- **攻击手法**`WITH RECURSIVE r(i) AS (SELECT 1 UNION ALL SELECT i+1 FROM r) SELECT count(*) FROM r;`(聚合,不向外层吐行,`ROW_LIMIT` 也拦不到)。
- **实测**
- 直喂 child.ts无父进程`timeout 12` 强杀,**子进程 12s 内从不自终**(引擎 deadline 对单语句无效)。
- 走父进程 `index.ts` `runSqlCase`timeLimitMs=1000`elapsed_ms=25005`,返回 `{"ok":false,"result":1,"message":"SQL 执行超时"}`。
- **完整提交链路**1s 限的 query 题):从提交到出 `RESULT 1`TLE实测 **26 秒**。
- **判题池影响**`config.ts:68` `judgeConcurrency` 默认 **2**SQL 判题作业与所有语言共用同一 BullMQ 判题 worker。用户级限流 `capacity=20``throttling.ts:24`)。单个学生一次性提交 20 条死循环 SQL ≈ `20×25s / 2 = 250s`**两个判题槽被占满约 4 分钟**,期间全体学生(含 Python/C提交排队。考试/比赛期这是可用性风险。
- **旧实现表现**`progress_handler` 在 ~1s`timeLimitMs`即中断worker 几乎立刻释放。新实现慢 25 倍且完全依赖父进程(子进程独立运行时永不自终)。
- **修复方向**:把 `HARD_TIMEOUT_SLACK_MS` 从 15s 收紧、`runSqlCase` budget 别乘 5或给 sql.js 编译进 `sqlite3_progress_handler` 等价的中断回调,在语句内按墙钟中断。
---
## Minor
### M-1 学生内存受固定 512MB `ulimit -d` 约束,与题目 `memoryLimit` 脱钩;`max_page_count` 学生可调大
- **位置**`index.ts:23``CHILD_DATA_LIMIT_KB = 512 * 1024`**硬编码**,不读题目 `memoryLimit``index.ts:38-41``sh -c "ulimit -d …; exec …"``engine.ts:101-108``newDatabase``max_page_count = memoryLimitMb*256`)。
- **核对"`ulimit -d` 是否真作用到最终 bun 进程"(重点关注项)****成立**。通过同款 `sh -c "ulimit -d 524288; exec bun …"` 包装起子进程后读 `/proc/self/limits`
```
Max data size 536870912 536870912 bytes
```
`exec` 保留 rlimit`-d` 确实落到最终 bun 进程。`hex(zeroblob(3e8))` 实测被拦,返回 `RESULT 3`MEMORY_LIMIT_EXCEEDED"单个数据值超出内存限制"),完整链路复现一致。所以 `-d` 而非 `-v` 的选型与"作用到最终进程"的声明都成立。
- **不足**
1. 512MB 是**全局固定值**,与题目 `memoryLimit`(如 64MB无关。旧实现 `setlimit(SQLITE_LIMIT_LENGTH, memory_limit_mb*1MB)` 把单值上限贴着题目内存限64MB。新实现下 64MB 的题,学生单值/瞬时内存可到 ~512MB 才报 MLE8x
2. 题目内存限唯一的落点 `max_page_count` 是一条 **PRAGMA学生可自行 `PRAGMA max_page_count=1000000` 调大**(同 I-1 根因PRAGMA 不设防)。实测该 PRAGMA 被接受、无 authorizer 拒绝。因此题目 `memoryLimit` 对学生**不是权威约束**,真正的硬顶只有那 512MB。
- **危害边界**512MB 仍能兜住机器(单进程封顶、`ROW_LIMIT=10000` 兜结果集行数),不至于拖垮宿主;只是"每题内存限"名不副实。故 Minor。
### M-2 测试点只强制"≥2 个",不强制"数据不同";作弊防线依赖出题人
- **位置**`services/test-case.ts:102-105``if (options.sql && selected.length < 2) throw …`)。
- **核对作弊假设(题目要求至少 2 个数据不同的测试点,硬编码常量过不了)****假设成立,但仅在测试点数据确实不同的前提下**。
- 实测TC1 数据 `1,2,3`TC2 数据 `10,20,30,40`):提交硬编码 `SELECT 1 UNION SELECT 2 UNION SELECT 3;` → `RESULT -1`WRONG_ANSWER。多测试点防线有效。
- 但代码只校验 `.sql` 文件个数 ≥2**不校验两个测试点跑出的期望结果是否不同**。题目页展示的是**测试点 1** 的期望结果(`buildDisplay` 取 test case 1`admin/problem.ts:160-173`),学生能看到 TC1 的答案。若出题人不慎让 TC2 的数据产出与 TC1 相同的结果集,硬编码即可 AC。
- **旧实现表现**:同样基于"跑标准答案 + 多测试点比对",无"数据必须不同"的强校验,行为一致(非回归)。故 Minor属出题人操作风险。
### M-3 出题人侧预览可挂 25s教师自伤有限
- **位置**`admin/problem.ts:653``/sql-test-cases/preview` → `buildSqlDisplay`走同一子进程隔离budget 10s + slack 15s。
- **实测**:教师 `refSql` 放死循环递归 CTE → 预览 HTTP 请求阻塞 **25005ms** 后返回 400 `生成展示数据超时或内存超限`。占用一个 HTTP 处理 25s。教师半可信、且限于自身请求Minor。
---
## 试过但没打穿(覆盖面界定)
- **ATTACH 写宿主文件**query & modify & 受信 init 三种上下文):`ATTACH DATABASE '/tmp/…/evil.db' AS evil; CREATE TABLE evil.x…` → `unable to open database`,宿主上**无文件生成**。WASMsql.js默认 MEMFS无 NODEFS 绑定,结构性够不到宿主 FS。
- **ATTACH 读已存在的宿主 sqlite 文件**modify 模式,目标是真实 `secret.db`):同样 `unable to open database`,读不到。
- **受信脚本(出题人 init里的 ATTACH**:预览接口同样 `unable to open database`。旧实现靠 `_trusted_authorizer` 拒 ATTACH 且跑在有宿主 FS 的 Django 进程里;新实现结构性隔离,**更强**。
- **超大单值撑爆内存**`hex(zeroblob(3e8))` → 被 512MB `ulimit -d` / WASM 堆拦MEMORY_LIMIT_EXCEEDED未 OOM 宿主。
- **结果集撑爆**`ROW_LIMIT=10000` 在 `step()` 循环内计数,超限即 MLE`engine.ts:136-138`)。
- **硬编码常量答案**(跨数据不同的 2 测试点WRONG_ANSWER。
- **多条语句的循环/超时**:语句之间的 `Date.now() > deadline``engine.ts:126`)能在 ~timeLimit 处以 `interrupted` → TLE 拦下(仅**单条**长语句拦不到,见 I-2
- **子进程 `finish()` 的 SIGKILL 自杀导致结果丢失/截断**:未复现。`child.ts:47-51` 用 `writeSync(1,…)` 循环写满全部字节再 `SIGKILL`;父进程 `index.ts:49-52` 并发 `new Response(child.stdout).text()` 持续排空管道。判题结果 payload 仅数百字节,全部实测被完整解析。理论残余风险:若 stdout payload 超过管道缓冲64KB且父进程未及时排空、fd 1 为非阻塞,`writeSync` 可能 EAGAIN 抛错丢输出——但当前 payload 体量下不触发,且父进程始终并发排空,判定低风险。设计中"跳过 Bun teardown 避免 `ulimit` 下 SIGILL panic 误判超时"的推理合理。
- **空 stdout → 超时误判**:父进程 `index.ts:58-66` 在子进程被 SIGKILLstdout 空)时按 `@phase` 标记区分:卡在 display 报 SYSTEM_ERROR卡在 judge 报 CPU_TIME_LIMIT_EXCEEDED。递归 CTE 死循环实测正确落到 TLE。
## 备注
- 未发现真正的沙箱逃逸(文件/进程/宿主)或能让错误答案判 Accepted 的作弊路径。最接近"严重"的是 I-2 的判题池拒绝服务(可用性),其次 I-1 的只读破防(安全控制失效但危害受限)。
- 环境已还原:题目 id=43 及其 5 条提交、`problem_tags` 关联、两个测试点目录(`d6sps…`、`v3h9…`)已删;`data/test_case/` 仅剩原有 `79343208b704f6e75b4b6d885285280e``student` 已改回 Regular User / None无残留 `child.ts`/攻击进程。未改动任何 OJ2 / OnlineJudge / ojnext 源码(两旧仓 `git status` 干净)。

View File

@@ -1,597 +0,0 @@
# 阶段 5切换手册与演练报告
演练日期2026-08-08
演练用快照:`db_backup_2026_08_07_10_39_19.sql`pg_dumpall 集群备份230MB2026-08-07 10:39
演练方式:本机 Docker`docker/compose.debian.yml` **本身**跑,不是简化版。
真实数据量:**1710 用户 / 956 题 / 123140 提交 / 64 比赛 / 16 题单 / 38 成就**。
---
## 一、演练结论
### 出口标准30 分钟内完成,回滚路径已验证
| 项 | 实测 |
|---|---|
| 停旧栈 | 11s |
| 起新栈(镜像已构建) | 34s再次 start 6s |
| 数据迁移 | **0不需要** —— 见下 |
| 全链路验证 | 约 2 分钟 |
| **合计** | **不到 1 分钟的停机**,远在 30 分钟内 |
真正的时间风险不在切换本身,而在**构建镜像**:首次构建约 5 分钟(依赖下载占大头)。
**镜像必须在停机窗口之前就构建好**,见下面的检查清单。
### 两个原本以为要做、实测不需要做的事
**1. 切换本身不需要执行任何 DDL。** 把生产 dump 的表结构和新后端在用的库逐列对比:
两边都是 **278 列,完全一致**,没有新增、没有删除、没有新表。新后端直接跑在现有结构上。
> 后来追加了一个**性能索引**(提交列表默认视图从全表扫变成索引扫),并且改用
> drizzle migration 管理,见 `apps/api/src/db/0001_add_submission_public_create_time_idx.sql`。
> 它**不属于切换流程**,切换前后任何时候单独跑一次即可,跑早点更好,
> **别塞进停机窗口**12.3 万行实测建索引 74ms但没必要占用窗口时间
>
> 对生产库第一次执行前,**必须先手插 `__drizzle_migrations` 的基线行**,否则 migrate
> 会从 `0000` 的完整建表跑起、撞表回滚,而且**失败时不打印任何错误**。
> 基线 SQL 和完整说明见 `CLAUDE.md` 的「改 schema 走 drizzle migration」。执行
>
> ```bash
> # 1) 先插基线行(见 CLAUDE.md
> # 2) 再跑
> DATABASE_URL=<生产库> bun run db:migrate
> ```
>
> 这会在生产库里新建一个 `drizzle` schema 和 `__drizzle_migrations` 表。
**2. 不需要重置序列。** 我在 `phase3-coverage.md` 里记过一条「切换必做:重置各表 sequence」
**那条是错的** —— 它来自我手工造的本地库(用显式 id 导入、没有 setval。真实的
pg_dumpall 备份带 30 条 `setval`,而且直接查生产快照里所有序列 vs `max(id)`
**错位 0 个**。切换当天不用管序列。
### 回滚保证(已实测)
> ⚠️ **2026-08-26本节已作废只作历史记录。** 旧 Django 后端确认不再使用,
> `0002_drop_django_leftovers` 删掉了它的 7 张框架表,**该迁移已在生产库执行完毕**。
> 旧栈现在起不来了:本节和第七节描述的「停新栈起旧栈」「把 NPM 上游改回 8080」
> 全部失效,**唯一退路是从数据库备份恢复**。
新栈跑完登录、提交、判题之后,再和生产 dump 的结构比一次:**逐列一致,零差异**。
新后端不会给旧后端留下任何它不认识的东西。
加上数据目录布局照抄旧后端(`test_case``public/upload``public/avatar` 原样不动),
**回滚 = 停新栈 + 起旧栈,约 20 秒,不涉及任何数据操作。**
> 未实测的部分:本机没有构建旧后端的 Django 镜像,所以「起旧栈」这一步本身没跑过。
> 但旧栈在服务器上一直是跑着的,它的镜像和 compose 都没动过。
> **⚠️ 2026-08-16 补:上面这句「零差异」只成立在表结构层面,不成立在语义层面。**
>
> 新后端原本会在登录成功时把 Django 的 pbkdf2 哈希升级成 argon2id 写回 `user.password`。
> 升级过的账号**旧后端再也验不了**(格式不同,而且旧后端连 argon2-cffi 都没装),
> 于是「登录过新站的学生,回滚之后登不上旧站」—— 一道结构比对发现不了的单向门。
> 试跑第一天就撞上了,现象是旧站登录失败 + WS 连不上Channels 认 session
>
> 当时的修法是加 `PASSWORD_HASH_UPGRADE` 开关、默认关闭。
>
> **⚠️ 2026-08-26 更新:开关已经删掉,现在无条件写 argon2。**
>
> 两个原因。**一是那个开关一直是假的**:新后端写密码的地方有五个(登录时自动
> 升级、注册、管理员改密码、批量导入用户、**重置密码**),开关只管住了第一处,
> 后面四处照写 argon2 不误 —— 也就是说开关关得好好的,老师给学生点一次
> 「重置密码」,那个账号照样回不去旧站,而这正是老师天天在用的功能。
>
> **二是旧站已经下线**,没有回滚路径要照顾了。五处现在统一走
> `auth/password.ts` 的 `hashPassword()`,只有一个地方写密码。
>
> 真要把旧站拉回来,正经退路是下面「万一已经改坏了」那节的脚本 —— 它拿
> `raw_password` 重算 Django 的 `make_password`,能修**已经**变成 argon2 的账号。
> 开关只能拦住将来,修不了已经发生的,这也是它不值得留的原因。
>
> `verifyPassword` 的 pbkdf2 分支**永远保留**:生产库 1710 个账号全是 Django 写的
> pbkdf2迭代次数 1200001200000它们只会在各自下次登录时才迁移成 argon2。
---
## 二、演练验证了什么
全部通过真实生产数据、经 Caddy、在容器里跑
| 检查项 | 结果 |
|---|---|
| 首页 SPA | 200Caddy 从镜像里伺服 |
| 站点配置 `/api/site` | 200读出真实站点名「判题狗」 |
| 题目列表 | 200首条是 `SQL09` |
| 题目标签 / 公告 | 200 |
| 未登录访问后台 | 401 `login-required` |
| 登录 | 200argon2 新哈希 + Django pbkdf2 旧哈希都支持) |
| 个人主页 / 排行榜 | 200真实学生数据 |
| 后台首页 / 题目 / 判题机 | 200`userCount: 1711` |
| **判题机心跳** | 判题容器自行注册成功(新路径 `/api/judge-server/heartbeat` |
| **完整判题** | 提交 Python A+B → **AC1.2 秒**,两个测试点全过 |
### WebSocket 实时推送(经 Caddy
学生盯着「判题中…」变成结果就靠这条路,而它经过 Caddy 的 `handle /ws/*`
是配置最容易写错、又只在生产才暴露的一段。单独验过:
```
WS 经 Caddy upgrade: 已连接
收到 2 条推送301ms
→ {"type":"submission_update","result":7,"status":"judging"}
→ {"type":"submission_update","result":0,"status":"finished","time_cost":4,...}
```
### 机房那套(`compose.school.yml`
这套配置和服务器那套差别不小(没有 postgres、连远程库、端口 81、
`COOKIE_SECURE=false`),单独跑过一遍:留下 `oj-postgres` 当「远程库」,
其余容器换成 school 栈,`DB_HOST` 指向宿主机 IP。
| 检查项 | 结果 |
|---|---|
| 连上「远程」库 | oj-api healthy日志零错误 |
| 首页 / 题目列表 | 200数据来自远程库 |
| 登录 | 200 |
| **Cookie 没带 Secure** | ✓ —— 带了的话机房http 直连 IP会「登录成功又立刻变未登录」 |
| WS + 完整判题 | 连接成功300ms 内 judging → finishedAC |
也就是说机房用的是**本地 Redis + 本地判题沙箱 + 远程库**,判题不跨公网,
只有数据库查询走公网。
---
## 三、切换前准备(停机窗口之前做完)
### ⚠️ 演练没暴露的一个坑:两套 compose 的数据目录不是同一个地方
演练时是把生产 dump 恢复进 `OJ2/data/postgres` 的(见第十节),所以这件事被盖住了。
真实部署目录 `/root/OJDeploy` 的实际情况:
| | 旧栈(`docker-compose.yml` | 新栈默认值 |
|---|---|---|
| 库 | `/root/OJDeploy/data/postgres` | `/root/OJDeploy/OJ2/data/postgres` |
| 测试点 / 上传 / 头像 | `/root/OJDeploy/data/backend/` | `/root/OJDeploy/OJ2/data/backend/` |
新 compose 在 `OJ2/docker/` 下,`../data` 解析到的是 `OJ2/data`。**照默认值切过去,
postgres 会在一个空目录上初始化一个全新的空库** —— 站点能起来,但没有用户、没有题、
判题全挂、题面图片 404。旧数据完好无损回滚正常但当天会白吓一场。
因此 `compose.debian.yml` / `compose.school.yml` 加了 `DATA_DIR` 等三组变量,
**下面的流程按「只换前后端」形态写**:旧栈的 postgres / redis 容器继续跑,
新栈只起 api / worker / web / judge。数据库进程根本不重启库和文件一个字节都不用挪。
### 已经做掉的2026-08-16
- **`docker/.env.school` 已写好**`.gitignore` 排除,不进版本库):判题 token 新生成一条、
`JUDGE_CONCURRENCY=4``DB_HOST=150.158.29.156``DB_PORT=5445``COOKIE_SECURE=false`
`docker compose config` 验过插值正确。**还差两个值**,见下面第 1 步。
- **构建复验通过。** 演练之后又改过两次前端(`/api2` 前缀漏改 5 处、接回配置推送与
MaxKB在本机重新构建`oj2-web` 75MB 构建通过、单起容器首页 200`oj2-api`
完全命中缓存,说明后端源码在演练之后没动过。
这只证明「还能构建出来」—— 镜像不走镜像仓库,是各站点本地构建的,
**服务器和机房当地各自还要 build 一次**
- **「只换前后端」形态本机实跑验过。** 用 `docs/specs/schema.sql` 起了一个发布在宿主机
5445 的 postgres 冒充旧栈,新栈按下面的 env 起来4 个容器(没有 postgres / redis
`oj-api` healthy、首页与 `/api/site` `/api/problems` 200、未登录进后台 401。
读写两个方向都验了 —— 那个库的 `pg_stat_activity` 里有一条来自 172.17.0.1 的
`postgres.js` 连接,`judge_server` 表里也出现了新判题机写进去的心跳行。
### 1. 填两份 env各站一份都不进版本库
服务器 `docker/.env`(照 `docker/.env.example` 拷一份再填):
| 变量 | 填什么 |
|---|---|
| `POSTGRES_PASSWORD` | **生产库现有的口令**。库是原地不动的、不是新建的,这个值改不了(旧 compose 里是 `onlinejudge` |
| `DATA_DIR` | `/root/OJDeploy/data` —— **旧数据目录的绝对路径**。不填就是上面那个空数据坑 |
| `DB_HOST` / `DB_PORT` | `host.docker.internal` / `5445` —— 旧 postgres 已经把 5445 发布在宿主机上,走 host-gateway 过去,不出本机 |
| `REDIS_HOST` / `REDIS_PORT` | `host.docker.internal` / `5446` —— 同理,旧 redis 发布的是 5446 |
| `OJ2_JUDGE_TOKEN` | 可以换新的,`openssl rand -hex 32`;后端和判题机读同一个变量,一起换即可 |
| `JUDGE_CONCURRENCY` | 服务器 `2` |
| `AI_KEY` | 服务器那把 DeepSeek key |
机房 `docker/.env.school`:已写好,只差三个 ——
| 变量 | 填什么 |
|---|---|
| `POSTGRES_PASSWORD` | 同上,**和服务器那份一模一样**(连的是同一个库) |
| `DATA_DIR` | 机房那台的旧数据目录绝对路径。**机房也有测试点和上传文件**,同样不能用默认值 |
| `AI_KEY` | 机房那把,**和服务器不是同一把,别填串** |
漏填 `POSTGRES_PASSWORD` 不会静默起一个坏服务compose 里写的是 `${POSTGRES_PASSWORD:?}`
没填直接报错退出。**但 `DATA_DIR` 漏填不会报错**,它有默认值 —— 这条只能靠人盯。
> 想让新栈自带 postgres / redis本机开发、或将来旧栈彻底拆掉不设 `DB_HOST`
> `REDIS_HOST` `DATA_DIR`,起栈时加 `--profile local-data` 即可。
### 2. 先把镜像构建好(**别在停机窗口里干这件事**
服务器:
```bash
docker compose -f OJ2/docker/compose.debian.yml --env-file OJ2/docker/.env build
```
机房:
```bash
docker compose -f OJ2/docker/compose.school.yml --env-file OJ2/docker/.env.school build
```
首次约 5 分钟,依赖下载占大头。构建完确认两个镜像都在:
```bash
docker images | grep oj2-
# oj2-api latest 487MB
# oj2-web latest 75MB
```
### 3. 兜底备份
```bash
docker exec oj-postgres pg_dumpall -U onlinejudge > ~/oj-before-cutover-$(date +%F).sql
```
**不是为了迁移**(不需要 DDL、不需要迁数据是为了出事有退路。真要用它重建
先读第八节第 1 条 —— 这份备份会把角色口令覆盖回备份当时的值。
### 4. 确认 `DATA_DIR` 指对了
```bash
ls /root/OJDeploy/data/backend/test_case \
/root/OJDeploy/data/backend/public/upload \
/root/OJDeploy/data/backend/public/avatar
```
判题测试点、题面图片、头像都在这三个目录里。**这个路径必须和 env 里的 `DATA_DIR` 一致。**
再用 compose 自己确认一遍解析结果,别靠脑补:
```bash
docker compose -f OJ2/docker/compose.debian.yml --env-file OJ2/docker/.env config | grep source:
# 每一条都应该在 /root/OJDeploy/data 下,出现 OJ2/data 就是 DATA_DIR 没生效
```
### 5. 出发前对一遍
- [ ] 两份 env 都填完了,两边的 `POSTGRES_PASSWORD` 一致
- [ ] **两边的 `DATA_DIR` 都指向旧数据目录**`config | grep source:` 核对过
- [ ] 两边镜像都构建完了
- [ ] 备份做了
- [ ] 决定好走哪条路:先并行试跑(第四节,推荐),还是直接切(第五节)
## 四、并行试跑oj2.xuyue.cc
正式切换之前,先让新栈挂在一个独立域名上跑几天,**旧站原样不动、一个容器都不用停**。
这是双跑 —— 设计文档原本明确否掉过(「不双跑不灰度」)。改主意的理由是:
「只换前后端」形态下新栈本来就不碰数据库进程,双跑的增量风险只剩「两个后端同时写同一个库」
这一条;而换来的是**正式切换退化成改一行 NPM 上游**,反而比停机切换更稳。
代价见下面「试跑期间要知道的」,不是零。
### 1. env 比「只换前后端」多两个
```
WEB_PORT=8090
JUDGE_STATE_DIR=/root/OJDeploy/data/judge_server_oj2
```
`WEB_PORT` 是因为 8080 还被旧 backend 占着。`JUDGE_STATE_DIR` 是因为**两个判题机不能
共用运行目录** —— 不设的话新旧两个 judger 会同时往 `data/judge_server/{run,log}` 里写。
`test_case``public/upload` 仍然共享,**那是故意的**:测试点和题面图片两边必须看到同一份。
### 2. 起
有脚本,**在服务器上**跑:
```bash
cd /root/OJDeploy/OJ2
docker/deploy.sh # 自检 → 构建 → 起栈 → 冒烟
docker/deploy.sh --check # 只自检,只读,不动任何容器
docker/deploy.sh --no-build # 只改了 env / compose 时跳过构建
```
代码怎么上到服务器不归它管rsync 或以后的 `git pull`,命令在脚本头部注释里)。
起栈前有五道自检,前两道正是这次在服务器上真撞到的:
| 自检 | 拦什么 |
|---|---|
| compose 版本 | `depends_on.required` 要 ≥ 2.20,老版本解析就会失败 |
| `DATA_DIR` | 卷指向 `OJ2/data` → 中止(空数据,静默) |
| `DB_HOST` | `DATABASE_URL` 还指着 `oj-postgres` → 中止(试跑形态下它不存在) |
| `JUDGE_STATE_DIR` | 没设的话新旧两个判题机共用运行目录 |
| 旧栈 | `oj-postgres` / `oj-redis` 得还活着,新栈连的就是它们 |
起完等 `oj-api` healthy再跑四条冒烟题目数是 0 也中止 —— 那意味着连错库了。
手动等价于:
```bash
mkdir -p /root/OJDeploy/data/judge_server_oj2/log /root/OJDeploy/data/judge_server_oj2/run
cd /root/OJDeploy
docker compose -f OJ2/docker/compose.debian.yml --env-file OJ2/docker/.env up -d
curl -s -o /dev/null -w '%{http_code}\n' http://localhost:8180/ # 期望 200
```
旧站这时候完全没受影响8080 上照常服务。
### 3. NPM 加一台 proxy host
| 项 | 值 |
|---|---|
| Domain | `oj2.xuyue.cc`(先把 DNS 解析加上) |
| Forward | `<宿主机 IP>` : `8090` |
| **Websockets Support** | **必须打开** |
| SSL | 签一张证书,`COOKIE_SECURE=true` 依赖 https |
| client_max_body_size | `200M`,和 Caddyfile 里的 `200MiB` 对齐(上传测试用例压缩包) |
WebSocket 那个开关是双跑最容易漏的一格:漏了的话页面一切正常,唯独学生盯着的
「判题中…」永远不动 —— 而这恰恰是最不容易在自测里发现的一条,因为刷新一下结果就出来了。
### 4. 试跑期间要知道的
- **两边登录态不互通**。旧站是 Django session新站是 Redis opaque token。
学生到 oj2 要重新登录一次,这不是 bug。
- **在 oj2 登录过、注册的、被改过或重置过密码的账号,哈希会变成 argon2
回旧站就登不上了** —— 试跑第一天真撞过,现象是旧站登录失败 + WS 连不上。
2026-08-26 起这是无条件行为(`PASSWORD_HASH_UPGRADE` 开关已删,见上面那条
补充说明为什么它本来就拦不住)。修复见下面的「万一已经改坏了」。
- **同一个库,双写**。结构完全兼容不会写坏,但提交、统计、成就都是**真实数据**
不是沙盒。别拿它做破坏性试验。
- 后台判题机列表会出现**两台**(新旧各自心跳),正常。
- 比赛排名等缓存两边各存各的 Redis可能短暂不一致。
**试跑期间不要在 oj2 上办正式比赛。**
### 5. 试跑要盯的是这些
都是只有真实数据 + 真实浏览器才暴露的:
- 机房那种 **Chrome < 94** 打开正常(`mermaid-legacy` 这条 fallback 只在老浏览器上生效)
- 带图片的题面(`/public/upload/*` 走的是反代,不是 Caddy 直接读盘)
- AI 分析的 **SSE 流式输出**经过 NPM 之后还是不是逐块下发(这一层最容易被缓冲住)
- WebSocket 在 NPM 后面长时间挂着稳不稳(超时断连会不会自动重连)
- 后台**上传测试用例压缩包**这条 200MB 的路
- 老师日常用的后台:出题、建比赛、题单
## 五、正式切换
**两个站点必须同一天切。** 机房那套连的是服务器的库,只切一边的话,另一边的旧后端
还在读写同一个库。
### 试跑过了 —— 那就只是改一行上游
新栈已经在 8090 上跑着、验过了,切换不需要重启任何容器:
1. NPM 里把 `xuyue.cc` 那台 proxy host 的上游从 `8080` 改成 `8090`**Websockets Support 同样要开**
2. 看一眼首页和一次提交,正常
3. 确认没问题之后再停旧栈:`docker compose -f docker-compose.yml stop oj-backend oj-judge`
**停机时间约等于零**,回滚就是把 NPM 那一行改回 `8080` —— 旧栈这时候都还没停。
这正是设计文档最初写的那个回滚故事:「改一行上游」。
`oj2.xuyue.cc` 可以留着,它和 `xuyue.cc` 指向同一个新栈,没有坏处。
### 没试跑、直接切 —— 走下面这套
服务器(`/root/OJDeploy`
```bash
cd /root/OJDeploy
# 只停应用postgres / redis 留着继续跑 —— 新栈接着用它们
docker compose -f docker-compose.yml stop oj-backend oj-judge
docker compose -f docker-compose.yml ps # 确认 oj-postgres / oj-redis 还在 Up
docker compose -f OJ2/docker/compose.debian.yml --env-file OJ2/docker/.env up -d
docker compose -f OJ2/docker/compose.debian.yml --env-file OJ2/docker/.env ps
# 应该正好 4 个oj-api / oj-worker / oj-web / oj-judge等 oj-api 变 healthy
```
旧判题机必须一起停:它和新判题机会争同一个 `data/judge_server/run`
旧 backend 也必须停8080 端口要交给 `oj-web`
机房:
```bash
cd <机房部署目录>
docker compose -f docker-compose.yml stop oj-backend oj-judge
docker compose -f OJ2/docker/compose.school.yml --env-file OJ2/docker/.env.school up -d
docker compose -f OJ2/docker/compose.school.yml --env-file OJ2/docker/.env.school ps
```
(机房本来就没有 postgres它的 redis 是新栈自带的,和旧 redis 不冲突 ——
旧的那个没往宿主机发布端口。)
起不来先看这两条日志,绝大多数问题在里面直说了:
```bash
docker logs oj-api --tail 50 # 连不上库、token 不对都在这里
docker logs oj-judge --tail 20 # 判题机注册不上看这条
```
## 六、切换后验证
这一节试跑刚起来时也照着跑一遍,只是端口换成试跑用的(`WEB_PORT`,例如 8090
先用命令快速过一遍(服务器 8080机房 81
```bash
BASE=http://localhost:8080
curl -s -o /dev/null -w '首页 %{http_code}\n' $BASE/
curl -s -o /dev/null -w '站点配置 %{http_code}\n' $BASE/api/site
curl -s -o /dev/null -w '题目列表 %{http_code}\n' $BASE/api/problems
curl -s -o /dev/null -w '未登录进后台 %{http_code}\n' $BASE/api/admin/dashboard # 期望 401
```
前三条 200、第四条 401 才算过。两个失败模式各有各的症状,别搞混:
- **题目列表 `"total":0`** → 连错库了(`DB_HOST` 没设,或误加了 `--profile local-data`
起了个自带的空 postgres。立刻停下来查别往下走。
- **库是对的,但判题全错、题面图片 404** → `DATA_DIR` 指错了,挂上去一堆空目录。
然后照着点一遍 —— 下面这几步是命令测不到的:
1. 打开首页,能看到题目列表
2. 用一个学生账号登录,看得到自己的提交历史
3. 提交一道题,**看判题结果是否实时刷出来**(这一步同时验证了 WebSocket
4. 后台 → 判题机列表,确认判题机在线(心跳走 `/api/judge-server/heartbeat`
5. 后台 → 题目列表能翻页
6. 题面里带图片的题,图片能显示(`/public/upload/*`
机房那边额外确认一条:**登录之后刷新页面还是登录态**。如果「登录成功又立刻变未登录」,
就是 `COOKIE_SECURE` 没设成 false。
## 七、回滚
> ⚠️ **执行 `0002_drop_django_leftovers` 之后本节作废**,理由见上面「回滚保证」一节:
> 旧 Django 后端的表被删除后旧栈起不来,退路只剩「从备份恢复数据库」,不是秒级操作。
> 下面内容保留作历史记录。
**试跑之后切的**推荐路径NPM 里把 `xuyue.cc` 的上游从 `8090` 改回 `8080`
旧栈这时候还跑着,**秒级生效,什么都不用停不用起**。等确认稳定了再决定何时收掉新栈。
**直接切的**
```bash
cd /root/OJDeploy
docker compose -f OJ2/docker/compose.debian.yml --env-file OJ2/docker/.env down
docker compose -f docker-compose.yml start oj-backend oj-judge
```
机房同理,换成 `compose.school.yml`
约 20 秒,比演练时还快一点 —— **postgres / redis 全程没停过**,回滚只是把旧的
backend 和判题机再 start 起来。**不需要恢复数据库,不需要动任何文件。**
这也是「只换前后端」形态的主要好处:切换和回滚都不碰数据库进程,
库出问题的可能性从流程里被整个拿掉了。
### ⚠️ 回滚要额外处理密码
「不动任何数据」不适用于 `user.password` 这一列。在新站登录过、注册的、被老师
改过或重置过密码的账号,哈希已经是 argon2旧后端验不了 —— 回滚之后那些学生
登不上。2026-08-26 起这是无条件行为,`PASSWORD_HASH_UPGRADE` 开关已删。)
所以回滚流程里**必须**加一步:按下面那节把 argon2 的账号用 `raw_password`
重算回 Django 的 pbkdf2。
### 万一已经改坏了
先看范围(`$argon2` 开头的就是被改过的):
```bash
docker exec oj-postgres psql -U onlinejudge -d onlinejudge -c \
"select id, username, raw_password is not null as 有明文 from \"user\" where password like '\$argon2%'"
```
用**旧后端自己**改回 pbkdf2 —— `raw_password` 那个明文列(老师查学生密码用的)
正好派上用场:
```bash
docker exec -i <旧 backend 容器> python manage.py shell <<'PYEOF'
from django.contrib.auth.hashers import make_password
from account.models import User
for u in User.objects.filter(password__startswith='$argon2'):
if u.raw_password:
u.password = make_password(u.raw_password)
u.save(update_fields=['password'])
print('已修复', u.username)
else:
print('没有明文密码,需要手动改:', u.username)
PYEOF
```
没有 `raw_password` 的(多半是超管账号)用 `python manage.py changepassword <用户名>`
改回 pbkdf2 之后两边都认:新后端本来就支持 Django 的 pbkdf2。
---
## 八、演练中踩到的坑(写下来是因为它们只在容器里出现)
### 1. pg_dumpall 备份会覆盖数据库口令 ⚠️
演练时恢复完快照,新后端立刻报:
```
PostgresError: password authentication failed for user "onlinejudge"
```
原因:`pg_dumpall` 的集群备份里带
```sql
ALTER ROLE onlinejudge WITH SUPERUSER ... PASSWORD 'md5……';
```
**恢复这份备份,会把角色口令覆盖成备份时生产的那个口令**compose 里的
`POSTGRES_PASSWORD` 就对不上了。
- 正常切换:**不受影响**,因为根本不恢复备份。
- 灾难恢复(真要从备份重建):恢复之后要么把 `POSTGRES_PASSWORD` 设成生产的口令,
要么恢复后手动 `ALTER ROLE onlinejudge PASSWORD '<新口令>'`
这一条不写下来,恢复现场会被一个看起来毫不相干的报错卡住。
(另记:该哈希是 md5PG16 默认已是 scram-sha-256。不影响切换但值得择日换掉。
### 2. 恢复备份前必须先停应用
应用连着库时dump 里的 `DROP DATABASE` 会失败:
```
ERROR: database "onlinejudge" is being accessed by other users
```
演练时因为目标库本来是空的,数据照样灌进去了 —— 那是运气。目标库有数据的话,
接下来就是满屏主键冲突。灾难恢复流程:**先停 oj-api / oj-worker再恢复。**
### 3. 「本地能过、容器里过不了」的四个坑(构建期)
都已修好并写进 Dockerfile 的注释,这里只留索引:
- 构建上下文吸进 `data/`,判题沙箱用别的 uid 建的目录 docker 连 stat 都做不了 → `.dockerignore`
- `mermaid@9.4.3`(机房老 Chrome 的 legacy 依赖)从容器里连 npmjs 稳定失败 → 换 npmmirror + 重试
- 容器里 bun 用 isolated 布局、本地是扁平的,靠「提升」解析的包在容器里一律找不到
→ 把真正直接 import 的 4 个包补成直接依赖
- **`apt-get update` 在服务器上卡死**2026-08-16 试跑时撞上,本机构建从来没事)
→ 换清华源。两个细节trixie 的源是 deb822 格式、在
`/etc/apt/sources.list.d/debian.sources`(老的 `sources.list` 在这个基底里是空文件);
**只能换主机名、必须保持 http** —— ca-certificates 正是这一步要装的,
换成 https 会在没有根证书的情况下证书校验失败。`ARG APT_MIRROR` 可覆盖。
---
## 九、镜像体积(没达到设计文档的预期,说明原因)
| 镜像 | 体积 |
|---|---|
| `oj2-api` | **487MB** |
| `oj2-web` | 75MB |
设计文档写的是「降至数十 MB」**没做到**。拆开看:
| 层 | 体积 |
|---|---|
| `clang-format`apt | **269MB**,其中 `libLLVM.so.19.1` 一个 124MB |
| `oj2-api` 二进制 | 112MBBun 运行时 + 内嵌的 wasm/原生模块/4.8MB 词典) |
| `ruff` | 28MB |
| debian-trixie-slim 基底 | 79MB |
也就是说**一半以上是 clang-format 拖进来的 LLVM**。旧的 Python 镜像同样装了
clang-format再加整个 Python 运行时和 Django 依赖,所以新镜像仍然明显更小,
但「数十 MB」是当初没把 clang-format 算进去。
想再瘦下来只有一条路:换成静态链接的 clang-format 独立二进制PyPI 的
`clang-format` wheel 里就是),能砍掉约 265MB。没做因为镜像是各站点本地构建的、
不走镜像仓库,磁盘不是瓶颈;等哪天真嫌大了再说。
---
## 十、演练产生的临时数据(已清理)
演练在 `OJ2/data/postgres` 下留下了**一份完整的生产数据副本**,其中包含 1710 名
学生的 `raw_password` 明文列。演练结束后已删除该目录。
以后再演练记得同样处理 —— 那不是测试数据,是真实学生数据。