Skip to content

Issue 70 asgi body cap - #80

Merged
ZhuchkaTriplesix merged 5 commits into
devfrom
issue-70-asgi-body-cap
Apr 27, 2026
Merged

ZhuchkaTriplesix merged 5 commits into
devfrom
issue-70-asgi-body-cap

Conversation

@ZhuchkaTriplesix

@ZhuchkaTriplesix ZhuchkaTriplesix commented Apr 27, 2026 •

Copy link
Copy Markdown
Member

Summary

  • What this PR does (1–3 sentences).
  • If it closes a ticket: Closes #N (replace N).

Base branch

  • This PR targets dev, not main (unless maintainers asked for a hotfix).

Checklist

  • cargo build (and cargo clippy if you touched Rust)
  • maturin develop + tests as in docs/development.md (e.g. pytest from a clean cwd / installed wheel)
  • No unrelated drive-by refactors; commits are atomic and scoped

Note on .github/ISSUE_BACKLOG/

If you only touch issue template files under .github/ISSUE_BACKLOG/ (not application code), say so in the summary. Prefer a separate PR for backlog-only edits when possible.

…dy cap

Switch ASGI body collection to linear bytearray accumulation and stop oversized chunked requests during read with a 413 response aligned to existing payload-too-large semantics.
Add regression tests for chunked JSON body reconstruction and early 413 behavior when chunked input exceeds the configured ASGI body limit.
Add non-http scope behavior checks and a dedicated suite for ASGI body-limit parsing, boundary handling, and disconnect semantics.
Add websocket edge-case tests and unit-level coverage for CORS, CSRF, HTTPException, security headers, and SSE helpers.
@ZhuchkaTriplesix ZhuchkaTriplesix added this to the v0.3.0 milestone Apr 27, 2026
@ZhuchkaTriplesix ZhuchkaTriplesix self-assigned this Apr 27, 2026

@ZhuchkaTriplesix ZhuchkaTriplesix left a comment

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

123

@ZhuchkaTriplesix
ZhuchkaTriplesix merged commit 8779750 into dev Apr 27, 2026
16 checks passed
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.

perf(asgi): avoid quadratic request body aggregation and add explicit ASGI body cap

1 participant