feat(video-share): 视频分享后端 - 密码保护/有效期/浏览下载计数 #737

Merged

xiaoxia merged 5 commits from feat/video-share-backend into develop 2026-07-23 09:38:22 +08:00

Conversation 4 Commits 5 Files Changed 11

+1643

xiaoxia commented 2026-07-22 19:20:53 +08:00

Owner

Copy Link

Copy Source

功能概述

视频分享功能后端实现，支持创建分享链接、密码保护、有效期设置、浏览/下载计数。

新增内容

• Migration 050: video_shares 表（id, video_id, user_id, share_token, password_hash, has_password, expires_at, view_count, download_count, is_active, created_at, updated_at）
• 领域层: VideoShare 实体，SHA-256加盐密码哈希，base32易读token生成（去除易混淆字符）
• 应用层: 9个use case - Create/GetByToken/Access/ListByVideo/ListByUser/Update/Revoke/RecordDownload
• 持久层: SQLAlchemy实现，软删除（is_active=False）保留统计数据
• API层: 8个端点
  • POST /api/v1/videos/{video_id}/share - 创建分享
  • GET /api/v1/videos/{video_id}/shares - 某视频分享列表
  • GET /api/v1/shares - 用户所有分享（分页）
  • PATCH /api/v1/shares/{share_id} - 更新分享配置
  • DELETE /api/v1/shares/{share_id} - 撤销分享（软删除）
  • GET /api/v1/share/{token}/meta - 分享元信息（公开）
  • POST /api/v1/share/{token}/access - 访问分享内容（验证密码+返回视频信息+播放地址）
  • POST /api/v1/share/{token}/download - 记录下载计数
• 单元测试: 56个测试覆盖领域实体+全部9个use case

设计决策

• 密码用SHA-256加盐（分享链接安全级别要求不高，避免引入bcrypt重依赖）
• token用易读字母表，去掉l/I/1/0/O等易混淆字符
• 撤销分享用软删除（is_active=False），保留浏览/下载统计数据

## 功能概述 视频分享功能后端实现，支持创建分享链接、密码保护、有效期设置、浏览/下载计数。 ## 新增内容 - **Migration 050**: `video_shares` 表（id, video_id, user_id, share_token, password_hash, has_password, expires_at, view_count, download_count, is_active, created_at, updated_at） - **领域层**: `VideoShare` 实体，SHA-256加盐密码哈希，base32易读token生成（去除易混淆字符） - **应用层**: 9个use case - Create/GetByToken/Access/ListByVideo/ListByUser/Update/Revoke/RecordDownload - **持久层**: SQLAlchemy实现，软删除（is_active=False）保留统计数据 - **API层**: 8个端点 - `POST /api/v1/videos/{video_id}/share` - 创建分享 - `GET /api/v1/videos/{video_id}/shares` - 某视频分享列表 - `GET /api/v1/shares` - 用户所有分享（分页） - `PATCH /api/v1/shares/{share_id}` - 更新分享配置 - `DELETE /api/v1/shares/{share_id}` - 撤销分享（软删除） - `GET /api/v1/share/{token}/meta` - 分享元信息（公开） - `POST /api/v1/share/{token}/access` - 访问分享内容（验证密码+返回视频信息+播放地址） - `POST /api/v1/share/{token}/download` - 记录下载计数 - **单元测试**: 56个测试覆盖领域实体+全部9个use case ## 设计决策 - 密码用SHA-256加盐（分享链接安全级别要求不高，避免引入bcrypt重依赖） - token用易读字母表，去掉l/I/1/0/O等易混淆字符 - 撤销分享用软删除（is_active=False），保留浏览/下载统计数据

xiaoxia added 1 commit 2026-07-22 19:20:53 +08:00

feat(video-share): add video share backend with password protection and expiry ... 9f61a68b6e

- Add video_shares table with migration 050
- Domain entity with SHA-256 password hashing and token generation
- 9 use cases: create, get by token, access, list by video/user, update, revoke, record download
- SQLAlchemy repository implementation
- 8 API endpoints (create/list/update/revoke/meta/access/download)
- 56 unit tests, all passing

gitea-actions bot commented 2026-07-22 19:50:29 +08:00

Copy Link

Copy Source

🚀 预览环境已部署

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

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

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

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

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

xiaoxia added 1 commit 2026-07-22 20:10:30 +08:00

fix(ci): auto-fix lint/format issues 066e235595

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

fix(migration): 修复video_shares重复创建索引导致迁移失败 ... 59c473f096

op.create_table中Column(index=True)会自动创建索引，
后面再op.create_index就重复了。按项目规范，单列索引
通过Column(index=True)声明，去掉重复的create_index。

auto-approve-bot commented 2026-07-22 23:10:58 +08:00

Collaborator

Copy Link

Copy Source

代码审查结果 - PR #737

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

1. packages/adapters/sqlalchemy_impl/video_share_repository.py 第 133-145 行：计数器更新存在并发安全问题。

   • 问题描述：increment_view 和 increment_download 方法采用“读取-修改-写入”的模式（先查询模型，修改属性，再提交）。在高并发场景下，多个请求同时读取相同的计数值并分别加1写入，会导致计数丢失，统计数据不准确。
   • 修改建议：使用原子更新操作，例如 session.query(VideoShareModel).filter(...).update({"view_count": VideoShareModel.view_count + 1}, synchronize_session=False)，或者使用数据库层面的 FOR UPDATE 锁。

2. packages/adapters/sqlalchemy_impl/video_share_repository.py 第 28, 64, 95, 109, 133, 145 行：Repository 层不应直接管理事务提交。

   • 问题描述：Repository 方法内部直接调用了 self.session.commit()。这违反了 Unit of Work 模式，导致事务边界破碎。如果业务层（Use Case）需要在一个事务中执行多个 Repository 操作（如创建分享并记录日志），一旦某个操作失败，无法回滚之前已提交的操作。
   • 修改建议：移除 Repository 中的 commit() 调用，仅进行 flush() 或 add()。事务提交应由依赖注入的 Session 上下文管理器（通常在 FastAPI 的 dependency 中处理）统一控制。

3. packages/adapters/sqlalchemy_impl/models.py 第 582 行：share_token 字段存在冗余索引。

   • 问题描述：share_token 字段同时设置了 unique=True 和 index=True。在 SQLAlchemy 中，unique=True 会自动创建一个唯一索引，显式添加 index=True 会创建第二个非唯一索引。这会浪费磁盘空间并降低写入性能。
   • 修改建议：移除 index=True，仅保留 unique=True。

💡 建议（2个可选）

1. apps/api/app/api/routes/share.py 第 233, 243, 258, 268 行：异常捕获过于宽泛且缺少日志。

   • 具体内容：在获取 OSS 下载地址时使用了 except Exception，这会掩盖所有错误（包括网络中断、配置错误等），且静默失败可能导致前端显示错误的 URL。建议捕获具体的异常类型（如 OSSServiceError），并记录 warning 级别日志以便排查问题。

2. apps/api/app/api/routes/share.py 第 233-270 行：存在重复代码。

   • 具体内容：access_share 函数中生成 download_url 和 thumbnail_url 的逻辑完全相同（try-except 块）。建议提取一个私有辅助函数 _get_signed_url(storage, raw_url) 来消除重复。

---

✅ 格式检查通过 | ❌ 逻辑审查需修改 | ⚠️ 建议关注性能

---

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

## 代码审查结果 - PR #737 ### ⚠️ 问题（3个需要修改） 1. **packages/adapters/sqlalchemy_impl/video_share_repository.py 第 133-145 行**：计数器更新存在并发安全问题。 - **问题描述**：`increment_view` 和 `increment_download` 方法采用“读取-修改-写入”的模式（先查询模型，修改属性，再提交）。在高并发场景下，多个请求同时读取相同的计数值并分别加1写入，会导致计数丢失，统计数据不准确。 - **修改建议**：使用原子更新操作，例如 `session.query(VideoShareModel).filter(...).update({"view_count": VideoShareModel.view_count + 1}, synchronize_session=False)`，或者使用数据库层面的 `FOR UPDATE` 锁。 2. **packages/adapters/sqlalchemy_impl/video_share_repository.py 第 28, 64, 95, 109, 133, 145 行**：Repository 层不应直接管理事务提交。 - **问题描述**：Repository 方法内部直接调用了 `self.session.commit()`。这违反了 Unit of Work 模式，导致事务边界破碎。如果业务层（Use Case）需要在一个事务中执行多个 Repository 操作（如创建分享并记录日志），一旦某个操作失败，无法回滚之前已提交的操作。 - **修改建议**：移除 Repository 中的 `commit()` 调用，仅进行 `flush()` 或 `add()`。事务提交应由依赖注入的 Session 上下文管理器（通常在 FastAPI 的 `dependency` 中处理）统一控制。 3. **packages/adapters/sqlalchemy_impl/models.py 第 582 行**：`share_token` 字段存在冗余索引。 - **问题描述**：`share_token` 字段同时设置了 `unique=True` 和 `index=True`。在 SQLAlchemy 中，`unique=True` 会自动创建一个唯一索引，显式添加 `index=True` 会创建第二个非唯一索引。这会浪费磁盘空间并降低写入性能。 - **修改建议**：移除 `index=True`，仅保留 `unique=True`。 ### 💡 建议（2个可选） 1. **apps/api/app/api/routes/share.py 第 233, 243, 258, 268 行**：异常捕获过于宽泛且缺少日志。 - **具体内容**：在获取 OSS 下载地址时使用了 `except Exception`，这会掩盖所有错误（包括网络中断、配置错误等），且静默失败可能导致前端显示错误的 URL。建议捕获具体的异常类型（如 `OSSServiceError`），并记录 `warning` 级别日志以便排查问题。 2. **apps/api/app/api/routes/share.py 第 233-270 行**：存在重复代码。 - **具体内容**：`access_share` 函数中生成 `download_url` 和 `thumbnail_url` 的逻辑完全相同（try-except 块）。建议提取一个私有辅助函数 `_get_signed_url(storage, raw_url)` 来消除重复。 --- ✅ 格式检查通过 | ❌ 逻辑审查需修改 | ⚠️ 建议关注性能 --- _{🤖 由 AI 代码审查机器人自动生成 | 2026-07-22 23:10:58 | 模型:} <!-- AI_CODE_REVIEW_AUTO_COMMENT -->

xiaoxia referenced this pull request 2026-07-23 06:38:36 +08:00

fix: 修复AI审查严重问题检测正则，正确匹配需要修改的问题 #747

xiaoxia added 1 commit 2026-07-23 06:58:55 +08:00

fix: 计数器改为数据库原子更新，解决并发安全问题 40ed7b8e2f

xiaoxia added 1 commit 2026-07-23 06:59:12 +08:00

fix: 移除share_token冗余索引（unique已自动创建唯一索引） a29c0e0b21

xiaoxia reviewed 2026-07-23 09:38:21 +08:00

xiaoxia left a comment

Author

Owner

Copy Link

Copy Source

CI全绿，approve

CI全绿，approve

xiaoxia merged commit 833c604fd7 into develop 2026-07-23 09:38:22 +08:00

gitea-actions bot commented 2026-07-23 09:39:03 +08:00

Copy Link

Copy Source

🗑️ 预览环境已清理

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

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

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

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

3 Participants

Notifications

Subscribe

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: xiaoxia/xiaoxia-saas#737