ChristopherSchultz commented on issue #232: Fix handling of query parameters with no value, like `?foo` URL: https://github.com/apache/tomcat/pull/232#issuecomment-573055772 I agree that `getParameter` should remain: a parameter present with no value should return `""` and not `null`. `getParameterValues` (and also `getParameterMap`) should return `new String[0]` if there are no values but the parameter is present, not `null`. Otherwise, there is no way to tell the difference between parameters which are present (e.g. `foo` in your example) and those that are not (e.g. `bar`, which isn't in your example URL). I think this means I disagree 100% with the premise of this PR.
---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org For additional commands, e-mail: dev-h...@tomcat.apache.org
