z0mt3c / node-restify-validation

Validation for REST Services built with node-restify in node.js
MIT License
91 stars 49 forks source link

Fixed unit tests and major bug concerning query-string validation #50

Closed schaermu closed 8 years ago

schaermu commented 8 years ago

Since Restify 2.x, the query-string parameters have to be fetched using req.query() instead of req.query. While fixing the unit tests, i also upgraded the validator package to 3.22.x due to a major security issue in all versions below (see https://blog.liftsecurity.io/2014/11/03/regular-expression-dos-and-node.js).

gchauvet commented 8 years ago

Seems good for me, I merge it. I wait 1 week before pushing release on npm