fix(cover): 修复预览视频存储路径错误导致封面生成 500 #1357

Merged

auto-approve-bot merged 2 commits from fix/double-slash-oss-path into develop 2026-08-13 17:51:00 +08:00

Conversation 4 Commits 2 Files Changed 5

+48 -2

xiaoxia commented 2026-08-13 17:31:43 +08:00

Owner

Copy Link

Copy Source

问题

封面生成 API 返回 500，MediaKit 报 DownloadFileError, http status code is 404。

根因（两个独立 bug）

Bug 1: 统一渲染引擎 storage_key 路径不匹配（主要问题）

• RenderAdapter 实际上传到 rendered/{plan_id}/{job_id}.mp4
• 但 edit_plan_generation.py 硬编码存储 rendered/{plan_id}/output.mp4
• cover.py 通过 storage_key 构造的 URL 指向不存在的文件 → 404
• 验证：rendered/f86959d8.../output.mp4 → 404，rendered/gen_f86959d8.../d54bcb32....mp4 → 200

Bug 2: Worker _upload_and_record 双斜杠路径

• project_id 为空时（模板编辑器草稿），f"generated/projects/{project_id}/tasks/..." 产生 projects//tasks/
• MediaKit HTTP 客户端规范化 URL（合并 // 为 /）→ 404

修复

文件	修复
edit_plan_generation.py	从 adapter 返回的 output_url 提取实际 storage_key，不再硬编码 output.mp4
worker/generation.py	_upload_and_record 过滤空路径段，新文件不再产生 //

注意

旧数据中已存在的 rendered_storage_key（指向 output.mp4）和 generated_videos.file_url（含 //）需要用户重新生成预览才能修复。新任务不再产生这些问题。

## 问题 封面生成 API 返回 500，MediaKit 报 `DownloadFileError, http status code is 404`。 ## 根因（两个独立 bug） ### Bug 1: 统一渲染引擎 storage_key 路径不匹配（主要问题） - RenderAdapter 实际上传到 `rendered/{plan_id}/{job_id}.mp4` - 但 `edit_plan_generation.py` 硬编码存储 `rendered/{plan_id}/output.mp4` - cover.py 通过 storage_key 构造的 URL 指向不存在的文件 → 404 - 验证：`rendered/f86959d8.../output.mp4` → 404，`rendered/gen_f86959d8.../d54bcb32....mp4` → 200 ### Bug 2: Worker _upload_and_record 双斜杠路径 - `project_id` 为空时（模板编辑器草稿），`f"generated/projects/{project_id}/tasks/..."` 产生 `projects//tasks/` - MediaKit HTTP 客户端规范化 URL（合并 `//` 为 `/`）→ 404 ## 修复 | 文件 | 修复 | |---|---| | `edit_plan_generation.py` | 从 adapter 返回的 `output_url` 提取实际 storage_key，不再硬编码 `output.mp4` | | `worker/generation.py` | `_upload_and_record` 过滤空路径段，新文件不再产生 `//` | ## 注意 旧数据中已存在的 `rendered_storage_key`（指向 `output.mp4`）和 `generated_videos.file_url`（含 `//`）需要用户重新生成预览才能修复。新任务不再产生这些问题。

xiaoxia added 1 commit 2026-08-13 17:31:43 +08:00

fix(cover): 修复 OSS 路径双斜杠导致 MediaKit 404 ... 8de7c5188f

根因：模板编辑器草稿不属于任何项目(project_id='')，Worker 构建
storage_key 时 f'generated/projects/{project_id}/tasks/...' 产生
projects//tasks/ 双斜杠路径。文件实际上传到 OSS（URL 返回 200），
但 MediaKit 的 HTTP 客户端规范化 URL（合并 // 为 /），导致 404。

修复（4 层防御）：
1. Worker: _upload_and_record 过滤空路径段，新文件不再产生 //
2. cover.py: 获取 URL 后规范化，处理历史脏数据
3. generation.py: _get_task_output_url 规范化复用产物 URL
4. ai_service.py: _call_ai_cover_service 入口规范化，不依赖调用方

新增单测: test_call_ai_cover_url_double_slash_normalized

gitea-actions bot commented 2026-08-13 17:33:08 +08:00

Copy Link

Copy Source

🚀 预览环境已部署

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

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

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

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

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

auto-approve-bot added 1 commit 2026-08-13 17:37:30 +08:00

style: auto-format with black + isort + prettier [skip ci-format-check] f770124f28

auto-approve-bot approved these changes 2026-08-13 17:45:29 +08:00

auto-approve-bot left a comment

Collaborator

Copy Link

Copy Source

CI全绿，自动审批通过。

CI全绿，自动审批通过。

auto-approve-bot approved these changes 2026-08-13 17:45:30 +08:00

auto-approve-bot left a comment

Collaborator

Copy Link

Copy Source

CI全绿，自动审批通过。

CI全绿，自动审批通过。

auto-approve-bot merged commit 68fa7fd163 into develop 2026-08-13 17:51:00 +08:00

auto-approve-bot deleted branch fix/double-slash-oss-path 2026-08-13 17:51:01 +08:00

gitea-actions bot commented 2026-08-13 17:51:45 +08:00

Copy Link

Copy Source

🗑️ 预览环境已清理

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

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

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

xiaoxia changed title from fix(cover): 修复 OSS 路径双斜杠导致 MediaKit 404 to fix(cover): 修复预览视频存储路径错误导致封面生成 500 2026-08-13 17:56:33 +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

3 Participants

Notifications

Subscribe

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: xiaoxia/xiaoxia-saas#1357