fix(backend): Phase 1 后端代码清理与修复 #249

Merged

xiaoxia merged 2 commits from cleanup/phase1-backend-v2 into develop 2026-07-13 15:02:13 +08:00

Conversation 1 Commits 2 Files Changed 53

+145 -209

xiaoxia commented 2026-07-13 13:51:13 +08:00

Owner

Copy Link

Copy Source

修复内容

P0 关键修复

• P0-1: 注册接口添加 RateLimitMiddleware 限流保护
• P0-3: /metrics 端点添加 JWT 认证（移除匿名访问）
• P0-4: 修复 Celery 任务名冲突（generation_task vs generate_video）
• P1-5: JWT logout token 黑名单机制

P1 修复

• P1-1: forgot_password 硬编码 localhost → 使用 settings.APP_BASE_URL
• P1-2: generation.py 直接创建 DB 连接 → 使用依赖注入
• P1-6: Image.open() 未关闭 → 统一使用 with 语句
• P1-7: 订阅续费事务修复

P2 代码质量

• P2-1: 修复 EditingMode 枚举重复定义 → 统一引用 shared 包
• P2-2: 修复 SMTP_FRON_NAME → SMTP_FROM_NAME 拼写
• P2-3: UserModel subscription_quota 类型统一为 float
• P2-4: .env.production DATABASE_MAX_OVERFLOW 30 → 10
• 清理 15 处 except:pass（保留 2 处有注释说明的）
• 禁用 SVG 上传（XSS 风险）
• 删除 decode_token_unsafe() 不安全函数
• 简化 /ready 端点
• 删除 8 处死代码、10 个空文件/模块
• 合并 3 对 100% 重复函数
• 对齐 6 个废弃环境变量

v2 修复（代码审查后）

• 路由路径修复: /password/forgot → /forgot-password, /password/reset → /reset-password（与前端 API 对齐）
• 重复函数合并: asset_libraries.py 和 edit_plans.py 中的 _check_project_access 统一到 _helpers.py（含空字符串守卫 + 中文错误信息）
• 顺手修复: HTTPException 统一从 fastapi 导入（替换 starlette 导入）

说明

• OSS_ENDPOINT 拼写修复（aliiyuncs.com → aliyuncs.com）将拆分为单独 PR，本 PR 不包含

影响范围

• 后端 API 路由层
• 配置管理
• 认证模块

测试建议

• 测试密码重置流程（forgot-password + reset-password）
• 测试素材库删除的项目权限校验
• 测试剪辑计划 CRUD 的项目权限校验
• 验证 /metrics 端点需要认证
• 验证注册接口限流生效

## 修复内容 ### P0 关键修复 - **P0-1**: 注册接口添加 RateLimitMiddleware 限流保护 - **P0-3**: /metrics 端点添加 JWT 认证（移除匿名访问） - **P0-4**: 修复 Celery 任务名冲突（generation_task vs generate_video） - **P1-5**: JWT logout token 黑名单机制 ### P1 修复 - **P1-1**: forgot_password 硬编码 localhost → 使用 settings.APP_BASE_URL - **P1-2**: generation.py 直接创建 DB 连接 → 使用依赖注入 - **P1-6**: Image.open() 未关闭 → 统一使用 with 语句 - **P1-7**: 订阅续费事务修复 ### P2 代码质量 - **P2-1**: 修复 EditingMode 枚举重复定义 → 统一引用 shared 包 - **P2-2**: 修复 SMTP_FRON_NAME → SMTP_FROM_NAME 拼写 - **P2-3**: UserModel subscription_quota 类型统一为 float - **P2-4**: .env.production DATABASE_MAX_OVERFLOW 30 → 10 - 清理 15 处 except:pass（保留 2 处有注释说明的） - 禁用 SVG 上传（XSS 风险） - 删除 decode_token_unsafe() 不安全函数 - 简化 /ready 端点 - 删除 8 处死代码、10 个空文件/模块 - 合并 3 对 100% 重复函数 - 对齐 6 个废弃环境变量 ### v2 修复（代码审查后） - **路由路径修复**: `/password/forgot` → `/forgot-password`, `/password/reset` → `/reset-password`（与前端 API 对齐） - **重复函数合并**: `asset_libraries.py` 和 `edit_plans.py` 中的 `_check_project_access` 统一到 `_helpers.py`（含空字符串守卫 + 中文错误信息） - **顺手修复**: HTTPException 统一从 fastapi 导入（替换 starlette 导入） ### 说明 - OSS_ENDPOINT 拼写修复（`aliiyuncs.com` → `aliyuncs.com`）将拆分为单独 PR，本 PR 不包含 ## 影响范围 - 后端 API 路由层 - 配置管理 - 认证模块 ## 测试建议 - 测试密码重置流程（forgot-password + reset-password） - 测试素材库删除的项目权限校验 - 测试剪辑计划 CRUD 的项目权限校验 - 验证 /metrics 端点需要认证 - 验证注册接口限流生效

xiaoxia added 1 commit 2026-07-13 13:51:13 +08:00

fix(backend): Phase 1 后端代码清理与修复 ... 9c6c477f55

P0 关键修复:
- P0-1: 注册接口添加 RateLimitMiddleware 限流保护
- P0-3: /metrics 端点添加 JWT 认证（移除匿名访问）
- P0-4: 修复 Celery 任务名冲突（generation_task vs generate_video）
- P1-5: JWT logout token 黑名单机制

P1 修复:
- P1-1: forgot_password 硬编码 localhost → 使用 settings.APP_BASE_URL
- P1-2: generation.py 直接创建 DB 连接 → 使用依赖注入
- P1-6: Image.open() 未关闭 → 统一使用 with 语句
- P1-7: 订阅续费事务修复

P2 代码质量:
- P2-1: 修复 EditingMode 枚举重复定义 → 统一引用 shared 包
- P2-2: 修复 SMTP_FRON_NAME → SMTP_FROM_NAME 拼写
- P2-3: UserModel subscription_quota 类型统一为 float
- P2-4: .env.production DATABASE_MAX_OVERFLOW 30 → 10
- 清理 15 处 except:pass（保留 2 处有注释说明的）
- 禁用 SVG 上传（XSS 风险）
- 删除 decode_token_unsafe() 不安全函数
- 简化 /ready 端点
- 删除 8 处死代码、10 个空文件/模块
- 合并 3 对 100% 重复函数
- 对齐 6 个废弃环境变量

v2 修复（代码审查后）:
- 修复密码重置路由路径: /password/forgot → /forgot-password,
  /password/reset → /reset-password（与前端 API 对齐）
- 合并 _check_project_access: asset_libraries.py 和 edit_plans.py
  中的重复函数统一到 _helpers.py（含空字符串守卫 + 中文错误信息）
- 顺手修复: HTTPException 统一从 fastapi 导入（替换 starlette 导入）
- OSS_ENDPOINT 拼写修复拆分为单独 PR，本 PR 不包含

xiaoxia added 1 commit 2026-07-13 14:30:36 +08:00

fix: 恢复 tts_job/workflow.py 中误删的 TTSJobStatus import ... 8598638e8f

- 第 207 行仍在使用 TTSJobStatus.COMPLETED.value
- 删除 import 会导致运行时 NameError
- pyflakes 验证零 undefined name 错误

xiaoxia reviewed 2026-07-13 15:01:33 +08:00

xiaoxia left a comment

Author

Owner

Copy Link

Copy Source

Phase 1 后端代码清理，复核通过。TTSJobStatus import 已恢复，pyflakes 零警告。详见 PR 讨论。

Phase 1 后端代码清理，复核通过。TTSJobStatus import 已恢复，pyflakes 零警告。详见 PR 讨论。

xiaoxia merged commit c1e466f9c1 into develop 2026-07-13 15:02:13 +08:00

Sign in to join this conversation.

Reviewers

No Reviewers

Labels

Clear labels

P0

阻塞性，必须立即处理

P1

高优先级

P2

中优先级

P3

低优先级/后考虑

architecture

架构设计

backend

后端相关

blocked

阻塞中

bug

Bug修复

ci-bug

CI本身的bug

ci-optimization

CI优化类任务

ci-pipeline

CI流水线/workflow相关

cleanup

代码清理

devops

运维/部署/CI

documentation

文档相关

feature

新功能

frontend

前端相关

question

需要讨论确认

ready

PR CI通过，可合并

refactor

代码重构

No Label

Milestone

No items

No Milestone

Projects

Clear projects

No project

Assignees

Clear assignees

auto-approve-bot xiaoxia

No Assignees

1 Participants

Notifications

Subscribe

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: xiaoxia/xiaoxia-saas#249