From 7ce02798782b1f69ccf69fc75b8f30ab85226adb Mon Sep 17 00:00:00 2001 From: Jacob Ward Date: Mon, 6 Feb 2017 23:42:00 +0000 Subject: [PATCH] Fix duplicate query params error (Issue: #352) (#465) --- src/server/router/plural.js | 4 ++++ test/server/plural.js | 8 ++++++++ 2 files changed, 12 insertions(+) diff --git a/src/server/router/plural.js b/src/server/router/plural.js index 0c8b117..5ac910c 100644 --- a/src/server/router/plural.js +++ b/src/server/router/plural.js @@ -92,6 +92,10 @@ module.exports = (db, name) => { if (q) { // Full-text search + if (Array.isArray(q)) { + q = q[0] + } + q = q.toLowerCase() chain = chain.filter((obj) => { diff --git a/test/server/plural.js b/test/server/plural.js index 8e1c55f..ca918bc 100644 --- a/test/server/plural.js +++ b/test/server/plural.js @@ -198,6 +198,14 @@ describe('Server', () => { .expect(200, done) }) + it('should ignore duplicate q query parameters', (done) => { + request(server) + .get('/comments?q=foo&q=bar') + .expect('Content-Type', /json/) + .expect([ db.comments[0] ]) + .expect(200, done) + }) + it('should support filtering by boolean value false', (done) => { request(server) .get('/comments?published=false')