Skip to content

httputil: Reject header lines beginning with invalid whitespace#3488

Merged
bdarnell merged 1 commit intotornadoweb:masterfrom
bdarnell:obs-fold
Apr 25, 2025
Merged

httputil: Reject header lines beginning with invalid whitespace#3488
bdarnell merged 1 commit intotornadoweb:masterfrom
bdarnell:obs-fold

Conversation

@bdarnell
Copy link
Member

The obs-fold feature is defined only for tabs and spaces. The str.isspace() method also accepts other whitespace characters. These characters are not valid in HTTP headers and should be treated as errors instead of triggering line folding.

Fixes #3480

The obs-fold feature is defined only for tabs and spaces.
The str.isspace() method also accepts other whitespace characters.
These characters are not valid in HTTP headers and should be treated
as errors instead of triggering line folding.

Fixes tornadoweb#3480
@bdarnell bdarnell merged commit 5976db7 into tornadoweb:master Apr 25, 2025
15 checks passed
@bdarnell bdarnell deleted the obs-fold branch April 25, 2025 19:46
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.

obs-fold is implemented incorrectly

1 participant