Author: remy.maucherat(a)jboss.com Date: 2010-08-27 07:54:53 -0400 (Fri, 27 Aug 2010) New Revision: 1546 Modified: trunk/java/org/apache/catalina/connector/Request.java trunk/webapps/docs/changelog.xml Log: - Port a patch: add back a fix I had reverted years ago. Modified: trunk/java/org/apache/catalina/connector/Request.java =================================================================== --- trunk/java/org/apache/catalina/connector/Request.java 2010-08-26 15:15:51 UTC (rev 1545) +++ trunk/java/org/apache/catalina/connector/Request.java 2010-08-27 11:54:53 UTC (rev 1546) @@ -2178,12 +2178,7 @@ * Return the query string associated with this request. */ public String getQueryString() { - String queryString = coyoteRequest.queryString().toString(); - if (queryString == null || queryString.equals("")) { - return (null); - } else { - return queryString; - } + return coyoteRequest.queryString().toString(); } Modified: trunk/webapps/docs/changelog.xml =================================================================== --- trunk/webapps/docs/changelog.xml 2010-08-26 15:15:51 UTC (rev 1545) +++ trunk/webapps/docs/changelog.xml 2010-08-27 11:54:53 UTC (rev 1546) @@ -48,6 +48,10 @@ <bug>49428</bug>: Specify a namespace rather than use a default to please MS clients. Submitted by Panagiotis Astithas. (markt) </fix> + <fix> + <bug>38113</bug>: Empty query Strings should be acceptable. This may be incompatible with + some old non compliant applications. (markt) + </fix> </changelog> </subsection> <subsection name="Coyote">