Repository navigation
Static files, write cap, slow-client + pipelining tests - #8
Merged
Merged
Conversation
…t tests
StaticFiles: strict percent-decode before resolution; traversal/encoded/backslash/absolute refused 404 pre-disk; weakly_canonical symlink containment, case-folded on Windows; index, MIME, 405, HEAD honest Content-Length. Handler now returns Reply{status,content_type,body}; handler exceptions become 500 at the reactor seam.
Slow-client limits: per-connection max_queued_bytes caps the output buffer; a throttle stops reading (real flow control), POLLOUT or next read resumes the drain. Slow-loris partial head -> 408 test; 10-request pipelined burst through a 100-byte cap test.
Test harness: readResponse now honors carry-over of coalesced pipelined responses (recv-after-check, not recv-before-check). Container smoke serves a scratch root and asserts traversal 404.
Tree-wide clang-format applied (175 latent violations; clang-format 23.1.2 now on PATH); ck.ps1 format gate is live.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
StaticFiles handler (decode-before-resolve, traversal/absolute/backslash 404, weakly_canonical + case-folded containment, index/MIME/405/HEAD). Reply-based handlers, 500 seam guard. max_queued_bytes throttle with POLLOUT/read resume; slow-loris 408 + 10x burst under 100-byte cap tests. Fixed readResponse carry bug. clang-format applied tree-wide (gate now live, clang-format on PATH). Gates: Windows ck.ps1 green (format check ran), Linux ASan+UBSan container green incl. updated smoke. CI adds TSan.