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 ac4ea104d7 More query escaping refactoring
ac4ea104d7 is described below
commit ac4ea104d731117ead13a6b44041d037900b0ded
Author: Mark Thomas <[email protected]>
AuthorDate: Wed Apr 8 13:28:16 2026 +0100
More query escaping refactoring
---
.../catalina/valves/AbstractAccessLogValve.java | 29 ++++++++++------------
.../catalina/valves/ExtendedAccessLogValve.java | 2 +-
2 files changed, 14 insertions(+), 17 deletions(-)
diff --git a/java/org/apache/catalina/valves/AbstractAccessLogValve.java
b/java/org/apache/catalina/valves/AbstractAccessLogValve.java
index af523d4f26..3136e5c2af 100644
--- a/java/org/apache/catalina/valves/AbstractAccessLogValve.java
+++ b/java/org/apache/catalina/valves/AbstractAccessLogValve.java
@@ -1375,14 +1375,7 @@ public abstract class AbstractAccessLogValve extends
ValveBase implements Access
buf.append(request.getMethod());
buf.append(' ');
escapeAndAppend(request.getRequestURI(), buf);
- String query = request.getQueryString();
- if (query != null) {
- buf.append('?');
- // Don't want to write "-" if the query string is empty
- if (!query.isEmpty()) {
- escapeAndAppend(query, buf);
- }
- }
+ appendQueryString(request.getQueryString(), buf, false);
buf.append(' ');
buf.append(request.getProtocol());
}
@@ -1694,15 +1687,19 @@ public abstract class AbstractAccessLogValve extends
ValveBase implements Access
if (request != null) {
query = request.getQueryString();
}
- if (query != null) {
- buf.append('?');
- // Don't want to write "-" if the query string is empty
- if (!query.isEmpty()) {
- escapeAndAppend(query, buf);
- }
- } else {
- buf.append('-');
+ appendQueryString(query, buf, true);
+ }
+ }
+
+ protected static void appendQueryString(String query, CharArrayWriter buf,
boolean writeDashOnNull) {
+ if (query != null) {
+ buf.append('?');
+ // Don't want to write "-" if the query string is empty
+ if (!query.isEmpty()) {
+ escapeAndAppend(query, buf);
}
+ } else if (writeDashOnNull) {
+ buf.append('-');
}
}
diff --git a/java/org/apache/catalina/valves/ExtendedAccessLogValve.java
b/java/org/apache/catalina/valves/ExtendedAccessLogValve.java
index 2be0851bb6..c9cfc9f714 100644
--- a/java/org/apache/catalina/valves/ExtendedAccessLogValve.java
+++ b/java/org/apache/catalina/valves/ExtendedAccessLogValve.java
@@ -615,7 +615,7 @@ public class ExtendedAccessLogValve extends AccessLogValve {
buf.append('?');
// Don't want to write "-" if the query
string is present but empty
if (!query.isEmpty()) {
- buf.append(request.getQueryString());
+ buf.append(query);
}
}
} else {
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
