duplicate invalid param 'text' warning for /v1/search

[Peter Johnson]

/v1/search

/v1/autocomplete

[Julian Simioni]

Wow, never noticed that before. I wonder where it comes from.

[Peter Johnson]

here?

• https://github.com/pelias/api/blob/master/sanitizer/defer_to_pelias_parser.js#L21
• https://github.com/pelias/api/blob/master/routes/v1.js#L215-L224

[Peter Johnson]

I think the core issue is that the 'combining sanitizers' (such as https://github.com/pelias/api/blob/master/sanitizer/search.js) should return immediately upon error rather than running through the rest of the route chain.

[Peter Johnson]

ie. here https://github.com/pelias/api/blob/master/sanitizer/sanitizeAll.js#L94 it should return something to trigger the err in express middleware (req, res, err).