MockHttpServletRequest fails to handle parameters with empty values #1515

Closed
dmclean62 opened this Issue Jan 23, 2014 · 1 comment

3 participants

@dmclean62

It is legal, and common, for get requests in the real world to specify parameters with empty values, either using "parameter=" or leaving out the = and just including the name. Unfortunately, MockHttpServletRequest throws an exception when encountering either of these forms, throwing an "invalid query string" exception.

Invalid query string: "=1389304329528&sEcho=7&sSearch=&iDisplayLength=10"
java.lang.IllegalArgumentException: Invalid query string: "
=1389304329528&sEcho=7&sSearch=&iDisplayLength=10"
at net.liftweb.mocks.MockHttpServletRequest$$anonfun$queryString_$eq$1.apply(MockHttpServletRequest.scala:219)

ML conversation: https://groups.google.com/d/msg/liftweb/TNIJoeSxFUM/ctsx_VOqXOIJ .

@Shadowfiend Shadowfiend modified the milestone: 2.6-M4 Apr 9, 2014
@Shadowfiend
Lift Web Framework member

This was taken care of by #1516.

@Shadowfiend Shadowfiend closed this Apr 9, 2014
@fmpwizard fmpwizard added this to the 2.6-M3 milestone Apr 9, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment