Sitelet https://github.com/rack/rack/pull/2512
Skip to content

Backport #2416 to 3-2-stable. - #2512

Open
Skulli wants to merge 1 commit into
rack:3-2-stablefrom
Skulli:backport-2416-3-2-stable
Open

Skulli wants to merge 1 commit into
rack:3-2-stablefrom
Skulli:backport-2416-3-2-stable

Conversation

@Skulli

@Skulli Skulli commented Sep 21, 2026

Copy link
Copy Markdown

Backport of #2416 to 3-2-stable, as agreed in #2510.

Released versions still raise a bare Encoding::CompatibilityError out of Request#POST
for a multipart part whose charset is not ASCII compatible. Since that is not a
Rack::BadRequest, middleware that guards param parsing — including Rack::MethodOverride
— cannot rescue it, and it escapes the application's error handling entirely.

Clean cherry-pick apart from the changelog entry, which is placed under the existing
### Fixed heading here.

Not backported to 3-1-stable, per @jeremyevans: that branch is in security maintenance
mode.

🤖 Generated with Claude Code

The query parser now raises `Rack::QueryParser::IncompatibleEncodingError`
if we try to parse params that are not ASCII compatible.

(cherry picked from commit 603b799)

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.

2 participants