Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

add a test to ensure that chunked encoding obeys request size limits #620

Open
wants to merge 1 commit into
base: sunshowers/spr/main.add-a-test-to-ensure-that-chunked-encoding-obeys-request-size-limits
Choose a base branch
from

Conversation

sunshowers
Copy link
Contributor

@sunshowers sunshowers commented Mar 21, 2023

There were no functional changes required -- chunked encoding already
obeys request size limits. Add a test which checks that.

Also simplify the test to not have to create a temporary file, instead
serving a stream purely from memory.

Depends on #617 and #618, since the test uses the StreamingBody and max
request size functionality introduced in those two PRs respectively.

Closes #545.

Created using spr 1.3.4
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.

None yet

1 participant