You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@tomcat.apache.org by ma...@apache.org on 2017/04/24 12:52:26 UTC

svn commit: r1792468 - /tomcat/trunk/test/org/apache/catalina/core/TestAsyncContextImpl.java

Author: markt
Date: Mon Apr 24 12:52:26 2017
New Revision: 1792468

URL: http://svn.apache.org/viewvc?rev=1792468&view=rev
Log:
Fix test broken by update to encoding.
Generally, it is far too difficult to re-encode characters that were encoded in the original URKL when they didn't need to be.

Modified:
    tomcat/trunk/test/org/apache/catalina/core/TestAsyncContextImpl.java

Modified: tomcat/trunk/test/org/apache/catalina/core/TestAsyncContextImpl.java
URL: http://svn.apache.org/viewvc/tomcat/trunk/test/org/apache/catalina/core/TestAsyncContextImpl.java?rev=1792468&r1=1792467&r2=1792468&view=diff
==============================================================================
--- tomcat/trunk/test/org/apache/catalina/core/TestAsyncContextImpl.java (original)
+++ tomcat/trunk/test/org/apache/catalina/core/TestAsyncContextImpl.java Mon Apr 24 12:52:26 2017
@@ -2226,7 +2226,8 @@ public class TestAsyncContextImpl extend
     // https://bz.apache.org/bugzilla/show_bug.cgi?id=57559
     @Test
     public void testAsyncRequestURI_24() throws Exception {
-        doTestAsyncRequestURI("/foo/%24/bar");
+        // '$' is permitted in a path
+        doTestAsyncRequestURI("/foo/$/bar");
     }
 
 



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