Skip to content

Only accept a chunk size at the start of the chunk-size line - #364

Open
mame wants to merge 1 commit into
ruby:masterfrom
mame:chunk-size-line
Open

mame wants to merge 1 commit into
ruby:masterfrom
mame:chunk-size-line

Conversation

@mame

@mame mame commented Nov 13, 2026

Copy link
Copy Markdown
Member

read_chunked took the first run of hex digits anywhere in the line, so lines such as "0x1f" or ";ext=1f" were read as different sizes. Require the size at the start of the line, followed only by optional whitespace and a chunk extension, as RFC 9112 does.

Co-authored-by: Clément Joubert (https://hackerone.com/czjfr)

read_chunked took the first run of hex digits anywhere in the line, so
lines such as "0x1f" or ";ext=1f" were read as different sizes.  Require
the size at the start of the line, followed only by optional whitespace
and a chunk extension, as RFC 9112 does.

Co-authored-by: Clément Joubert (https://hackerone.com/czjfr)
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

This branch has not been deployed

No deployments
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.

1 participant