Skip to content

Propagate streaming errors again#5772

Merged
GabrielBianconi merged 2 commits intomainfrom
gb/improve-streaming-errors
Jan 22, 2026
Merged

Propagate streaming errors again#5772
GabrielBianconi merged 2 commits intomainfrom
gb/improve-streaming-errors

Conversation

@GabrielBianconi
Copy link
Member

@GabrielBianconi GabrielBianconi commented Jan 21, 2026

Note

Strengthens error propagation so provider error details surface to clients.

  • Enhances convert_stream_error to include response body (when available) and request_id in messages; attaches raw_response to FatalStreamError
  • Improves inject_extra_request_data_and_send_eventsource_with_headers to read response bodies for InvalidStatusCode/InvalidContentType and return detailed FatalStreamError with raw_response
  • Adds e2e tests (tests/e2e/providers/errors.rs) verifying provider error messages propagate through HTTP and embedded gateways for both streaming and non-streaming (e.g., Anthropic "budget_tokens")

Written by Cursor Bugbot for commit 739f7ee. This will update automatically on new commits. Configure here.

@GabrielBianconi GabrielBianconi self-assigned this Jan 21, 2026
@virajmehta virajmehta enabled auto-merge January 21, 2026 20:12
@virajmehta virajmehta added this pull request to the merge queue Jan 21, 2026
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Jan 22, 2026
@GabrielBianconi GabrielBianconi added this pull request to the merge queue Jan 22, 2026
Merged via the queue into main with commit 60b19ef Jan 22, 2026
62 checks passed
@GabrielBianconi GabrielBianconi deleted the gb/improve-streaming-errors branch January 22, 2026 03:33
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.

2 participants