Repository navigation
fix: 发布后全量审查修复批次(12 项) - #102
Merged
Merged
Conversation
added 4 commits
October 31, 2026 08:36
- data-submit-once 多提交按钮表单:禁用字段不进 FormData,处理器禁用第一个按钮(=暂停提交器)导致请求丢失 action 字段、批量暂停必然「未知操作」;恢复/触发则双击重复提交——改为先把 event.submitter 的 name/value 写入隐藏域再禁用全部按钮 - interactionOpen 查 .cron-editor:not([hidden]),而 hidden 在父行 tr 上——可操作模式恒判「交互中」,自动刷新两条路径全短路;改为查 .cron-row 并新增焦点在输入控件内不刷新守卫 - Cron 预览过期响应竞态:请求序号,慢的旧响应不再覆盖新表达式的预览 - flash dismiss 后 document 级 visibilitychange 监听以 AbortController 解绑
- /api/tasks ETag 改为排序后 (JobId, UpdatedAt.Ticks) 哈希 + 分页参数:旧「数量+秒级截断」指纹在同一秒连续两次写入时误命中 304(客户端拿过期缓存),且不同分页共用 ETag;304 分支补 X-Total-Count - ReadOnlyGuardFilter 放行 /login /logout /api/auth——只读模式此前连登录提交与登出都被 403 - /error 按状态段给语义标题:401/403/429 不再被标成「服务器内部错误」 - 机器客户端 401 补 WWW-Authenticate: Basic realm="TaskRunner"(与 Dashboard/openapi 守卫一致)
- RunUnderBootstrapLockAsync 锁等待受默认 30s CommandTimeout 限制:并发首启遇长迁移超时崩溃而非等待——锁获取/持有期间 CommandTimeout=0,finally 恢复 - 滚动升级版本偏斜防护:Api 新版+Worker 旧版并行重启时旧进程发现集不含新任务,立即删除会误删配置并摘 recurring——RemoveMissingAsync 增加 10 分钟宽限期;prune 保留清单并入库中现存行 - PlatformSelfCheckJob 探针写删之间异常留 .self-check-*.tmp 永不回收——finally 兜底清理 - LogCleanupJob 单文件被占用/只读抛异常中断整轮清扫且统计失真——逐文件容错,汇总单列「失败 N 个」 - 测试 +8:ETag 同秒双写/分页、只读登录放行+写拦截、401 语义标题、Basic 质询头、宽限期两阶段、prune 保留清单、只读文件容错(Windows 场景)、自检探针无残留
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
双代理审查(前端/平台任务 + 后端核心)发现 12 项问题,全部经我对照源码核实后修复,配套 +8 测试(136/136 绿)与真机浏览器验证:
功能级
契约级
健壮性
真机浏览器验证:批量暂停/恢复、Cron 编辑+预览+保存全链通过。