https://bz.apache.org/bugzilla/show_bug.cgi?id=62739

--- Comment #8 from Mark Thomas <ma...@apache.org> ---
Patch looks good - thanks for including the tests.

My main thought at this point is what should HttpServletRequest.getServerName()
return? My reading of the Javadoc is if the Host header is present, its value
should be returned by HttpServletRequest.getServerName(). That requires a
slightly different fix (in the parser to allow empty values).

Either solution enables the new tests to pass. Leaving this open for a little
while to think about it.

-- 
You are receiving this mail because:
You are the assignee for the bug.
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org

Reply via email to