Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Make query parameters without = have empty string values #2098

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Make query parameters without = have empty string values #2098

wants to merge 1 commit into from

Commits on Jul 19, 2023

  1. Make query parameters without = have empty string values 

    This was reverted in 77cf057
so that Rails could have historical Rack behavior without the
complexity of splitting form/query parsing.

Now that splitting form/query parsing has been added back in
3855d1d, this changes the
behavior to what originally shipped in Rack 3.

This matches URL spec section 5.1.3.3.  Frameworks that want
Rack's historical behavior of using nil values instead of
empty string values can reparse QUERY_STRING and use
rack.request.form_pairs to get the behavior they want.
    @jeremyevans
    jeremyevans committed Jul 19, 2023
    Configuration menu
    Copy the full SHA
    d28fa2a View commit details
    Browse the repository at this point in the history