You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@tomcat.apache.org by re...@apache.org on 2020/05/13 13:48:29 UTC

[tomcat] branch master updated: Revert

This is an automated email from the ASF dual-hosted git repository.

remm pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/tomcat.git


The following commit(s) were added to refs/heads/master by this push:
     new 43b10e3  Revert
43b10e3 is described below

commit 43b10e37e6eda920e764cc94cd0cd079e614f2ab
Author: remm <re...@apache.org>
AuthorDate: Wed May 13 15:48:12 2020 +0200

    Revert
---
 java/org/apache/catalina/connector/CoyoteAdapter.java | 8 +++-----
 1 file changed, 3 insertions(+), 5 deletions(-)

diff --git a/java/org/apache/catalina/connector/CoyoteAdapter.java b/java/org/apache/catalina/connector/CoyoteAdapter.java
index 98a33e8..ab113a7 100644
--- a/java/org/apache/catalina/connector/CoyoteAdapter.java
+++ b/java/org/apache/catalina/connector/CoyoteAdapter.java
@@ -72,8 +72,6 @@ public class CoyoteAdapter implements Adapter {
             System.getProperty("java.vm.vendor") + "/" +
             System.getProperty("java.runtime.version") + ")";
 
-    private static final String INVALID_URI = "Invalid URI ";
-
     private static final EnumSet<SessionTrackingMode> SSL_ONLY =
         EnumSet.of(SessionTrackingMode.SSL);
 
@@ -612,7 +610,7 @@ public class CoyoteAdapter implements Adapter {
                 connector.getService().getContainer().logAccess(request, response, 0, true);
                 return false;
             } else {
-                response.sendError(400, INVALID_URI);
+                response.sendError(400, "Invalid URI");
             }
         }
 
@@ -630,7 +628,7 @@ public class CoyoteAdapter implements Adapter {
             try {
                 req.getURLDecoder().convert(decodedURI.getByteChunk(), connector.getEncodedSolidusHandlingInternal());
             } catch (IOException ioe) {
-                response.sendError(400, INVALID_URI + ioe.getMessage());
+                response.sendError(400, "Invalid URI: " + ioe.getMessage());
             }
             // Normalization
             if (normalize(req.decodedURI(), connector.getAllowBackslash())) {
@@ -640,7 +638,7 @@ public class CoyoteAdapter implements Adapter {
                 // Therefore it is not necessary to check that the URI remains
                 // normalized after character decoding
             } else {
-                response.sendError(400, INVALID_URI);
+                response.sendError(400, "Invalid URI");
             }
         } else {
             /* The URI is chars or String, and has been sent using an in-memory


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@tomcat.apache.org
For additional commands, e-mail: dev-help@tomcat.apache.org