fix: 完善鉴权模块，修复 CI/CD 中 dubious ownership 错误 #143

huanghaosheng · 2026-06-20T17:24:02+08:00

huanghaosheng commented

2026-06-20 17:24:02 +08:00

No description provided.

huanghaosheng added 8 commits 2026-06-20 17:24:02 +08:00

feat: JWT Claims 增加 TokenType 字段，ValidateAccess/ValidateRefresh 区分校验类型 87c54b80c0

- Claims 新增 TokenType 字段（"access" / "refresh"）
- GeneratePair 为 access/refresh token 分别设置 token_type
- ValidateAccess 校验后检查 token_type == "access"
- ValidateRefresh 校验后检查 token_type == "refresh"
- 增加 token 类型交叉校验测试

feat: Refresh Token 复用检测 — 检测到已 rotation 的 token 被复用时吊销用户全部 refresh token 898e30b526

- Refresh 方法中 FindRefreshToken 返回 not found 时，检查 JWT 是否有效
- JWT 有效但 DB 不存在 → 判定为复用，调用 DeleteUserRefreshTokens 吊销该用户所有 token
- 增加 TestRefresh_ReuseDetectedRevokesAllTokens 测试覆盖复用场景

feat: CachedUserRepository — refresh token 的 Redis 缓存装饰器 a2a28a9f56

- 装饰 UserRepository，仅缓存 refresh token 相关操作
- SaveRefreshToken: Write-Through，先写 DB 再写 Redis（SET + SADD）
- FindRefreshToken: Read-Through，Redis miss 时查 DB 并回填
- DeleteRefreshToken: 双删 DB + Redis
- DeleteUserRefreshTokens: 通过 Redis Set 批量清理缓存后删 DB
- Redis 操作失败时降级到纯 DB，不阻断主流程

feat: main.go 注入 Redis 到 CachedUserRepository，启用 refresh token 二级缓存 ea70d2efc6

- 将 rdb 变量提升到外层作用域，供 session 和 auth 共用
- Redis 启用时用 CachedUserRepository 包装 userRepo
- backfillTTL 使用 cfg.Auth.RefreshTTL 与 token 实际过期时间一致

feat: 前端 401 拦截器 — access token 过期时自动刷新并重试原请求 d78cdb509d

api.ts:
- 增加 setAuthCallbacks 回调注入机制（避免 api/auth 循环依赖）
- request() 对非公开路径自动附加 Authorization header
- 收到 401 时自动触发 refresh token 刷新，成功后重试原请求
- 并发保护：多个 401 只触发一次 refresh，其余等待同一 Promise
- refreshTokenDirect 内部方法绕过 401 拦截避免递归

auth.tsx:
- 用 ref 保存 persistAuth/scheduleRefresh 最新引用（避免闭包陈旧）
- 初始化时调用 setAuthCallbacks 注入认证回调

fix: 修复前端 lint 错误 — 移除 ref 模式，用 useEffect 依赖注入回调 970f10a274

- api.ts: 移除 catch 中未使用的 err 变量
- auth.tsx: 去掉 useRef 模式，直接在 useEffect 中注册回调并声明依赖

Merge pull request 'feat: 完善鉴权模块' (#142 ) from fix/auth into develop 3572b867c0

Reviewed-on: http://8.161.227.145:3000/XEngineers/CamTalk/pulls/142

fix: 添加 git safe.directory 配置，修复 CI/CD 中 dubious ownership 错误 39073b7673

huanghaosheng merged commit 97e125234b into v2

2026-06-20 17:24:06 +08:00

huanghaosheng referenced this issue from a commit

2026-06-20 17:24:07 +08:00

Merge pull request 'fix: 完善鉴权模块，修复 CI/CD 中 dubious ownership 错误' (#143) from develop into v2

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: XEngineers/CamTalk#143