## 上游来源 - PR:https://github.com/QuantumNous/new-api/pull/5817 - 标题:`fix: close response bodies on non-OK responses` - 记录时提交:`5839d84f6ff288f3ebcbd5cd68fff830bacdaf76` - 状态:Open / DIRTY;4 files,+65/-14;CodeRabbit 通过 ## 轻量审查 - **有用价值:中高。** 修复余额查询、模型拉取、Suno 轮询在 401/429/500 等路径不关闭 body,避免连接复用受阻和资源逐步泄漏。 - **合理程度:高。** `client.Do` 成功后立即 defer close 是标准做法,改动集中,并补了非 200 body 被关闭的回归测试。 - **当前建议:可优先跟踪。** ## 引入前关注点 - 当前分支已 DIRTY,应在最新 main 上检查这些路径是否已被其他重构覆盖。 - `GetResponseBody` 把 `Close` error 作为成功调用的 error 返回是否符合调用方预期;通常可记录而非覆盖成功结果。 - 搜索同类 `Do` 后先 status-return 再 defer close 的剩余路径,避免只修一部分。 ## 跟踪策略 - [ ] 关注上游解决冲突及 merge 状态 - [ ] 上游合并后随正式上游更新引入 - [ ] 若长期未处理,在最新代码上重做最小修复 - [ ] 引入前运行 controller/service 回归
上游来源
fix: close response bodies on non-OK responses5839d84f6ff288f3ebcbd5cd68fff830bacdaf76轻量审查
client.Do成功后立即 defer close 是标准做法,改动集中,并补了非 200 body 被关闭的回归测试。引入前关注点
GetResponseBody把Closeerror 作为成功调用的 error 返回是否符合调用方预期;通常可记录而非覆盖成功结果。Do后先 status-return 再 defer close 的剩余路径,避免只修一部分。跟踪策略