Ya I was wondering if that was the case. I saw the duplicate path, removed
the non-pre instance, and the unit tests still passed, so I thought maybe
it would be OK to pull. Perhaps just a comment would be more helpful
instead?

Thanks,
~pete
On Feb 10, 2014 12:51 AM, "Gergely Nemeth" <notifications@github.com> wrote:

> In lib/plugins/index.js:
>
> > @@ -18,7 +18,6 @@ module.exports = {
> >      jsonBodyParser: require('./json_body_parser'),
> >      multipartBodyParser: require('./multipart_parser'),
> >      queryParser: require('./query'),
> > -    sanitizePath: require('./pre/pre_path'),
>
> I am nut sure if it is a good idea to remove it, as it restify users may
> already depend on the restify.plugins. sanitizePath export as well as on
> the restify.plugins.pre.sanitizePath
>
> --
> Reply to this email directly or view it on GitHub<https://github.com/mcavage/node-restify/pull/527/files#r9575303>
> .
>