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.

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

querystring stuff #5

Open
jonathanong opened this issue Aug 14, 2014 · 5 comments
Open

querystring stuff #5

jonathanong opened this issue Aug 14, 2014 · 5 comments
Assignees
Labels

Comments

@jonathanong
Copy link
Member

i think @dougwilson and i discussed a querystring parser/stringifier that is more configurable and strict. specifically, i only ever want strings. haven't looked at the issue right now.

@dougwilson dougwilson self-assigned this Aug 14, 2014
@dougwilson
Copy link

+1000; basically a way to get predictable types back

@Raynos
Copy link

Raynos commented Aug 14, 2014

👍 We use querystring build into node core and then use our JSON schema definition of the HTTP request so that we can get predictable types.

We return a validation error from the querystring parser if it doesnt match the schema.

@rlidwka
Copy link

rlidwka commented Aug 14, 2014

yeah, maybe just add a validation/transformation step and use existing querystring module?

@dougwilson
Copy link

We could at first, but it would make it slightly slower :)

@Fishrock123
Copy link
Member

👍 from all the people who don't need all the extra qs stuff. (fast would be nice though...)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

5 participants