Skip to content

fix(requestconfig): parse top-level API error payloads#656

Open
fengfeng-zi wants to merge 1 commit into
openai:mainfrom
fengfeng-zi:fix/apierror-top-level-payload-644
Open

fix(requestconfig): parse top-level API error payloads#656
fengfeng-zi wants to merge 1 commit into
openai:mainfrom
fengfeng-zi:fix/apierror-top-level-payload-644

Conversation

@fengfeng-zi
Copy link
Copy Markdown

Summary

  • fall back to decoding the full response body when a downstream provider returns top-level code / message error JSON
  • keep preferring nested error objects when present in OpenAI-style payloads
  • add focused regression tests for both top-level and nested error payloads

Closes #644

Testing

  • Not run locally: go is not installed in this environment

@fengfeng-zi fengfeng-zi requested a review from a team as a code owner May 4, 2026 08:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

apierror.Error has empty Message and Code for downstream 404 errors

1 participant