Make bodyParser merging into req.params optional about node-restify HOT 6 CLOSED

Oh, and I should add, I'm happy to code this up and submit a pull request if you think it is something you'd like to see in restify.

from node-restify.

Hi Damon,

So - what you're looking for is for req.body to just be the parsed JS object (currently it's the raw content from the client, which for JSON is probably not that useful I suppose)? Or you want it somewhere else? This is pretty easy to change, and I'm happy to do that (doing what you said and not hardcoding it) - the reason it's the way it is so that in simple cases a server doesn't need to care about whether the client sent params in the query string or JSON (admittedly that only works for k/v parameters, not complex objects).

Anyway, assuming that's what you're looking for, I can get that in today (i.e., just parse req.body in place). Thanks for using it!

m

from node-restify.

Hi Damon,

Dropped this in, please take a look and let me know if that works for you (I also updated the docs to cover all the plugins, which covers this: http://mcavage.github.com/node-restify/v1.0/guide.html#Bundled-Plugins) - I made it opt-out, since everybody else using restify would be broken by this behavior.

m

from node-restify.

Thanks Mark - I figured opt-out would probably be the best way to go. I'll have a look.

from node-restify.

Worked a treat, and love the new docs too :)

Thanks again.

from node-restify.

Awesome, and n/p.

from node-restify.