Skip to content

feat: add error log for oversized crash reports (HTTP 413)#155

Merged
jpnurmi merged 3 commits intogetsentryfrom
jpnurmi/feat/http-413
May 7, 2026
Merged

feat: add error log for oversized crash reports (HTTP 413)#155
jpnurmi merged 3 commits intogetsentryfrom
jpnurmi/feat/http-413

Conversation

@jpnurmi
Copy link
Copy Markdown
Collaborator

@jpnurmi jpnurmi commented May 7, 2026

Same as getsentry/sentry-native#1647 but for crashpad. The /minidump endpoint was fixed to respond with 413 in getsentry/relay#5838.

Closes: getsentry/sentry-native#1487

Co-Authored-By: OpenAI Codex <noreply@openai.com>
@jpnurmi jpnurmi requested review from JoshuaMoelans and mujacica May 7, 2026 12:49
Copy link
Copy Markdown

@cursor cursor Bot left a comment

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit 1e88ec6. Configure here.

Comment thread util/net/http_transport_mac.mm
@jpnurmi jpnurmi merged commit a43f8fb into getsentry May 7, 2026
23 checks passed
@jpnurmi jpnurmi deleted the jpnurmi/feat/http-413 branch May 7, 2026 13:13
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 [Native]

2 participants