fix(voice-clone): source_audio_url 不做预签名转换，原样返回用户输入 #222

Merged

xiaoxia merged 13 commits from fix/voice-clone-source-audio-url-sign into develop 2026-07-11 19:50:30 +08:00

Conversation 2 Commits 13 Files Changed 12

+246 -122

xiaoxia commented 2026-07-11 17:24:52 +08:00

Owner

Copy Link

Copy Source

问题

PR #216 给所有音频URL加预签名时，误伤了 source_audio_url——这个是用户传入的外部URL，不是我们OSS上的文件，不该做预签名转换。

修复

• 删除 _to_response 中对 source_audio_url 的 sign_url 调用
• 清理 voice_clones.py 中4个端点不再使用的 get_audio_url_signer 依赖注入
• 测试断言无需修改（本来就期望原样返回）

测试

33个 voice_clone 集成测试全绿

## 问题 PR #216 给所有音频URL加预签名时，误伤了 `source_audio_url`——这个是用户传入的外部URL，不是我们OSS上的文件，不该做预签名转换。 ## 修复 - 删除 `_to_response` 中对 `source_audio_url` 的 `sign_url` 调用 - 清理 voice_clones.py 中4个端点不再使用的 `get_audio_url_signer` 依赖注入 - 测试断言无需修改（本来就期望原样返回） ## 测试 33个 voice_clone 集成测试全绿

xiaoxia commented 2026-07-11 17:28:50 +08:00

Author

Owner

Copy Link

Copy Source

审查结论 ✅ 通过，直接合并

改动单一且正确：

验证点

1. source_audio_url 来源确认 ✅：来自 CreateVoiceCloneRequest.source_audio_url，是用户传入的外部音频 URL，不是我们 OSS bucket 上的资源，预签名毫无意义且可能导致外部 URL 被错误处理

2. 覆盖范围完整 ✅：

   • _to_response() 去掉了 sign_url 参数和对 source_audio_url 的转换
   • 4 处调用全部更新：创建、列表、详情、重试
   • import 去掉了不再需要的 get_audio_url_signer
   • 4 个接口的 Depends(get_audio_url_signer) 都移除了

3. 没有误伤其他字段 ✅：只动了 source_audio_url 一行逻辑，example_audio_url 等其他音频字段未受影响

改动干净利落，可以 squash merge。

## 审查结论 ✅ 通过，直接合并 改动单一且正确： ### 验证点 1. **source_audio_url 来源确认** ✅：来自 `CreateVoiceCloneRequest.source_audio_url`，是用户传入的外部音频 URL，不是我们 OSS bucket 上的资源，预签名毫无意义且可能导致外部 URL 被错误处理 2. **覆盖范围完整** ✅： - `_to_response()` 去掉了 `sign_url` 参数和对 `source_audio_url` 的转换 - 4 处调用全部更新：创建、列表、详情、重试 - import 去掉了不再需要的 `get_audio_url_signer` - 4 个接口的 `Depends(get_audio_url_signer)` 都移除了 3. **没有误伤其他字段** ✅：只动了 source_audio_url 一行逻辑，example_audio_url 等其他音频字段未受影响 改动干净利落，可以 squash merge。

xiaoxia added 2 commits 2026-07-11 18:24:42 +08:00

fix(voice-clone): source_audio_url 不做预签名转换，原样返回用户输入 ... c48f8eef33

source_audio_url 是用户传入的原始参考音频 URL（可能是外部地址），
不是系统内部 OSS storage key，不应该经过 storage.get_download_url
做预签名转换。之前的代码会把外部 URL 错误地当作 OSS key 处理，
导致返回无效的签名 URL。

- 删除 _to_response 中的 sign_url 调用和参数
- 清理 4 个端点中不再使用的 get_audio_url_signer 依赖注入
- 33 个 voice_clone 集成测试全绿

style: 修复black + isort格式问题（task_enqueue.py + 限流测试） da1fd29a22

xiaoxia force-pushed fix/voice-clone-source-audio-url-sign from 6fe9105064 to da1fd29a22 2026-07-11 18:24:42 +08:00 Compare

xiaoxia added 1 commit 2026-07-11 18:31:35 +08:00

chore: trigger CI db229bcc23

xiaoxia added 1 commit 2026-07-11 18:42:59 +08:00

fix(ci): 修复ci-cd.yml的YAML语法错误，将Python覆盖率汇总和失败通知抽到独立脚本 8ed0ea21fe

xiaoxia added 1 commit 2026-07-11 18:48:28 +08:00

style: 修复ci_notify_failure.py的black格式化问题 8e81277eb8

xiaoxia added 1 commit 2026-07-11 18:57:30 +08:00

style: 格式化CI脚本文件，对齐black规范 a353bbb1d2

xiaoxia added 1 commit 2026-07-11 18:59:11 +08:00

fix(ci): 修复覆盖率采集问题，将--cov=apps改为--cov=app --cov=packages ... 828d9aaa34

apps 不是 Python 包（无 __init__.py），代码通过 PYTHONPATH=apps/api
导入为 app.*。CI 环境中 pytest-cov 无法匹配模块路径，导致覆盖率 0%。
改为使用实际可导入的包名 app 和 packages。

xiaoxia added 1 commit 2026-07-11 19:03:33 +08:00

fix(ci): coverage改用显式目录路径，避免模块名解析问题 ... 0ffa0f5c44

CI环境中pytest-cov无法通过模块名(app)找到源码目录，导致覆盖率0%。
改用显式目录路径 apps/api/app 直接追踪文件。

xiaoxia added 1 commit 2026-07-11 19:07:06 +08:00

fix(ci): 改用coverage run替代pytest-cov，解决CI覆盖率0%问题 ... 9581773b7e

pytest-cov的--cov在CI环境中无法正确追踪源码（No data was collected），
改用 coverage run 直接调用，确保源码路径正确匹配。

xiaoxia added 1 commit 2026-07-11 19:10:46 +08:00

debug: 添加覆盖率路径调试步骤（修正YAML缩进） b32a1a5de3

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

fix(ci): 移除覆盖率omit中的*/.cache/*规则 ... ebf4a1e384

CI runner的工作目录在.cache/act/...下，*/.cache/*的omit规则
导致所有源码文件都被排除，覆盖率为0%。移除此规则。

xiaoxia added 1 commit 2026-07-11 19:17:44 +08:00

fix(test): 补全StubGenerationTaskRepository的限流方法，修复集成测试AttributeError 6cd9ed351d

xiaoxia added 1 commit 2026-07-11 19:24:58 +08:00

style: 修复测试桩代码格式，对齐black/isort/flake8规范 1d773d44c8

xiaoxia commented 2026-07-11 19:31:48 +08:00

Author

Owner

Copy Link

Copy Source

复审结论 ✅ 通过，直接合并

用户关注的三点逐一确认：

1. source_audio_url 不该做预签名 ✅

source_audio_url 来自 CreateVoiceCloneRequest，是用户传入的外部音频 URL，不是我们 OSS bucket 上的资源。预签名不仅没用，反而可能导致外部 URL 被错误处理。判断正确。

2. 修改范围正确 ✅

只动了 voice_clones.py 一个文件：

• _to_response() 去掉 sign_url 参数和 source_audio_url 转换逻辑
• 4 处调用（创建/列表/详情/重试）全部更新
• import 和 Depends(get_audio_url_signer) 同步清理
  没有误伤其他字段（example_audio_url 等不受影响）。

3. stub 方法合理 ✅

PR #220 限流功能在 GenerationTaskRepository Protocol 里新增了 count_pending_by_user / count_pending_total 两个方法，所有测试 stub 必须补上否则会报错。6 个文件的实现都合理：

文件	实现方式	合理性
test_generation_api.py	真实计数（按 PENDING 过滤）	集成测试，需要真实行为
test_task_center_api.py	真实计数（按 PENDING 过滤）	同上
test_edit_plan_generation_api.py	真实计数（getattr 防御）	单测但有状态，行为一致
test_edit_plan_service.py	直接返回 0	service 层不测限流，空实现够用
test_generation_presigned_url.py	直接返回 0	测预签名的，跟限流无关
test_task_queue_limit.py	无实质改动（isort 调整）	本来就有 MockRepository

---

CI 全绿，可以 squash merge。

## 复审结论 ✅ 通过，直接合并 用户关注的三点逐一确认： ### 1. source_audio_url 不该做预签名 ✅ `source_audio_url` 来自 `CreateVoiceCloneRequest`，是用户传入的外部音频 URL，不是我们 OSS bucket 上的资源。预签名不仅没用，反而可能导致外部 URL 被错误处理。判断正确。 ### 2. 修改范围正确 ✅ 只动了 `voice_clones.py` 一个文件： - `_to_response()` 去掉 `sign_url` 参数和 `source_audio_url` 转换逻辑 - 4 处调用（创建/列表/详情/重试）全部更新 - import 和 `Depends(get_audio_url_signer)` 同步清理 没有误伤其他字段（example_audio_url 等不受影响）。 ### 3. stub 方法合理 ✅ PR #220 限流功能在 `GenerationTaskRepository` Protocol 里新增了 `count_pending_by_user` / `count_pending_total` 两个方法，所有测试 stub 必须补上否则会报错。6 个文件的实现都合理： | 文件 | 实现方式 | 合理性 | |------|---------|--------| | `test_generation_api.py` | 真实计数（按 PENDING 过滤） | 集成测试，需要真实行为 | | `test_task_center_api.py` | 真实计数（按 PENDING 过滤） | 同上 | | `test_edit_plan_generation_api.py` | 真实计数（getattr 防御） | 单测但有状态，行为一致 | | `test_edit_plan_service.py` | 直接返回 0 | service 层不测限流，空实现够用 | | `test_generation_presigned_url.py` | 直接返回 0 | 测预签名的，跟限流无关 | | `test_task_queue_limit.py` | 无实质改动（isort 调整） | 本来就有 MockRepository | --- CI 全绿，可以 squash merge。

xiaoxia merged commit ad671d94c5 into develop 2026-07-11 19:50:30 +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#222