This is an automated email from the ASF dual-hosted git repository.
markt pushed a commit to branch 10.1.x
in repository https://gitbox.apache.org/repos/asf/tomcat.git
The following commit(s) were added to refs/heads/10.1.x by this push:
new 59a2ce23f5 Complete fix for BZ 69285. Optimize creation of ParameterMap
59a2ce23f5 is described below
commit 59a2ce23f5ee0d3aba078e6ee234fab1f2ce9331
Author: Mark Thomas <ma...@apache.org>
AuthorDate: Fri Jan 31 11:56:13 2025 +0000
Complete fix for BZ 69285. Optimize creation of ParameterMap
Based on sample code and test cases provided by John Engebretson
---
java/org/apache/catalina/core/ApplicationHttpRequest.java | 7 ++++++-
webapps/docs/changelog.xml | 5 +++++
2 files changed, 11 insertions(+), 1 deletion(-)
diff --git a/java/org/apache/catalina/core/ApplicationHttpRequest.java
b/java/org/apache/catalina/core/ApplicationHttpRequest.java
index 50f594d6b8..af3eea1fb4 100644
--- a/java/org/apache/catalina/core/ApplicationHttpRequest.java
+++ b/java/org/apache/catalina/core/ApplicationHttpRequest.java
@@ -732,7 +732,12 @@ class ApplicationHttpRequest extends
HttpServletRequestWrapper {
return;
}
- parameters = new ParameterMap<>(getRequest().getParameterMap());
+ Map<String,String[]> requestParameters =
getRequest().getParameterMap();
+ if (requestParameters instanceof ParameterMap<String,String[]>) {
+ parameters = new ParameterMap<>((ParameterMap<String,String[]>)
requestParameters);
+ } else {
+ parameters = new ParameterMap<>(requestParameters);
+ }
mergeParameters();
((ParameterMap<String,String[]>) parameters).setLocked(true);
parsedParams = true;
diff --git a/webapps/docs/changelog.xml b/webapps/docs/changelog.xml
index f9e571896b..caadc5721c 100644
--- a/webapps/docs/changelog.xml
+++ b/webapps/docs/changelog.xml
@@ -123,6 +123,11 @@
attribute value will also be used by the default and WebDAV Servlets.
(remm)
</update>
+ <fix>
+ <bug>69285</bug>: Optimise the creation of the parameter map for
+ included requests. Based on sample code and test cases provided by John
+ Engebretson. (markt)
+ </fix>
<fix>
<bug>69527</bug>: Avoid rare cases where a cached resource could be set
with 0 content length, or could be evicted immediately. (remm)
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org
