feat(P1): 成片库按用户维度改造，修复一键生成查不到+越权漏洞 #563

Merged

auto-approve-bot merged 4 commits from fix/p1-video-library-user-level into develop 2026-07-19 10:28:33 +08:00

Conversation 5 Commits 4 Files Changed 12

+201 -3

xiaoxia commented 2026-07-19 09:18:37 +08:00

Owner

Copy Link

Copy Source

背景

• 一键生成的视频project_id为空，前端按项目查成片库查不到
• /videos接口不传project_id时能查全表，存在越权安全漏洞

方案

改为按user_id维度组织成片库，既解决一键生成查不到的问题，又修复越权漏洞。

改动

1. Migration: generated_videos表加user_id列及索引
2. Domain/Model/Repository: 全链路支持user_id字段
3. API层: /videos默认按当前用户查，project_id降级为可选过滤
4. Worker层: 写表时透传user_id（edit_plan_generation + one_take生成）
5. 测试: 补4个单元测试覆盖user_id过滤逻辑

## 背景 - 一键生成的视频project_id为空，前端按项目查成片库查不到 - /videos接口不传project_id时能查全表，存在越权安全漏洞 ## 方案 改为按user_id维度组织成片库，既解决一键生成查不到的问题，又修复越权漏洞。 ## 改动 1. **Migration**: generated_videos表加user_id列及索引 2. **Domain/Model/Repository**: 全链路支持user_id字段 3. **API层**: /videos默认按当前用户查，project_id降级为可选过滤 4. **Worker层**: 写表时透传user_id（edit_plan_generation + one_take生成） 5. **测试**: 补4个单元测试覆盖user_id过滤逻辑

xiaoxia added 1 commit 2026-07-19 09:18:37 +08:00

feat(P1): 成片库按用户维度改造，修复一键生成查不到+越权漏洞 ... 8d0c4c873f

- generated_videos表加user_id列及索引
- 写表时透传user_id（edit_plan_generation + one_take生成）
- /videos接口默认按当前用户查，不传project_id不再查全表
- project_id降级为可选过滤条件
- 补单元测试覆盖user_id过滤逻辑

gitea-actions bot commented 2026-07-19 09:19:25 +08:00

Copy Link

Copy Source

🚀 预览环境已部署

项目	详情
PR号	#563
预览链接	https://pr-563.preview.xiaoxiajianji.com
API环境	staging

💡 预览环境使用 staging API 数据，请勿在预览环境中操作重要数据。

🔄 每次提交新代码后预览环境会自动更新。

🗑️ PR 关闭或合并后，预览环境会自动清理。

🚀 **预览环境已部署** | 项目 | 详情 | |------|------| | PR号 | #563 | | 预览链接 | [https://pr-563.preview.xiaoxiajianji.com](https://pr-563.preview.xiaoxiajianji.com) | | API环境 | staging | > 💡 预览环境使用 staging API 数据，请勿在预览环境中操作重要数据。 > > 🔄 每次提交新代码后预览环境会自动更新。 > > 🗑️ PR 关闭或合并后，预览环境会自动清理。

xiaoxia added 1 commit 2026-07-19 09:30:24 +08:00

fix: GeneratedVideo.create() user_id改为可选参数，保持向后兼容 ... 8bb333ccfe

user_id移到关键字参数位置并给默认值空串，避免破坏已有测试调用

xiaoxia added 1 commit 2026-07-19 09:45:19 +08:00

merge origin/develop: 解决测试文件冲突 6b3a7226fe

xiaoxia added 1 commit 2026-07-19 10:22:24 +08:00

fix: 更新schema metadata snapshot，补充generated_videos.user_id 36759a80c3

auto-approve-bot commented 2026-07-19 10:25:22 +08:00

Collaborator

Copy Link

Copy Source

代码审查结果 - PR #563

⚠️ 问题（2个需要修改）

1. apps/worker/worker_app/tasks/generation.py 第1516行：变量 user_id 未定义直接使用。

   • 问题描述：在调用 _upload_and_record 时传入了 user_id=user_id，但在 generate_video 函数的作用域内并未看到该变量的定义或提取逻辑（对比 edit_plan_generation.py 中的 plan.created_by_user_id）。这将导致运行时直接抛出 NameError。
   • 修改建议：参考 edit_plan_generation.py 的处理方式，从 generation_task 对象中提取 user_id，并处理可能的 None 值（例如 user_id=task.user_id or ""）。

2. packages/domain/generated_video.py 第60行：潜在的 AttributeError 风险。

   • 问题描述：create 方法中执行了 user_id=user_id.strip()。虽然类型注解为 str，但 Python 不强制执行类型，若上游传入 None（例如数据库中旧数据或任务对象属性为空），此处会崩溃。且由于 GeneratedVideoModel 中该字段 nullable=False，若传入 None 会导致数据库插入错误。
   • 修改建议：增加空值保护，改为 user_id = (user_id or "").strip()，确保存入数据库的值不为 None。

💡 建议（1个可选）

1. 数据迁移策略说明：
   • 具体内容：Migration 脚本中使用了 server_default="" 处理存量数据。这意味着现有的成片记录 user_id 将为空字符串。配合 Repository 中的 if user_id: 过滤逻辑，这些存量成片将对所有用户不可见（即被“隐藏”）。如果业务上需要保留存量数据的可见性，需要编写额外的数据迁移脚本来归属这些历史记录。

---

✅ 格式检查通过 | ❌ 逻辑审查需修改 | ✅ 性能检查通过

---

_{🤖 由 AI 代码审查机器人自动生成 | 2026-07-19 10:25:22 | 模型:}

## 代码审查结果 - PR #563 ### ⚠️ 问题（2个需要修改） 1. **apps/worker/worker_app/tasks/generation.py 第1516行**：变量 `user_id` 未定义直接使用。 - 问题描述：在调用 `_upload_and_record` 时传入了 `user_id=user_id`，但在 `generate_video` 函数的作用域内并未看到该变量的定义或提取逻辑（对比 `edit_plan_generation.py` 中的 `plan.created_by_user_id`）。这将导致运行时直接抛出 `NameError`。 - 修改建议：参考 `edit_plan_generation.py` 的处理方式，从 `generation_task` 对象中提取 `user_id`，并处理可能的 `None` 值（例如 `user_id=task.user_id or ""`）。 2. **packages/domain/generated_video.py 第60行**：潜在的 `AttributeError` 风险。 - 问题描述：`create` 方法中执行了 `user_id=user_id.strip()`。虽然类型注解为 `str`，但 Python 不强制执行类型，若上游传入 `None`（例如数据库中旧数据或任务对象属性为空），此处会崩溃。且由于 `GeneratedVideoModel` 中该字段 `nullable=False`，若传入 `None` 会导致数据库插入错误。 - 修改建议：增加空值保护，改为 `user_id = (user_id or "").strip()`，确保存入数据库的值不为 `None`。 ### 💡 建议（1个可选） 1. **数据迁移策略说明**： - 具体内容：Migration 脚本中使用了 `server_default=""` 处理存量数据。这意味着现有的成片记录 `user_id` 将为空字符串。配合 Repository 中的 `if user_id:` 过滤逻辑，这些存量成片将对所有用户不可见（即被“隐藏”）。如果业务上需要保留存量数据的可见性，需要编写额外的数据迁移脚本来归属这些历史记录。 --- ✅ 格式检查通过 | ❌ 逻辑审查需修改 | ✅ 性能检查通过 --- _{🤖 由 AI 代码审查机器人自动生成 | 2026-07-19 10:25:22 | 模型:} <!-- AI_CODE_REVIEW_AUTO_COMMENT -->

auto-approve-bot approved these changes 2026-07-19 10:26:07 +08:00

auto-approve-bot left a comment

Collaborator

Copy Link

Copy Source

CI全绿，自动审批通过。

CI全绿，自动审批通过。

auto-approve-bot approved these changes 2026-07-19 10:26:07 +08:00

auto-approve-bot left a comment

Collaborator

Copy Link

Copy Source

CI全绿，自动审批通过。

CI全绿，自动审批通过。

auto-approve-bot merged commit fca186533b into develop 2026-07-19 10:28:33 +08:00

auto-approve-bot deleted branch fix/p1-video-library-user-level 2026-07-19 10:28:33 +08:00

gitea-actions bot commented 2026-07-19 10:28:54 +08:00

Copy Link

Copy Source

🗑️ 预览环境已清理

PR #563 已关闭或合并，对应的预览环境已被清理。

如有需要，可以重新打开 PR 来重新生成预览环境。

🗑️ **预览环境已清理** PR #563 已关闭或合并，对应的预览环境已被清理。 > 如有需要，可以重新打开 PR 来重新生成预览环境。

xiaoxia referenced this pull request 2026-07-19 10:57:20 +08:00

fix(P1): 成片库历史数据user_id回填 #572

Sign in to join this conversation.

Reviewers

No Reviewers

auto-approve-bot

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

3 Participants

Notifications

Subscribe

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: xiaoxia/xiaoxia-saas#563