This is an automated email from the ASF dual-hosted git repository. remm pushed a commit to branch 8.5.x in repository https://gitbox.apache.org/repos/asf/tomcat.git
commit f39353504f19d957d4fbb352f95bdd1150f29159 Author: remm <r...@apache.org> AuthorDate: Thu Feb 9 16:10:09 2023 +0100 Java 8 compat --- .../catalina/valves/ProxyErrorReportValve.java | 44 +++++++++++++--------- 1 file changed, 26 insertions(+), 18 deletions(-) diff --git a/java/org/apache/catalina/valves/ProxyErrorReportValve.java b/java/org/apache/catalina/valves/ProxyErrorReportValve.java index 0132c83a49..6541f89b63 100644 --- a/java/org/apache/catalina/valves/ProxyErrorReportValve.java +++ b/java/org/apache/catalina/valves/ProxyErrorReportValve.java @@ -19,12 +19,12 @@ package org.apache.catalina.valves; import java.io.IOException; import java.io.InputStream; import java.io.OutputStream; +import java.io.UnsupportedEncodingException; import java.net.HttpURLConnection; import java.net.URI; import java.net.URISyntaxException; import java.net.URL; import java.net.URLEncoder; -import java.nio.charset.StandardCharsets; import java.util.MissingResourceException; import java.util.ResourceBundle; import java.util.concurrent.atomic.AtomicBoolean; @@ -153,10 +153,14 @@ public class ProxyErrorReportValve extends ErrorReportValve { } else { stringBuilder.append("?"); } - stringBuilder.append("requestUri="); - stringBuilder.append(URLEncoder.encode(request.getDecodedRequestURI(), request.getConnector().getURICharset())); - stringBuilder.append("&statusCode="); - stringBuilder.append(URLEncoder.encode(String.valueOf(statusCode), StandardCharsets.UTF_8)); + try { + stringBuilder.append("requestUri="); + stringBuilder.append(URLEncoder.encode(request.getDecodedRequestURI(), request.getConnector().getURIEncoding())); + stringBuilder.append("&statusCode="); + stringBuilder.append(URLEncoder.encode(String.valueOf(statusCode), "UTF-8")); + } catch (UnsupportedEncodingException e) { + // Ignore + } String reason = null; String description = null; @@ -172,19 +176,23 @@ public class ProxyErrorReportValve extends ErrorReportValve { reason = smClient.getString("errorReportValve.unknownReason"); description = smClient.getString("errorReportValve.noDescription"); } - stringBuilder.append("&statusDescription="); - stringBuilder.append(URLEncoder.encode(description, StandardCharsets.UTF_8)); - stringBuilder.append("&statusReason="); - stringBuilder.append(URLEncoder.encode(reason, StandardCharsets.UTF_8)); - - String message = response.getMessage(); - if (message != null) { - stringBuilder.append("&message="); - stringBuilder.append(URLEncoder.encode(message, StandardCharsets.UTF_8)); - } - if (throwable != null) { - stringBuilder.append("&throwable="); - stringBuilder.append(URLEncoder.encode(throwable.toString(), StandardCharsets.UTF_8)); + try { + stringBuilder.append("&statusDescription="); + stringBuilder.append(URLEncoder.encode(description, "UTF-8")); + stringBuilder.append("&statusReason="); + stringBuilder.append(URLEncoder.encode(reason, "UTF-8")); + + String message = response.getMessage(); + if (message != null) { + stringBuilder.append("&message="); + stringBuilder.append(URLEncoder.encode(message, "UTF-8")); + } + if (throwable != null) { + stringBuilder.append("&throwable="); + stringBuilder.append(URLEncoder.encode(throwable.toString(), "UTF-8")); + } + } catch (UnsupportedEncodingException e) { + // Ignore } urlString = stringBuilder.toString(); --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org For additional commands, e-mail: dev-h...@tomcat.apache.org