@@ -37,7 +37,7 @@ module Rack params = {} (qs || '').split(d ? /[#{d}] */n : DEFAULT_SEP).each do |p| - k, v = unescape(p).split('=', 2) + k, v = p.split('=', 2).map { |x| unescape(x) } if cur = params[k] if cur.class == Array lib/rack/utils.rb @@ -28,6 +28,7 @@ context "Rack::Utils" do should.equal "foo" => "1", "bar" => "2" Rack::Utils.parse_query("my+weird+field=q1%212%22%27w%245%267%2Fz8%29%3F"). should.equal "my weird field" => "q1!2\"'w$5&7/z8)?" + Rack::Utils.parse_query("foo%3Dbaz=bar").should.equal "foo=baz" => "bar" end specify "should parse nested query strings correctly" do test/spec_rack_utils.rb 0bc566223c735d9efa3a1d0d925e8d7001760b85 tlrobinson tom@280north.com http://github.com/chneukirchen/rack/commit/7a55c98a30deea7fe198958d7c2770b087aa94b9 7a55c98a30deea7fe198958d7c2770b087aa94b9 2009-05-23T05:25:28-07:00 2009-05-22T15:46:01-07:00 Fix for form names containing "=": split first then unescape components Signed-off-by: Christian Neukirchen <chneukirchen@gmail.com> d941f078e694dfd69f269ecbf429fe921f471c17 Christian Neukirchen chneukirchen@gmail.com