Add Support for Nested Params Parsing in Rack::Request
Reported by Matt Todd | December 28th, 2008 @ 07:49 AM | in 1.0
Adding ticket for conversation going on at:
http://groups.google.com/group/r...
The conversation covers numerous topics so this ticket reflects specifically the proposal for parsing nested params.
To clarify, request params, when encoded in application/x-www-form-urlencoded, can be specified simply as a=b&x=y (flat) as well as a[b]=c&a[d]=e (nested) values.
The proposal is to include parsing these nested values into a
Hash instead of just including top-level keys as a[b]
and a[c] only.
Refer to the RFC for the specification of a query as between the @?@ and the @#@ of the URL:
http://tools.ietf.org/html/rfc39....4
Unfortunately, the RFC does not specify further regarding nested pairs.
Michael Fellinger (manveru) provided this implementation:
Comments and changes to this ticket
-
josh January 18th, 2009 @ 02:43 AM
- Milestone set to “1.0”
We really need Rack to support this. Rails, Merb, Ramaze, and Sinatra all need to implement it themselves. This should not be an extension or a flaggable option either.
http://github.com/bmizerany/sina...
Not to get all Railsy, but it would be nice if we picked an implementation that is compatible with Rails and keeps all our tests passing :) Yehuda said something about Merb's doing different things at deeper nesting levels.
+1
-
josh January 19th, 2009 @ 09:16 PM
- Assigned user set to “josh”
Ported Merbs params parser over. It still needs some polish.
Test coverage supports all the the features we need for Rails.
-
-
-
josh January 28th, 2009 @ 08:19 PM
- no changes were found...
-
josh January 29th, 2009 @ 06:58 PM
- no changes were found...
-
josh February 1st, 2009 @ 05:35 AM
- State changed from “new” to “resolved”
Please Sign in or create a free account to add a new ticket.
With your very own profile, you can contribute to projects, track your activity, watch tickets, receive and update tickets through your email and much more.
Create new ticket
josh
manveru
Matt Todd
Mislav