Skip to content

Handle multipart headers case-insensitively#252

Merged
Kludex merged 1 commit intomasterfrom
codex/lowercase-multipart-headers
Apr 10, 2026
Merged

Handle multipart headers case-insensitively#252
Kludex merged 1 commit intomasterfrom
codex/lowercase-multipart-headers

Conversation

@Kludex
Copy link
Copy Markdown
Owner

@Kludex Kludex commented Apr 10, 2026

Normalize multipart part header names to lower-case bytes before storing them internally, then look up content-disposition and content-transfer-encoding using lower-case keys.

This keeps the parser aligned with MIME multipart behavior and matches the approach used by Django for multipart part headers. It avoids converting header names to strings or title-casing them.


@Kludex Kludex force-pushed the codex/lowercase-multipart-headers branch from 8d25fa8 to 2b87947 Compare April 10, 2026 08:34
@Kludex Kludex changed the title [codex] Handle multipart headers case-insensitively Handle multipart headers case-insensitively Apr 10, 2026
@Kludex Kludex marked this pull request as ready for review April 10, 2026 08:38
@Kludex Kludex merged commit d8989ba into master Apr 10, 2026
12 checks passed
@Kludex Kludex deleted the codex/lowercase-multipart-headers branch April 10, 2026 08:41
@Kludex Kludex mentioned this pull request Apr 10, 2026
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.

Headers are being treated case sensitive

1 participant