fix: #1809 补充 对口型接口参数校验返回400而非500 #1818

Merged

auto-approve-bot merged 1 commits from fix/lipsync-error-handling-1809 into develop 2026-09-09 11:12:09 +08:00

Conversation 2 Commits 1 Files Changed 2

+74 -1

xiaoxia commented 2026-09-09 11:04:01 +08:00

Owner

Copy Link

Copy Source

问题

PR #1814 合并后，前端还在用旧参数调 /api/v1/lipsync/jobs，后端返回 500 而非 400/422。

原因：路由层只 catch 了 CosyVoiceError 和 MediaKitError，其他异常（ValueError、voice_clone_repo 查询失败、DI 依赖异常等）被全局 general_exception_handler 捕获后返回 500。

改动

1. 路由层 catch ValueError → 400：参数无效（voice_id 格式不对、文本过长等）
2. 路由层 catch Exception 兜底 → 400：任何未预期错误返回 400 + 可读信息，不再 500
3. _resolve_voice_id 加 try/except：voice_clone_repo 查询失败时返回 400 + "voice_id 无效"
4. 新增 3 个错误处理测试

不改的

• Schema 验证仍然走 FastAPI 默认 422（Pydantic 字段缺失/类型错误）
• CosyVoiceError → 502（TTS 基础设施故障，非客户端问题）
• MediaKitError → 502（MediaKit API 故障）

测试

• 24 个 lipsync 测试全部通过
• 14698 个其他测试无回归

Relates #1809

## 问题 PR #1814 合并后，前端还在用旧参数调 /api/v1/lipsync/jobs，后端返回 500 而非 400/422。 原因：路由层只 catch 了 CosyVoiceError 和 MediaKitError，其他异常（ValueError、voice_clone_repo 查询失败、DI 依赖异常等）被全局 general_exception_handler 捕获后返回 500。 ## 改动 1. **路由层 catch ValueError → 400**：参数无效（voice_id 格式不对、文本过长等） 2. **路由层 catch Exception 兜底 → 400**：任何未预期错误返回 400 + 可读信息，不再 500 3. **_resolve_voice_id 加 try/except**：voice_clone_repo 查询失败时返回 400 + "voice_id 无效" 4. 新增 3 个错误处理测试 ## 不改的 - Schema 验证仍然走 FastAPI 默认 422（Pydantic 字段缺失/类型错误） - CosyVoiceError → 502（TTS 基础设施故障，非客户端问题） - MediaKitError → 502（MediaKit API 故障） ## 测试 - 24 个 lipsync 测试全部通过 - 14698 个其他测试无回归 Relates #1809

xiaoxia added 1 commit 2026-09-09 11:04:01 +08:00

fix: #1809 补充 对口型接口参数校验返回400而非500 ... 2902d12d3c

- 路由层新增 ValueError → 400 处理（参数无效场景）
- 路由层新增 Exception 兜底 → 400（防止任何未预期异常变成500）
- _resolve_voice_id 加 try/except 防止 voice_clone_repo 查询异常泄漏为500
- 新增 3 个错误处理测试用例（24 tests passed）

gitea-actions bot commented 2026-09-09 11:05:02 +08:00

Copy Link

Copy Source

🚀 预览环境已部署

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

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

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

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

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

auto-approve-bot merged commit c3023cc11b into develop 2026-09-09 11:12:09 +08:00

auto-approve-bot deleted branch fix/lipsync-error-handling-1809 2026-09-09 11:12:09 +08:00

gitea-actions bot commented 2026-09-09 11:12:36 +08:00

Copy Link

Copy Source

🗑️ 预览环境已清理

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

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

🗑️ **预览环境已清理** PR #1818 已关闭或合并，对应的预览环境已被清理。 > 如有需要，可以重新打开 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 frontend-dev xiaoxia

No Assignees

2 Participants

Notifications

Subscribe

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: xiaoxia/xiaoxia-saas#1818