Skip to content

Conversation

@alexander-alderman-webb
Copy link
Contributor

@alexander-alderman-webb alexander-alderman-webb commented Jan 27, 2026

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

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.

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.

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
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.

SDK Handling HTTP 413 [Python]

3 participants