fix(worker): 修复素材下载——asset_id查assets表拿file_url + 统一OSS配置读取 #471

Merged

auto-approve-bot merged 3 commits from fix/worker-oss-settings-unified into develop 2026-07-17 15:37:41 +08:00

Conversation 2 Commits 3 Files Changed 5

+253 -186

xiaoxia commented 2026-07-17 14:40:35 +08:00

Owner

Copy Link

Copy Source

问题根因（两个）

根因1（主要）：render_adapter.py _download_assets 直接把 asset_id 当 OSS key 用

RenderAdapter._download_assets() 把 clip.asset_id（UUID主键）直接传给 download_asset() 当OSS存储路径，但 asset_id 是数据库主键，不是OSS上的文件路径。正确做法应该是先通过 asset_id 查 assets 表拿到 file_url（真正的OSS存储路径），再用 file_url 下载。

对比 edit_plan_generation.py 里的 legacy 路径和 _prepare_bgm() 方法，都是先查 AssetModel 拿 file_url 再下载，唯独 render_adapter 的 _download_assets 写错了。

根因2（次要）：两套独立的OSS配置读取路径

Worker端存在两套OSS配置读取：

• SharedStorageService → pydantic-settings + .env文件
• oss_helpers.oss_settings() → 直接 os.getenv()，不读.env

如果OSS配置写在.env文件里（大部分Docker部署），oss_helpers读不到配置 → oss_bucket()返回None → 下载静默失败。

修复内容

1. render_adapter.py 素材下载逻辑修复

• _download_assets() 从 staticmethod 改为实例方法
• 批量查询 AssetModel 获取 file_url（OSS存储路径）
• 用 file_url 作为 key 调用 download_asset()
• 保留原始文件扩展名，不再硬编码 .mp4
• 无 file_url 的素材跳过下载并标记失败

2. oss_helpers.py 配置读取统一

• oss_settings() 改用 get_shared_settings() 读取配置
• 与 SharedStorageService 配置来源保持一致
• 支持从 .env 文件加载OSS配置

影响范围

• apps/worker/video_processing/render_adapter.py — _download_assets 方法重写
• apps/worker/video_processing/oss_helpers.py — oss_settings 配置读取统一
• 主要影响：剪辑计划走统一渲染引擎的素材下载（P1-2链路）

## 问题根因（两个） ### 根因1（主要）：render_adapter.py _download_assets 直接把 asset_id 当 OSS key 用 `RenderAdapter._download_assets()` 把 `clip.asset_id`（UUID主键）直接传给 `download_asset()` 当OSS存储路径，但 asset_id 是数据库主键，不是OSS上的文件路径。正确做法应该是先通过 asset_id 查 `assets` 表拿到 `file_url`（真正的OSS存储路径），再用 file_url 下载。 对比 `edit_plan_generation.py` 里的 legacy 路径和 `_prepare_bgm()` 方法，都是先查 AssetModel 拿 file_url 再下载，唯独 render_adapter 的 _download_assets 写错了。 ### 根因2（次要）：两套独立的OSS配置读取路径 Worker端存在两套OSS配置读取： - `SharedStorageService` → pydantic-settings + .env文件 - `oss_helpers.oss_settings()` → 直接 os.getenv()，不读.env 如果OSS配置写在.env文件里（大部分Docker部署），oss_helpers读不到配置 → oss_bucket()返回None → 下载静默失败。 ## 修复内容 ### 1. render_adapter.py 素材下载逻辑修复 - `_download_assets()` 从 staticmethod 改为实例方法 - 批量查询 `AssetModel` 获取 `file_url`（OSS存储路径） - 用 `file_url` 作为 key 调用 `download_asset()` - 保留原始文件扩展名，不再硬编码 .mp4 - 无 file_url 的素材跳过下载并标记失败 ### 2. oss_helpers.py 配置读取统一 - `oss_settings()` 改用 `get_shared_settings()` 读取配置 - 与 SharedStorageService 配置来源保持一致 - 支持从 .env 文件加载OSS配置 ## 影响范围 - `apps/worker/video_processing/render_adapter.py` — _download_assets 方法重写 - `apps/worker/video_processing/oss_helpers.py` — oss_settings 配置读取统一 - 主要影响：剪辑计划走统一渲染引擎的素材下载（P1-2链路）

xiaoxia added 1 commit 2026-07-17 14:40:35 +08:00

fix(worker): 统一OSS配置读取为SharedSettings，解决oss_helpers读不到.env配置的问题 cd1f6f6b34

xiaoxia referenced this pull request 2026-07-17 14:42:25 +08:00

【P0】Staging环境OSS素材文件403 Forbidden，worker下载失败 #454

xiaoxia referenced this pull request 2026-07-17 14:42:26 +08:00

【P0】一键生成端到端出片验证 #406

xiaoxia added 1 commit 2026-07-17 14:50:17 +08:00

fix(worker): render_adapter素材下载先查assets表拿file_url而非直接用asset_id当key ... 3b96e0c6f6

- _download_assets() 从 staticmethod 改为实例方法，通过 self._db 查询 assets 表
- 批量查询 AssetModel 获取 file_url（OSS 存储路径），再用 file_url 下载
- asset_id 是 UUID 主键，不能直接当作 OSS 存储 key 使用
- 保留原始文件扩展名，不再硬编码 .mp4
- 配套更新单元测试，新增无 file_url 场景测试

xiaoxia changed title from fix(worker): 统一OSS配置读取为SharedSettings，修复素材下载静默失败 to fix(worker): 修复素材下载——asset_id查assets表拿file_url + 统一OSS配置读取 2026-07-17 14:50:35 +08:00

xiaoxia added 1 commit 2026-07-17 15:33:36 +08:00

fix(tests): 修复oss_helpers测试 — 改用patch oss_settings替代patch os.environ + black格式化 cefd97966a

auto-approve-bot approved these changes 2026-07-17 15:36:50 +08:00

auto-approve-bot left a comment

Collaborator

Copy Link

Copy Source

CI全绿，自动审批通过。

CI全绿，自动审批通过。

auto-approve-bot approved these changes 2026-07-17 15:36:50 +08:00

auto-approve-bot left a comment

Collaborator

Copy Link

Copy Source

CI全绿，自动审批通过。

CI全绿，自动审批通过。

auto-approve-bot merged commit 41e3fc7a58 into develop 2026-07-17 15:37:41 +08:00

auto-approve-bot deleted branch fix/worker-oss-settings-unified 2026-07-17 15:37:41 +08:00

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

2 Participants

Notifications

Subscribe

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: xiaoxia/xiaoxia-saas#471