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

Use Rack::QueryParser with nil missing value. #47652

Closed
wants to merge 3 commits into from
Closed

Use Rack::QueryParser with nil missing value. #47652

wants to merge 3 commits into from

Commits on Mar 13, 2023

  1. Use Rack::QueryParser with nil missing value. 

    Rack 3 follows the WhatWG standard for the query parser. This deviates from
Rack 2 as `?a` as a query string is was previously interpreted as
`{"a" => nil}` but is now interpreted as `{"a" => ""}`. Rails depends on
the previous behaviour for compatibility, so we introduced hooks into Rack
to restore compatibility.
    @ioquatix
    ioquatix committed Mar 13, 2023
    Configuration menu
    Copy the full SHA
    f336b47 View commit details
    Browse the repository at this point in the history

  2. Use keyed caches and fix tests.

    @ioquatix
    ioquatix committed Mar 13, 2023
    Configuration menu
    Copy the full SHA
    ea4eac4 View commit details
    Browse the repository at this point in the history

Commits on Mar 14, 2023

  1. Update for the latest PR in Rack.

    @ioquatix
    ioquatix committed Mar 14, 2023
    Configuration menu
    Copy the full SHA
    8afd5c6 View commit details
    Browse the repository at this point in the history