7 Commits

Include renames

Author	SHA1	Message	Date
xuyueandClaude Opus 5	64facc5701	fix(WebSocket): 断线不重连、评分结果会丢、登出后连接还活着 ... 排查 WS 这一块时发现的一批问题，多数是「机制写了但从没生效过」。 ## 重连 `disconnect()` 里 `enableAutoReconnect = false`，而 `connect()` 从不改回 true —— 登出再登录后，这条连接就永远失去了自动重连能力（configUpdate 那条 watch 上尤其 明显）。改成用 `closedByUser` 表达「用户主动断开」的意图，和 `enableAutoReconnect` 这个**配置**分开。 `scheduleDisconnect` 的回调里断完紧接着一句 `enableAutoReconnect = true`，而 close 是异步的 —— 等 onclose 跑到时标志已经翻回来了，1 秒后又自动连上。那个 「15 分钟空闲省资源」从来没真正断开过。现在只断开，不做事后翻转。 重连的 setTimeout 没存句柄，组件卸载后照样触发 `connect()`，在已销毁的组件上 又建一条连接。现在 `disconnect()` 里 clearTimeout。 退避从「线性 ×5 次」改成「指数 + 抖动、30 秒封顶、次数不封顶」。原来 1+2+3+4+5 只有 15 秒，后端 deploy 重启一次就超了，之后这条连接死到用户刷新为止。抖动是 为了避免一个班几十台机器在同一毫秒一起冲回刚起来的后端。另挂 online / visibilitychange，网络恢复或切回标签页立刻重连，不必等退避走完。 所有 socket 回调改成闭包住局部 ws 并在入口 `if (ws !== this.ws) return`，旧连接 迟到的 onclose 不再污染新连接的状态 —— 也是让 `disconnect()` 能被 onclose 识别 出来的关键。 ## 订阅重放 `pendingSubmissionId` 一发送成功就清空，它只解决了「还没连上就 subscribe」， **没解决断线重连**。而真正会丢结果的恰恰是后者：服务端收到 subscribe 会回一份 当前状态，掉线期间错过的推送就是靠这次重放补回来的；不重新订阅，重连后只收得到 「将来」的事件，可结果已经是过去式了。改成订阅意图保留到显式 `unsubscribe()`， 每次 onConnected 都重发。 这套逻辑原来只有 SubmissionWebSocket 有，FlowchartWebSocket 是 send 失败打一行 日志了事 —— socket 一掉，那次评分结果就再也回不来，按钮一直转圈。提成公共基类 SubscribingWebSocket，两条通道共用。 流程图另加轮询兜底：提交后 5 秒 WS 还没出结果就每 3 秒拉一次，读 status 2/3 结算，3 分钟上限。判题那边一直有兜底，流程图这边没有，而 Redis pub/sub 是发完 不管的，worker 推的那一刻连接不在就永远丢了。 `useSubmissionMonitor` 里 `watch(wsStatus, ..., { immediate: true })` 的回调在 watch() **返回之前**就同步跑了，已经连着时 `unwatch` 还是 null，if 不成立， watcher 永远停不掉：每提交一次泄漏一个，往后每次重连它们都会把各自那个早就判完 的旧 submissionId 重新订阅一遍。整块删掉，直接 subscribe —— 基类已经管了时序。 WS handler 原来不校验 submissionId。学生同时开着几道题的页面时，每条连接都订在 同一个用户 topic 上，别的页面的评分结果会被当成自己的。 ## 会话 握手时校验过一次会话就再也不管了，这条连接却能挂几个小时：用户在别的标签页 登出、或者会话本身到期，旧 socket 照样收推送。加一条 60 秒一轮的巡检，用 Redis EXPIRE 一条命令同时完成「判断存在」和「续期」（续期是必要的：只开着页面 挂 WS 的人一次 HTTP 请求都不发，不该被算成不活跃踢下线）。Redis 抛错时整轮 放弃，绝不因为一次抖动把全班踢下线。 禁用只改数据库的 isDisabled 列、不动 Redis 里的会话，巡检永远发现不了。加 `session:revoked` 频道主动通知，**两种作用域不能混**： { token } 用户登出。只断这一张会话 —— 同一个人在别的设备上是另一张 会话，按 userId 广播会把他手机上的登录一起踢掉 { userId } 账号被禁用。所有设备都得断 先发一帧 force_logout 再隔 100ms 断开。只断不发的话前端只看到一次普通掉线， 会照常重连、页面上还显示着登录态。token 不进帧里 —— 那是 httpOnly cookie 的值， 推到 WS 上就等于交给了 JS，匹配全在服务端做。 前端在协议层拦截 force_logout（和 pong 一样，不下发给业务 handler），并主动 disconnect —— 否则会一路 401 重连到退避上限，正是这机制要消掉的浪费。表现刻意 和 utils/api.ts 里 account-disabled / login-required 两支保持一致：同一件事从 HTTP 和 WS 两条路进来，学生看到的不该有两个样子。 ## 开销与安全 `void handleMessage(...)` 是裸的，里面有两次 DB 查询和一个会抛的 schema.parse， 库抖一下就是一个 unhandled rejection（隔壁 bridgeSubmissionEvents 两处都接住了， 只有这里漏了）。 ping 提到用户查询之前。原来的顺序是「先查 user 再看消息类型」，每个客户端每 30 秒都要为一次心跳打一趟数据库。禁用用户不会因此漏网：推送路径上 bridge 会查， subscribe 这条真正读数据的路径下面照样查。 bridge 两条 per-user 通道都先看 `server.subscriberCount(topic)`，没人订阅就别 查库了 —— 判题高峰期绝大多数事件的目标用户此刻并不在线。 flowchart 评分失败原来把 error.message 原样推给学生、前端直接弹出来，AI provider 的地址和内部报错就这么进了浏览器。改成真实原因写服务端日志。 加每连接令牌桶（20 突发 + 每秒回填 2）。一条 subscribe 在服务端是一到两次数据库 查询，一个学生开着一条 socket 狂发就能压住库。正常流量离阈值几十倍远。 升级时校验 Origin。会话 cookie 是 SameSite=Lax、WS 握手不是导航，跨站页面本来就 带不上 cookie，所以这是防御纵深不是唯一防线。同源放行；本机开发（Vite 5173 → API 3000）自动放行，且只在两边都是本机时成立 —— 生产环境 url.hostname 是正式 域名，这条永远不触发；跨域部署走 ALLOWED_WS_ORIGINS。不发 Origin 的一律放行： 真正的攻击面是带着受害者 cookie 的浏览器页面，而浏览器一定会带 Origin。 顺带：useConfigWebSocket 的 handler 从 onMounted 挪到同步注册（调用方在 setup 阶段就 connect() 了），删掉每条消息打完整内容的 console.log 和死字段 ws.data.username。 ## 没动的 题目页上并没有两条 /ws/submissions —— Form.vue 里 SubmitFlowchart 和 SubmitCode 是 v-if/v-else，互斥。学生实际是 2 条连接：全站一条 /ws/config + 一条 /ws/submissions，正常，不必合并。 ## 验证 55 个用例，分六组打桩跑（假 WebSocket + 假计时器；会话/限流/吊销三组对着真 Redis）： 重连语义 7 断开后不再自我复活、卸载后计时器已取消、退避封顶、 online 立即重连、旧 onclose 不污染新连接 订阅重放 9 未就绪时补发、重连后重新订阅、unsubscribe 后不再重放 会话巡检 8 EXPIRE 三态、只断失效的、同 token 只查一次、 Redis 抖动时一个都不踢 Origin/限流 13 跨站与跨端口拒绝、生产不因 localhost 开后门、 突发额度、按时间回填 强制登出 13 登出只断同 token（别的设备不受牵连）、禁用断所有设备、 巡检先通知再断 前端登出 5 两支表现、收到后不再重连、两条通道只处理一次 前两组做了改动前/后对比，老代码该挂的都挂了 —— 「重连后自动重新订阅」正是这么 跑出来的，此前我以为 pendingSubmissionId 已经覆盖了这种情况。 apps/api 的 tsc 和 apps/web 的 vue-tsc 都干净，仓库既有测试照常通过。 **SubmitFlowchart.vue 的轮询兜底未经运行时验证** —— 在 SFC 内，没搭组件挂载 环境，只过了类型检查和人工核对。要验的话，停掉 worker 提交一次流程图，看 5 秒 后是否转入轮询、3 分钟后是否给出超时提示。 这批改动动了 WS 的行为面（限流会断连接、Origin 会拒绝、巡检会踢会话），上线前 建议手测：提交代码看判题、切流程图看评分、后台改配置看全站生效、开两个标签页 在一个里登出、禁用一个在线学生看另一端反应。 Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>	2026-08-27 05:07:26 -06:00
xuyueandClaude Opus 5	dd6a6a04eb	refactor(密码): 删掉 PASSWORD_HASH_UPGRADE，无条件写 argon2 ... 上一个 commit 把五个写入点收进 hashPassword，顺手把开关默认值翻成了 true 并 留下 false 分支当退路。这个退路是假的，删掉。 **开关只能拦住将来，修不了已经发生的。** 等真到了要把旧站拉回来的那天，活跃 账号早就都被登录/重置密码升成 argon2 了，这时候设 `=false` 一个也救不回来。 正经退路是切换手册「万一已经改坏了」那节的脚本 —— 拿 `raw_password` 重算 Django 的 `make_password`，能修已经变成 argon2 的账号。留着一个永远轮不到它 上场的开关，只是给后来人多一件要读懂的东西。 删掉的：`config.passwordHashUpgrade`、`hashDjangoPbkdf2` 和它那套 Django 参数 （1200000 迭代 / 22 位 salt / RANDOM_STRING_CHARS）、两个 compose 里的 `PASSWORD_HASH_UPGRADE` env。auth.ts 的登录升级变成无条件。 保留的： - `hashPassword()` —— 虽然现在只是一行 argon2，但「只有一个地方写密码」正是上 一个 commit 的重点。五处各写各的才是当初那个 bug 的根。 - `verifyPassword` 的 **pbkdf2 分支** —— 生产库 1710 个账号全是 Django 写的 pbkdf2（迭代次数 120000～1200000），只会在各自下次登录时才迁移。删了就是 全站登不上。代码注释里写死了这句话。 - 切换手册里的 `raw_password` 恢复脚本，以及回滚流程里新加的「密码要额外处理」 那一步。 ## 验证 改完重跑了一遍关键路径（临时造一个 Django 生成的 120000 迭代老哈希）： - 存量账号登录 200 → 哈希变成 argon2 → 再登 200 → 错密码 401 - 后台重置密码 → argon2 → 新密码能登录 - 注册 → argon2 → 能登录 tsc(apps/api) 0 error、check:routes 168 条无遮蔽、vue-tsc 0 error、build 通过、 两个 compose `config -q` 通过。测试用户已删干净。 Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>	2026-08-25 23:50:55 -06:00
xuyueandClaude Opus 5	694e147a21	fix(密码): 五个写入点统一走 hashPassword，默认切 argon2 ... ## 那道「单向门」其实一直开着 `PASSWORD_HASH_UPGRADE` 是并行试跑第一天撞出来的补丁：新后端把 Django 的 pbkdf2 升级成 argon2 之后，旧后端验不了，登录过新站的学生回滚就登不上 （phase5 切换手册里记着这件事）。当时的修法是给**登录时的自动升级**加个开关， 默认关闭。 但写密码的地方有五个，开关只管住了一个： POST /users 注册 PUT /admin/users/:id 管理员改密码 POST /admin/users 批量导入用户 POST /admin/users/:id/reset-password 重置密码 ← 老师天天在用 登录成功后的自动升级 ← 只有这一处受开关管 后面四处无条件 `Bun.password.hash(argon2id)`。也就是说开关关得好好的， **老师给学生点一次「重置密码」，那个账号就回不去旧站了** —— 而「老师帮学生 查/改密码」正是这套系统的日常功能，`raw_password` 那一列存在的理由就是它。 所以那半年里「默认关闭 = 回滚安全」是个假象。 ## 改法 五处统一走 `auth/password.ts` 的 `hashPassword()`，开关两个方向都变成真的： - `true`：五处全写 argon2id，登录时把存量 pbkdf2 顺手升级掉。 - `false`：五处全写 Django 格式的 `pbkdf2_sha256$1200000$<22位salt>$<base64>`， 登录时也不动存量哈希 —— 两边都验得了。 新加的 `hashDjangoPbkdf2` 逐项对齐 Django 6 的 `PBKDF2PasswordHasher`： 1200000 迭代、22 位 salt、`RANDOM_STRING_CHARS` 字符集、sha256/32 字节。 **默认值定成 `true`** —— 旧站今天下线了，没有回滚路径要照顾。要把旧站拉回来 就先设 `PASSWORD_HASH_UPGRADE=false`，切换手册三处说明都改了。 ⚠️ `verifyPassword` 的 pbkdf2 分支**永远不能删**，代码和注释里都写了：生产库 1710 个账号全是 Django 写的 pbkdf2（迭代次数 120000～1200000，跨了好几个 Django 版本），它们只会在各自下次登录时才升级成 argon2。 ## 顺带：dev seed 只有学生号 `seed:dev` 只建 `student`（普通用户），本机想测后台得手工往库里塞 email 和 user_profile —— 而缺 user_profile 的表现极其隐蔽：`/api/me` 返回 profile-not-found → 前端 getMyProfile 抛异常 → localStorage 的 authed 存不进去 → **所有 /admin 路由被守卫静默弹回首页**，不报错。我在这上面卡了很久。 现在 seed 同时建学生和超管（`devadmin` / `devadmin123`），profile 和 email 一起 建好，并且写密码也走 hashPassword。另外加了一道防呆：DATABASE_URL 不是本机时 直接拒绝执行 —— 这个脚本会重置密码并把明文写进 raw_password，其中一个还是超管， 对着生产库跑一次就是把超管密码改掉。要绕过设 `OJ2_SEED_FORCE=true`。 ## 验证 全程拿**旧后端那个真的 Django venv** 对打，不是照着文档推： - 事实核对：Bun 写的 `$argon2id$…` 在 Django 里 `identify_hasher` 抛 `Unknown password hashing algorithm ''`；换成 Django 格式 `argon2$argon2id$…` 能识别，但 `verify()` 抛 `Couldn't load 'Argon2PasswordHasher' algorithm library: No module named 'argon2'`（旧后端确实没装 argon2-cffi）。 原注释说的「格式对了也验不了」结论对，机制略有出入。 - 双向互验：OJ2 写的哈希 Django `check_password` 通过、错密码不通过； Django `make_password` 写的哈希 OJ2 `verifyPassword` 也认。 - 默认（argon2）：拿 Django 现造一个 120000 迭代的老哈希塞进库，登录 200 → 哈希变成 argon2 → 再登一次仍 200。 - 退路（`=false`）：同一个老哈希登录 200 且**哈希不变**；走后台「重置密码」 之后落库是 `pbkdf2_sha256$1200000$…`，**旧站的 Django 验这个新密码通过**。 - 一次 1200000 迭代约 107ms（写密码时的开销；验旧哈希的开销本来就在）。 - tsc(apps/api) 0 error、check:routes 168 条无遮蔽、vue-tsc 0 error、build 通过。 测试用户（pwtest1 / legacyuser）已删干净，本机库用新 seed 复位。 Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>	2026-08-25 23:47:02 -06:00
xuyueandClaude Opus 5	c2a81209d7	refactor(前端): 拆掉 camelCase→snake_case 转换层，契约成为唯一真相 ... utils/legacy.ts 是迁移期的临时层：新后端一律 camelCase，而组件读的还是 旧 Django 的 snake_case，于是在 api 层做一次递归键名重写。它自己的注释就 写了「迁移完成后这一层应当整体拆掉」。现在拆了。 代价不只是那 96 处包装：每个响应都要递归遍历整个对象重写一遍键名，而且 utils/types.ts 和 packages/contract 是两份真相 —— 手抄的那份还抄歪了好几处。 做法是按域推进，每域都用 vue-tsc 相对基线做差，确认零新增错误后再往下走。 前端的类型现在一律以契约为准，只在必要处窄化（比如 languages/template 的键 窄化成 LANGUAGE），删掉的重复定义包括 WebsiteConfig、LoginSummary、 AchievementSummary、ProblemSet、Contest、User、Profile、AdminTag、 StuckProblem 等等，其中 ClassComparison 有两个组件各手抄了一份。 ## 顺带修掉的真 bug - 管理端公告列表的「可见」开关每次都 400：列表响应被契约 omit 掉了 content， 而更新接口要求 content 必填，toggleVisible 把列表行原样回传。而且是乐观 翻转、不 await 不 catch，管理员看到开关动了、实际没存也没有提示。 改成先 GET 整条再 PUT，加失败提示。 - 删有提交的题时只显示笼统的「删除失败」：前端还在 match 旧 Django 的英文 文案，而后端返回的是 problem-has-submissions + 中文。连同另外 8 处同类 匹配一起改成判错误码 —— 文案是后端随时能改的，match 文案改一个字就静默失效。 - SubmissionStatus.time_limit_exceeded 写成 `1 | 2`，TS 按位或算成 3，和 memory_limit_exceeded 撞了同一个值。后端 judge/status.ts 里这是分开的 两个码，按后端拆成 cpu_/real_ 两项。当前没有代码读这两个成员，但 CLAUDE.md 明确要求判题状态码三处同步。 - 流程图历史翻到没有提交的那一页会直接抛：契约里 submission 是 nullable， 被 any 掩盖成看起来非空。补了 null 分支。 ## 契约里被逼出来的三处不诚实 - grade 写成 z.string()，但 averageGrade() 在没有可用数据时返回空串， 前端三张图表拿它查 Record<Grade,...> 会查出 undefined。按实际收紧成 z.enum([...,""])，四个查表点都补了「无评级」分支。 - difficulty 写成 z.string()。核对过生产库 dump：956 道题只有 Low/Mid/High 三个值（761/149/46）。收紧成枚举。 - topReaction 写成 z.string()，既对不上前端渲染的 {type,count}，也对不上 旧后端 get_top_reactions 下发的形状。改成正确形状并注明当前恒传 null。 ## 明确保留 snake_case 的 54 处 判题沙箱原始输出（cpu_time/exit_code/output_md5/compile_output）、 statistic_info 内容（err_info/time_cost/ast_results）、submission_info JSONB（is_ac/ac_time/error_number，回滚时旧后端还要读）、SQL 判题引擎的 total_rows/order_sensitive/changed_tables、WebSocket 的 submission_id、 以及数据库选项键 enable_maxkb。每一处都在类型定义旁写了为什么不能改。 language 没有跟着收紧契约 —— 它是配置项、随时可能加语言，收紧会让新语言 在后端 parse 时直接抛。改在 api 边界一处窄化。 ## 另外 - utils/http.ts 整个模块已是死代码（四处引用全是 import type），删除。 - profile 的 blog/github/school/major/language 五个字段全链路空转，没有 任何组件读，从契约到类型一并摘除（数据库列不动）。 - admin/account.ts 往 user_profile 塞的 totalScore 是 OI 模式遗留，表里 没这一列。Drizzle 按表定义拼列名会把它静默丢弃，所以没出过错，是死代码。 验证：vue-tsc 143 → 54 条且无新增，apps/api tsc、check:routes、web build 全通过；各域响应形状逐条打接口核对过。 Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>	2026-08-25 13:44:24 -06:00
xuyueandClaude Opus 5	face92a5dc	fix(登录): 默认不再把 Django 的 pbkdf2 升级成 argon2 —— 这是道单向门 ... 试跑第一天就出事了：在 oj2 上登录过的账号，回旧站 oj.xuyue.cc 登不上，WS 也连不上 （旧站的 Channels 认 session，没登录自然连不上，两个症状同一个根因）。 `routes/auth.ts` 原来在登录成功后把 pbkdf2 哈希重算成 argon2id 写回库。问题是： - Bun 写出来的是 `$argon2id$v=19$…`，Django 存 argon2 是 `argon2$argon2id$v=19$…` （多一个算法标签、没有开头的 `$`）。Django 按 `$` 切第一段当算法名，拿到空串，认不出。 - 更彻底的一点：旧后端的 pyproject.toml 里**根本没有 argon2-cffi**， 格式对了也验不了。 所以这不只是双跑的问题 —— 一次性切换后的回滚窗口内同样成立：学生登录过新站， 回滚之后就登不上了。演练时比对的是表结构「逐列一致」，比不出这种语义上的单向门。 改成 `config.passwordHashUpgrade` 控制，**默认关闭**，并在两份 compose 的 environment 里显式声明 `PASSWORD_HASH_UPGRADE`（原来根本没透传，想开也开不了）。 等确定不会再回滚了再打开。 ## 验证（真库真容器，不是看代码） schema.sql 起库 + 造一个 Django 格式的 pbkdf2 用户（密码已知），跑新栈实测： - 默认形态：登录 200，库里的哈希**仍是** `pbkdf2_sha256$260000$…`，容器里 PASSWORD_HASH_UPGRADE 为空 - 对照组 PASSWORD_HASH_UPGRADE=true：登录 200，哈希变成 `$argon2id$v=19$m=655…` —— 正是生产库现在的样子，锁死可复现 Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>	2026-08-16 09:57:43 -06:00
xuyueandClaude Opus 5	8c00cdc947	feat(阶段3): oj 侧端点铺开（基线提交，未经评审） ... 由外部 agent (Codex) 在本会话额度中断期间完成。原样提交作为基线， 后续修复单独成 commit，便于区分与回退。 覆盖 oj 侧 65 个端点，新增 9 组路由（account/achievement/ai/classroom/ content/contest/flowchart/problemset/site）与对应 Zod 契约。 已核验：tsc --noEmit 退出码 0；API 可启动；/api/problems 返回真实数据； judge 与 flowchart worker 均 ready。 未核验：权限边界与数据泄露，评审进行中。 Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>	2026-08-07 01:25:36 -06:00
xuyue	ec274419c3	Build Phase 2 judge vertical slice	2026-08-06 22:42:39 -06:00