Error handling: array in lieu of query string

[divergentdave]

Following up from #139, http://localhost:3000/reports?query=&query= results in a query parsing exception from elasticsearch, because express returns an array for query, and the query_string query doesn't support arrays. It looks like we might have produced bogus links at some point in the past, where the whole query string was repeated two or three times, and the search engine crawlers picked up on that.

• [x] Handle repeated query argument, either pick one or join them all
• [x] Look into old search page bugs that might have caused this

Edit: example: /reports.xml?page=18&query=&inspector=education?page=18&query=&inspector=education

[divergentdave]

After some extensive searching, I can't figure out what would have led to doubling the query string. My best hypothesis is that it came from logic (or a bug) in the search engine crawler.