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

--- Comment #4 from Mark Thomas <ma...@apache.org> ---
A quick update:

1.
getMethod() is already effectively cached due to use of
MessageBytes.toStringType()

getQueryString() could be cached but there is a potential performance issue. If
applications parse the query string manually or use getParameter() and friends
all is well. If an application uses both, using MessageBytes.toStringType()
could trigger bytes -> String -> bytes if getQueryString() is called before
getParameter() and friends. I'm minded to leave this one alone for now.

requestURI(), contentType() and headerNames can all switch to
MessageBytes.toStringType(). I'll do that shortly.

2. Waiting for examples.

3. Cannot reproduce. I've reviewed the code and manually debugged an included
JSP. The original query string is parsed once and the additional query string
is parsed once.

-- 
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