Yonik Seeley wrote:
On Dec 3, 2007 12:44 AM, Ryan McKinley <[EMAIL PROTECTED]> wrote:
In trunk, check:
http://localhost:8983/solr/select
  or
http://localhost:8983/solr/select?q=

In 1.2, this returns 400, "Missing required parameter: q"

Fix could be easy -- add a check in QueryComponent.prepare(), the hitch
is that for qt=dismax, q is *not* a required param.

Could add the check in (Old)LuceneQParser.parse() for a null query
string, and could treat a zero length query string the same way I
guess.


that sounds good -- each QParser would be responsible to make sure the qstr is valid. We should also make sure the other QParsers give resonable errors for bad input.

ryan

Reply via email to