fix(task): preserve selected key for multi-key polling - #6870
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (2)
Walkthrough
ChangesTask key preservation
Estimated code review effort: 2 (Simple) | ~10 minutes Merge Risk: ⚪ Minimal · up to The change preserves the selected API key for asynchronous polling while keeping single-key and existing Gemini/Vertex AI behavior unchanged; no actionable merge-blocking risk remains after normal checks and review. Suggested reviewers: Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Important
📝 变更描述 / Description
多密钥渠道提交异步任务时,分发器已经选出了本次请求实际使用的 API key,但任务记录只为 Gemini/Vertex AI 保存该 key。阿里视频任务因此在后台轮询时回退到包含多个换行分隔 key 的渠道原始字段,写入
Authorization后触发非法 header 错误。本 PR 在初始化多密钥任务时保存本次请求实际选中的 key。现有轮询逻辑会优先读取该私有字段,因此后续查询使用与任务提交时相同的凭据;单密钥渠道行为保持不变。
新增表驱动回归测试,覆盖多密钥、普通单密钥以及 Gemini/Vertex AI 既有行为。
🚀 变更类型 / Type of change
🔗 关联任务 / Related Issue
✅ 提交前检查项 / Checklist
Bug fix,我已提交或关联对应 Issue,且不会将设计取舍、预期不一致或理解偏差直接归类为 bug。📸 运行证明 / Proof of Work
新增测试在修复前可以稳定复现失败:
应用修复后:
补充说明:本 PR 代码为 AI-assisted 生成,并已人工复核、执行回归测试及质量检查。
Summary by CodeRabbit
Bug Fixes
Tests