#48 Use req.params instead of body and query in lookup

[krampstudio]

To fix issue #48

[jaredhanson]

Thanks. req.params is typically created by express (and I'm assuming restify). For anyone just using Connect's bodyParser and query middleware, it won't be available and this will break that. Do you have a solution in mind for that? Just check all three places?

Also, can you point me to any docs as to what properties restify sets up on the req and res, prior to dispatching to the application?

[jaredhanson]

Rejected due to reasons detailed above. Closing.

[johnmave126]

Just a quick note if anyone else encounter the same problem
https://github.com/mcavage/node-restify/blob/master/lib/plugins/json_body_parser.js#L42
https://github.com/mcavage/node-restify/blob/master/lib/plugins/form_body_parser.js#L43

Remember to use
server.use(restify.bodyParser({mapParams: false});
to parse params into req.body to comply with passport-local