Thanks to visit codestin.com
Credit goes to github.com

Skip to content

feat(transport): Report 413 responses for oversized envelopes#5380

Merged
alexander-alderman-webb merged 3 commits intomasterfrom
webb/handle-413
Jan 27, 2026
Merged

feat(transport): Report 413 responses for oversized envelopes#5380
alexander-alderman-webb merged 3 commits intomasterfrom
webb/handle-413

Conversation

@alexander-alderman-webb
Copy link
Contributor

@alexander-alderman-webb alexander-alderman-webb commented Jan 27, 2026 •
edited
Loading

Description

Log an error message an record a client report with reason send_error when downstream envelope size limits are exceeded.

Issues

Closes #5361

Reminders

@alexander-alderman-webb alexander-alderman-webb requested a review from a team as a code owner January 27, 2026 09:00
@github-actions
Copy link
Contributor

github-actions bot commented Jan 27, 2026 •
edited
Loading

Semver Impact of This PR

🟡 Minor (new features)

📋 Changelog Preview

This is how your changes will appear in the changelog.
Entries from this PR are highlighted with a left border (blockquote style).

New Features ✨

Integrations

  • feat(integrations): OpenAI/OpenAI Agents detect and report the time to first token metric (TTFT) as gen_ai.response.time_to_first_token by constantinius in #5348
  • feat(integrations): openai-agents streaming support by constantinius in #5291

Other

  • feat(anthropic): Set system instruction attribute by alexander-alderman-webb in #5353
  • feat(asyncio): Allow to turn task spans off by sentrivana in #5367
  • feat(gen_ai): add function set_conversation_id and managing functions on the Scope and apply it on the Span on .finish() by constantinius in #5362
  • feat(google-genai): Set system instruction attribute by alexander-alderman-webb in #5354
  • feat(langchain): Set system instruction attribute by alexander-alderman-webb in #5357
  • feat(openai-agents): Set system instruction attribute by alexander-alderman-webb in #5355
  • feat(pydantic-ai): Set system instruction attribute by alexander-alderman-webb in #5356
  • feat(transport): Report 413 responses for oversized envelopes by alexander-alderman-webb in #5380

Internal Changes 🔧

  • ci(release): Fix changelog-preview permissions by BYK in #5368
  • ci: Fix path in AI integration tests by alexander-alderman-webb in #5347

🤖 This preview updates automatically when you update the PR.

sentrivana
sentrivana reviewed Jan 27, 2026
View reviewed changes
Copy link
Contributor

@sentrivana sentrivana left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, but let's also please add a test case that checks that we send the client report. There are some existing tests for the 429 behavior which can be used for inspiration.

sentrivana
sentrivana approved these changes Jan 27, 2026
View reviewed changes
Copy link
Contributor

@sentrivana sentrivana left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ty!

@alexander-alderman-webb alexander-alderman-webb merged commit 0d42476 into master Jan 27, 2026
153 checks passed
@alexander-alderman-webb alexander-alderman-webb deleted the webb/handle-413 branch January 27, 2026 15:53
This was referenced Jan 28, 2026
Closed
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@sentrivana sentrivana sentrivana approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

SDK Handling HTTP 413 [Python]

2 participants

@alexander-alderman-webb @sentrivana