Support for semicolon separators

tj / node-querystring

querystring parser for node and the browser - supporting nesting (used by Express, Connect, etc)

MIT License

455 stars 66 forks source link

Support for semicolon separators #101

Closed stuartmcfarlane closed 10 years ago

stuartmcfarlane commented 10 years ago

As far as I can tell W3C encourages server implementors to support the use of semicolons as a query separator.

sebs commented 10 years ago

Looked and the changeset and liked it

buschtoens commented 10 years ago

I'd rather not only support & and ;. If we're going down the options road, we should have options.separator or options.delimeter (defaults to &).

buschtoens commented 10 years ago

Also #1.

tj commented 10 years ago

I've never really seen anyone use ; in practice, but I agree with @silvinci, if it's going to be done it should just be an option

jonathanong commented 10 years ago

@stuartmcfarlane sorry for the delay. can you squash please?

jonathanong commented 10 years ago

landed in https://github.com/visionmedia/node-querystring/commit/d4287d319a8ab4a86e6048b79331a8eb90c03b45