Author: markt
Date: Tue Aug 30 16:41:12 2016
New Revision: 1758423
URL: http://svn.apache.org/viewvc?rev=1758423&view=rev
Log: (empty)
Modified:
tomcat/trunk/java/org/apache/catalina/Context.java
tomcat/trunk/java/org/apache/catalina/core/ApplicationFilterRegistration.java
tomcat/trunk/java/org/apache/catalina/core/ApplicationServletRegistration.java
tomcat/trunk/java/org/apache/catalina/core/StandardContext.java
tomcat/trunk/java/org/apache/catalina/startup/ContextConfig.java
tomcat/trunk/java/org/apache/catalina/startup/FailedContext.java
tomcat/trunk/java/org/apache/catalina/startup/Tomcat.java
tomcat/trunk/java/org/apache/tomcat/util/descriptor/web/FilterMap.java
tomcat/trunk/java/org/apache/tomcat/util/descriptor/web/JspPropertyGroup.java
tomcat/trunk/java/org/apache/tomcat/util/descriptor/web/SecurityCollection.java
tomcat/trunk/java/org/apache/tomcat/util/descriptor/web/SecurityConstraint.java
tomcat/trunk/java/org/apache/tomcat/util/descriptor/web/WebXml.java
tomcat/trunk/test/javax/servlet/http/TestHttpServlet.java
tomcat/trunk/test/org/apache/catalina/authenticator/TestDigestAuthenticator.java
tomcat/trunk/test/org/apache/catalina/authenticator/TestFormAuthenticator.java
tomcat/trunk/test/org/apache/catalina/authenticator/TestNonLoginAndBasicAuthenticator.java
tomcat/trunk/test/org/apache/catalina/authenticator/TestSSOnonLoginAndBasicAuthenticator.java
tomcat/trunk/test/org/apache/catalina/authenticator/TestSSOnonLoginAndDigestAuthenticator.java
tomcat/trunk/test/org/apache/catalina/connector/TestConnector.java
tomcat/trunk/test/org/apache/catalina/connector/TestCoyoteAdapter.java
tomcat/trunk/test/org/apache/catalina/connector/TestCoyoteOutputStream.java
tomcat/trunk/test/org/apache/catalina/connector/TestInputBuffer.java
tomcat/trunk/test/org/apache/catalina/connector/TestKeepAliveCount.java
tomcat/trunk/test/org/apache/catalina/connector/TestMaxConnections.java
tomcat/trunk/test/org/apache/catalina/connector/TestOutputBuffer.java
tomcat/trunk/test/org/apache/catalina/connector/TestRequest.java
tomcat/trunk/test/org/apache/catalina/connector/TestResponse.java
tomcat/trunk/test/org/apache/catalina/connector/TestSendFile.java
tomcat/trunk/test/org/apache/catalina/core/TestApplicationContext.java
tomcat/trunk/test/org/apache/catalina/core/TestApplicationContextGetRequestDispatcher.java
tomcat/trunk/test/org/apache/catalina/core/TestApplicationFilterConfig.java
tomcat/trunk/test/org/apache/catalina/core/TestApplicationHttpRequest.java
tomcat/trunk/test/org/apache/catalina/core/TestApplicationMapping.java
tomcat/trunk/test/org/apache/catalina/core/TestAsyncContextImpl.java
tomcat/trunk/test/org/apache/catalina/core/TestStandardContext.java
tomcat/trunk/test/org/apache/catalina/core/TestStandardContextAliases.java
tomcat/trunk/test/org/apache/catalina/core/TestStandardContextResources.java
tomcat/trunk/test/org/apache/catalina/core/TestStandardContextValve.java
tomcat/trunk/test/org/apache/catalina/core/TestStandardHostValve.java
tomcat/trunk/test/org/apache/catalina/core/TestStandardWrapper.java
tomcat/trunk/test/org/apache/catalina/core/TestSwallowAbortedUploads.java
tomcat/trunk/test/org/apache/catalina/filters/TestAddCharSetFilter.java
tomcat/trunk/test/org/apache/catalina/filters/TestExpiresFilter.java
tomcat/trunk/test/org/apache/catalina/filters/TestRemoteIpFilter.java
tomcat/trunk/test/org/apache/catalina/filters/TestRestCsrfPreventionFilter2.java
tomcat/trunk/test/org/apache/catalina/ha/context/TestReplicatedContext.java
tomcat/trunk/test/org/apache/catalina/loader/TestWebappClassLoaderExecutorMemoryLeak.java
tomcat/trunk/test/org/apache/catalina/loader/TestWebappClassLoaderMemoryLeak.java
tomcat/trunk/test/org/apache/catalina/loader/TestWebappClassLoaderThreadLocalMemoryLeak.java
tomcat/trunk/test/org/apache/catalina/mapper/TestMapperWebapps.java
tomcat/trunk/test/org/apache/catalina/nonblocking/TestNonBlockingAPI.java
tomcat/trunk/test/org/apache/catalina/realm/TestRealmBase.java
tomcat/trunk/test/org/apache/catalina/servlets/TestDefaultServlet.java
tomcat/trunk/test/org/apache/catalina/servlets/TestWebdavServlet.java
tomcat/trunk/test/org/apache/catalina/session/TestPersistentManagerIntegration.java
tomcat/trunk/test/org/apache/catalina/session/TestStandardSessionIntegration.java
tomcat/trunk/test/org/apache/catalina/startup/TestContextConfig.java
tomcat/trunk/test/org/apache/catalina/startup/TestContextConfigAnnotation.java
tomcat/trunk/test/org/apache/catalina/startup/TestTomcat.java
tomcat/trunk/test/org/apache/catalina/startup/TestTomcatClassLoader.java
tomcat/trunk/test/org/apache/catalina/valves/TestErrorReportValve.java
tomcat/trunk/test/org/apache/coyote/TestIoTimeouts.java
tomcat/trunk/test/org/apache/coyote/ajp/TestAbstractAjpProcessor.java
tomcat/trunk/test/org/apache/coyote/http11/TestHttp11InputBuffer.java
tomcat/trunk/test/org/apache/coyote/http11/TestHttp11OutputBuffer.java
tomcat/trunk/test/org/apache/coyote/http11/TestHttp11Processor.java
tomcat/trunk/test/org/apache/coyote/http11/filters/TestChunkedInputFilter.java
tomcat/trunk/test/org/apache/coyote/http11/upgrade/TestUpgrade.java
tomcat/trunk/test/org/apache/coyote/http11/upgrade/TestUpgradeInternalHandler.java
tomcat/trunk/test/org/apache/coyote/http2/Http2TestBase.java
tomcat/trunk/test/org/apache/jasper/runtime/TestPageContextImpl.java
tomcat/trunk/test/org/apache/jasper/servlet/TestJspServlet.java
tomcat/trunk/test/org/apache/naming/resources/TestNamingContext.java
tomcat/trunk/test/org/apache/tomcat/unittest/TesterContext.java
tomcat/trunk/test/org/apache/tomcat/util/descriptor/web/TestJspPropertyGroupDescriptorImpl.java
tomcat/trunk/test/org/apache/tomcat/util/descriptor/web/TestSecurityConstraint.java
tomcat/trunk/test/org/apache/tomcat/util/descriptor/web/TestWebXml.java
tomcat/trunk/test/org/apache/tomcat/util/http/CookiesBaseTest.java
tomcat/trunk/test/org/apache/tomcat/util/http/TestBug49158.java
tomcat/trunk/test/org/apache/tomcat/util/http/TestCookieParsing.java
tomcat/trunk/test/org/apache/tomcat/util/http/TestCookieProcessorGenerationHttp.java
tomcat/trunk/test/org/apache/tomcat/util/http/TestMimeHeadersIntegration.java
tomcat/trunk/test/org/apache/tomcat/util/net/TestSsl.java
tomcat/trunk/test/org/apache/tomcat/util/net/TesterSupport.java
tomcat/trunk/test/org/apache/tomcat/websocket/TestConnectionLimit.java
tomcat/trunk/test/org/apache/tomcat/websocket/TestWebSocketFrameClient.java
tomcat/trunk/test/org/apache/tomcat/websocket/TestWebSocketFrameClientSSL.java
tomcat/trunk/test/org/apache/tomcat/websocket/TestWsPingPongMessages.java
tomcat/trunk/test/org/apache/tomcat/websocket/TestWsRemoteEndpoint.java
tomcat/trunk/test/org/apache/tomcat/websocket/TestWsSubprotocols.java
tomcat/trunk/test/org/apache/tomcat/websocket/TestWsWebSocketContainer.java
tomcat/trunk/test/org/apache/tomcat/websocket/pojo/TestEncodingDecoding.java
tomcat/trunk/test/org/apache/tomcat/websocket/pojo/TestPojoEndpointBase.java
tomcat/trunk/test/org/apache/tomcat/websocket/pojo/TestPojoMethodMapping.java
tomcat/trunk/test/org/apache/tomcat/websocket/server/TestClassLoader.java
tomcat/trunk/test/org/apache/tomcat/websocket/server/TestClose.java
tomcat/trunk/test/org/apache/tomcat/websocket/server/TestCloseBug58624.java
tomcat/trunk/test/org/apache/tomcat/websocket/server/TestShutdown.java
tomcat/trunk/test/org/apache/tomcat/websocket/server/TestWsRemoteEndpointImplServer.java
tomcat/trunk/test/org/apache/tomcat/websocket/server/TestWsServerContainer.java
tomcat/trunk/webapps/docs/changelog.xml
Modified: tomcat/trunk/java/org/apache/catalina/Context.java
URL:
http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/catalina/Context.java?rev=1758423&r1=1758422&r2=1758423&view=diff
==============================================================================
--- tomcat/trunk/java/org/apache/catalina/Context.java (original)
+++ tomcat/trunk/java/org/apache/catalina/Context.java Tue Aug 30 16:41:12 2016
@@ -32,6 +32,7 @@ import org.apache.catalina.deploy.Naming
import org.apache.tomcat.ContextBind;
import org.apache.tomcat.InstanceManager;
import org.apache.tomcat.JarScanner;
+import org.apache.tomcat.util.buf.UDecoder;
import org.apache.tomcat.util.descriptor.web.ApplicationParameter;
import org.apache.tomcat.util.descriptor.web.ErrorPage;
import org.apache.tomcat.util.descriptor.web.FilterDef;
@@ -872,9 +873,14 @@ public interface Context extends Contain
* Add a new servlet mapping, replacing any existing mapping for
* the specified pattern.
*
- * @param pattern URL pattern to be mapped
- * @param name Name of the corresponding servlet to execute
+ * @param pattern URL pattern to be mapped. The pattern will be % decoded
+ * using UTF-8
+ * @param name Name of the corresponding servlet to execute
+ *
+ * @deprecated Will be removed in Tomcat 9. Use
+ * {@link #addServletMappingDecoded(String, String)}
*/
+ @Deprecated
public default void addServletMapping(String pattern, String name) {
addServletMapping(pattern, name, false);
}
@@ -884,12 +890,45 @@ public interface Context extends Contain
* Add a new servlet mapping, replacing any existing mapping for
* the specified pattern.
*
+ * @param pattern URL pattern to be mapped. The pattern will be %
+ * decoded using UTF-8
+ * @param name Name of the corresponding servlet to execute
+ * @param jspWildcard true if name identifies the JspServlet and pattern
+ * contains a wildcard; false otherwise
+ *
+ * @deprecated Will be removed in Tomcat 9. Use
+ * {@link #addServletMappingDecoded(String, String, boolean)}
+ */
+ @Deprecated
+ public default void addServletMapping(String pattern, String name,
+ boolean jspWildcard) {
+ addServletMappingDecoded(
+ UDecoder.URLDecode(pattern, "UTF-8"), name, jspWildcard);
+ }
+
+
+ /**
+ * Add a new servlet mapping, replacing any existing mapping for
+ * the specified pattern.
+ *
+ * @param pattern URL pattern to be mapped
+ * @param name Name of the corresponding servlet to execute
+ */
+ public default void addServletMappingDecoded(String pattern, String name) {
+ addServletMappingDecoded(pattern, name, false);
+ }
+
+
+ /**
+ * Add a new servlet mapping, replacing any existing mapping for
+ * the specified pattern.
+ *
* @param pattern URL pattern to be mapped
* @param name Name of the corresponding servlet to execute
* @param jspWildcard true if name identifies the JspServlet
* and pattern contains a wildcard; false otherwise
*/
- public void addServletMapping(String pattern, String name,
+ public void addServletMappingDecoded(String pattern, String name,
boolean jspWildcard);
Modified:
tomcat/trunk/java/org/apache/catalina/core/ApplicationFilterRegistration.java
URL:
http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/catalina/core/ApplicationFilterRegistration.java?rev=1758423&r1=1758422&r2=1758423&view=diff
==============================================================================
---
tomcat/trunk/java/org/apache/catalina/core/ApplicationFilterRegistration.java
(original)
+++
tomcat/trunk/java/org/apache/catalina/core/ApplicationFilterRegistration.java
Tue Aug 30 16:41:12 2016
@@ -96,6 +96,7 @@ public class ApplicationFilterRegistrati
}
if (urlPatterns != null) {
+ // % decoded (if necessary) using UTF-8
for (String urlPattern : urlPatterns) {
filterMap.addURLPattern(urlPattern);
}
Modified:
tomcat/trunk/java/org/apache/catalina/core/ApplicationServletRegistration.java
URL:
http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/catalina/core/ApplicationServletRegistration.java?rev=1758423&r1=1758422&r2=1758423&view=diff
==============================================================================
---
tomcat/trunk/java/org/apache/catalina/core/ApplicationServletRegistration.java
(original)
+++
tomcat/trunk/java/org/apache/catalina/core/ApplicationServletRegistration.java
Tue Aug 30 16:41:12 2016
@@ -31,6 +31,7 @@ import org.apache.catalina.Context;
import org.apache.catalina.LifecycleState;
import org.apache.catalina.Wrapper;
import org.apache.catalina.util.ParameterMap;
+import org.apache.tomcat.util.buf.UDecoder;
import org.apache.tomcat.util.res.StringManager;
public class ApplicationServletRegistration
@@ -189,7 +190,8 @@ public class ApplicationServletRegistrat
}
for (String urlPattern : urlPatterns) {
- context.addServletMapping(urlPattern, wrapper.getName());
+ context.addServletMappingDecoded(
+ UDecoder.URLDecode(urlPattern, "UTF-8"),
wrapper.getName());
}
return Collections.emptySet();
}
Modified: tomcat/trunk/java/org/apache/catalina/core/StandardContext.java
URL:
http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/catalina/core/StandardContext.java?rev=1758423&r1=1758422&r2=1758423&view=diff
==============================================================================
--- tomcat/trunk/java/org/apache/catalina/core/StandardContext.java (original)
+++ tomcat/trunk/java/org/apache/catalina/core/StandardContext.java Tue Aug 30
16:41:12 2016
@@ -114,7 +114,6 @@ import org.apache.tomcat.InstanceManager
import org.apache.tomcat.JarScanner;
import org.apache.tomcat.util.ExceptionUtils;
import org.apache.tomcat.util.IntrospectionUtils;
-import org.apache.tomcat.util.buf.UDecoder;
import org.apache.tomcat.util.descriptor.XmlIdentifiers;
import org.apache.tomcat.util.descriptor.web.ApplicationParameter;
import org.apache.tomcat.util.descriptor.web.ErrorPage;
@@ -2802,7 +2801,7 @@ public class StandardContext extends Con
*/
String[] jspMappings = oldJspServlet.findMappings();
for (int i=0; jspMappings!=null && i<jspMappings.length; i++) {
- addServletMapping(jspMappings[i], child.getName());
+ addServletMappingDecoded(jspMappings[i], child.getName());
}
}
}
@@ -3119,31 +3118,31 @@ public class StandardContext extends Con
* is not known to this Context
*/
@Override
- public void addServletMapping(String pattern, String name,
+ public void addServletMappingDecoded(String pattern, String name,
boolean jspWildCard) {
// Validate the proposed mapping
if (findChild(name) == null)
throw new IllegalArgumentException
(sm.getString("standardContext.servletMap.name", name));
- String decodedPattern = adjustURLPattern(UDecoder.URLDecode(pattern));
- if (!validateURLPattern(decodedPattern))
+ String adjustedPattern = adjustURLPattern(pattern);
+ if (!validateURLPattern(adjustedPattern))
throw new IllegalArgumentException
- (sm.getString("standardContext.servletMap.pattern",
decodedPattern));
+ (sm.getString("standardContext.servletMap.pattern",
adjustedPattern));
// Add this mapping to our registered set
synchronized (servletMappingsLock) {
- String name2 = servletMappings.get(decodedPattern);
+ String name2 = servletMappings.get(adjustedPattern);
if (name2 != null) {
// Don't allow more than one servlet on the same pattern
Wrapper wrapper = (Wrapper) findChild(name2);
- wrapper.removeMapping(decodedPattern);
+ wrapper.removeMapping(adjustedPattern);
}
- servletMappings.put(decodedPattern, name);
+ servletMappings.put(adjustedPattern, name);
}
Wrapper wrapper = (Wrapper) findChild(name);
- wrapper.addMapping(decodedPattern);
+ wrapper.addMapping(adjustedPattern);
- fireContainerEvent("addServletMapping", decodedPattern);
+ fireContainerEvent("addServletMapping", adjustedPattern);
}
Modified: tomcat/trunk/java/org/apache/catalina/startup/ContextConfig.java
URL:
http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/catalina/startup/ContextConfig.java?rev=1758423&r1=1758422&r2=1758423&view=diff
==============================================================================
--- tomcat/trunk/java/org/apache/catalina/startup/ContextConfig.java (original)
+++ tomcat/trunk/java/org/apache/catalina/startup/ContextConfig.java Tue Aug 30
16:41:12 2016
@@ -1373,7 +1373,7 @@ public class ContextConfig implements Li
}
for (Entry<String, String> entry :
webxml.getServletMappings().entrySet()) {
- context.addServletMapping(entry.getKey(), entry.getValue());
+ context.addServletMappingDecoded(entry.getKey(), entry.getValue());
}
SessionConfig sessionConfig = webxml.getSessionConfig();
if (sessionConfig != null) {
@@ -1426,7 +1426,7 @@ public class ContextConfig implements Li
}
if (context.findChild(jspServletName) != null) {
for (String urlPattern : jspPropertyGroup.getUrlPatterns()) {
- context.addServletMapping(urlPattern, jspServletName,
true);
+ context.addServletMappingDecoded(urlPattern,
jspServletName, true);
}
} else {
if(log.isDebugEnabled()) {
@@ -2445,6 +2445,7 @@ public class ContextConfig implements Li
urlPatterns = processAnnotationsStringArray(evp.getValue());
urlPatternsSet = urlPatterns.length > 0;
for (String urlPattern : urlPatterns) {
+ // % decoded (if required) using UTF-8
filterMap.addURLPattern(urlPattern);
}
} else if ("servletNames".equals(name)) {
@@ -2526,6 +2527,7 @@ public class ContextConfig implements Li
if (urlPatternsSet
&& (urlsPatterns == null || urlsPatterns.length == 0))
{
for (String urlPattern : filterMap.getURLPatterns()) {
+ // % decoded (if required) using UTF-8
descMap.addURLPattern(urlPattern);
}
}
Modified: tomcat/trunk/java/org/apache/catalina/startup/FailedContext.java
URL:
http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/catalina/startup/FailedContext.java?rev=1758423&r1=1758422&r2=1758423&view=diff
==============================================================================
--- tomcat/trunk/java/org/apache/catalina/startup/FailedContext.java (original)
+++ tomcat/trunk/java/org/apache/catalina/startup/FailedContext.java Tue Aug 30
16:41:12 2016
@@ -567,7 +567,7 @@ public class FailedContext extends Lifec
public void removeSecurityRole(String role) { /* NO-OP */ }
@Override
- public void addServletMapping(String pattern, String name,
+ public void addServletMappingDecoded(String pattern, String name,
boolean jspWildcard) { /* NO-OP */ }
@Override
public String findServletMapping(String pattern) { return null; }
Modified: tomcat/trunk/java/org/apache/catalina/startup/Tomcat.java
URL:
http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/catalina/startup/Tomcat.java?rev=1758423&r1=1758422&r2=1758423&view=diff
==============================================================================
--- tomcat/trunk/java/org/apache/catalina/startup/Tomcat.java (original)
+++ tomcat/trunk/java/org/apache/catalina/startup/Tomcat.java Tue Aug 30
16:41:12 2016
@@ -865,9 +865,9 @@ public class Tomcat {
servlet.setOverridable(true);
// Servlet mappings
- ctx.addServletMapping("/", "default");
- ctx.addServletMapping("*.jsp", "jsp");
- ctx.addServletMapping("*.jspx", "jsp");
+ ctx.addServletMappingDecoded("/", "default");
+ ctx.addServletMappingDecoded("*.jsp", "jsp");
+ ctx.addServletMappingDecoded("*.jspx", "jsp");
// Sessions
ctx.setSessionTimeout(30);
Modified: tomcat/trunk/java/org/apache/tomcat/util/descriptor/web/FilterMap.java
URL:
http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/tomcat/util/descriptor/web/FilterMap.java?rev=1758423&r1=1758422&r2=1758423&view=diff
==============================================================================
--- tomcat/trunk/java/org/apache/tomcat/util/descriptor/web/FilterMap.java
(original)
+++ tomcat/trunk/java/org/apache/tomcat/util/descriptor/web/FilterMap.java Tue
Aug 30 16:41:12 2016
@@ -57,6 +57,24 @@ public class FilterMap implements Serial
private int dispatcherMapping = NOT_SET;
+ private String encoding = null;
+ public void setEncoding(String encoding) {
+ this.encoding = encoding;
+ }
+ /**
+ * Obtain the encoding of the XML source that was used to populated this
+ * object.
+ *
+ * @return The encoding of the associated XML source or <code>UTF-8</code>
+ * if the encoding could not be determined
+ */
+ public String getEncoding() {
+ if (encoding == null || encoding.length() == 0) {
+ return "UTF-8";
+ }
+ return encoding;
+ }
+
private String filterName = null;
public String getFilterName() {
@@ -127,6 +145,9 @@ public class FilterMap implements Serial
}
public void addURLPattern(String urlPattern) {
+ addURLPatternDecoded(UDecoder.URLDecode(urlPattern, getEncoding()));
+ }
+ public void addURLPatternDecoded(String urlPattern) {
if ("*".equals(urlPattern)) {
this.matchAllUrlPatterns = true;
} else {
Modified:
tomcat/trunk/java/org/apache/tomcat/util/descriptor/web/JspPropertyGroup.java
URL:
http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/tomcat/util/descriptor/web/JspPropertyGroup.java?rev=1758423&r1=1758422&r2=1758423&view=diff
==============================================================================
---
tomcat/trunk/java/org/apache/tomcat/util/descriptor/web/JspPropertyGroup.java
(original)
+++
tomcat/trunk/java/org/apache/tomcat/util/descriptor/web/JspPropertyGroup.java
Tue Aug 30 16:41:12 2016
@@ -21,10 +21,31 @@ import java.util.Collection;
import java.util.LinkedHashSet;
import java.util.Set;
+import org.apache.tomcat.util.buf.UDecoder;
+
/**
* Representation of a jsp-property-group element in web.xml.
*/
public class JspPropertyGroup {
+
+ private String encoding = null;
+ public void setEncoding(String encoding) {
+ this.encoding = encoding;
+ }
+ /**
+ * Obtain the encoding of the XML source that was used to populated this
+ * object.
+ *
+ * @return The encoding of the associated XML source or <code>UTF-8</code>
+ * if the encoding could not be determined
+ */
+ public String getEncoding() {
+ if (encoding == null || encoding.length() == 0) {
+ return "UTF-8";
+ }
+ return encoding;
+ }
+
private Boolean deferredSyntax = null;
public void setDeferredSyntax(String deferredSyntax) {
this.deferredSyntax = Boolean.valueOf(deferredSyntax);
@@ -75,6 +96,9 @@ public class JspPropertyGroup {
private LinkedHashSet<String> urlPattern = new LinkedHashSet<>();
public void addUrlPattern(String urlPattern) {
+ addUrlPatternDecoded(UDecoder.URLDecode(urlPattern, getEncoding()));
+ }
+ public void addUrlPatternDecoded(String urlPattern) {
this.urlPattern.add(urlPattern);
}
public Set<String> getUrlPatterns() { return this.urlPattern; }
Modified:
tomcat/trunk/java/org/apache/tomcat/util/descriptor/web/SecurityCollection.java
URL:
http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/tomcat/util/descriptor/web/SecurityCollection.java?rev=1758423&r1=1758422&r2=1758423&view=diff
==============================================================================
---
tomcat/trunk/java/org/apache/tomcat/util/descriptor/web/SecurityCollection.java
(original)
+++
tomcat/trunk/java/org/apache/tomcat/util/descriptor/web/SecurityCollection.java
Tue Aug 30 16:41:12 2016
@@ -38,6 +38,25 @@ public class SecurityCollection implemen
private static final long serialVersionUID = 1L;
+ private String encoding = null;
+ public void setEncoding(String encoding) {
+ this.encoding = encoding;
+ }
+ /**
+ * Obtain the encoding of the XML source that was used to populated this
+ * object.
+ *
+ * @return The encoding of the associated XML source or <code>UTF-8</code>
+ * if the encoding could not be determined
+ */
+ public String getEncoding() {
+ if (encoding == null || encoding.length() == 0) {
+ return "UTF-8";
+ }
+ return encoding;
+ }
+
+
// ----------------------------------------------------------- Constructors
@@ -209,6 +228,9 @@ public class SecurityCollection implemen
* @param pattern The pattern
*/
public void addPattern(String pattern) {
+ addPatternDecoded(UDecoder.URLDecode(pattern, "UTF-8"));
+ }
+ public void addPatternDecoded(String pattern) {
if (pattern == null)
return;
@@ -220,7 +242,6 @@ public class SecurityCollection implemen
}
results[patterns.length] = decodedPattern;
patterns = results;
-
}
Modified:
tomcat/trunk/java/org/apache/tomcat/util/descriptor/web/SecurityConstraint.java
URL:
http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/tomcat/util/descriptor/web/SecurityConstraint.java?rev=1758423&r1=1758422&r2=1758423&view=diff
==============================================================================
---
tomcat/trunk/java/org/apache/tomcat/util/descriptor/web/SecurityConstraint.java
(original)
+++
tomcat/trunk/java/org/apache/tomcat/util/descriptor/web/SecurityConstraint.java
Tue Aug 30 16:41:12 2016
@@ -61,6 +61,25 @@ public class SecurityConstraint implemen
StringManager.getManager(Constants.PACKAGE_NAME);
+ private String encoding = null;
+ public void setEncoding(String encoding) {
+ this.encoding = encoding;
+ }
+ /**
+ * Obtain the encoding of the XML source that was used to populated this
+ * object.
+ *
+ * @return The encoding of the associated XML source or <code>UTF-8</code>
+ * if the encoding could not be determined
+ */
+ public String getEncoding() {
+ if (encoding == null || encoding.length() == 0) {
+ return "UTF-8";
+ }
+ return encoding;
+ }
+
+
// ----------------------------------------------------------- Constructors
/**
@@ -280,6 +299,9 @@ public class SecurityConstraint implemen
if (collection == null)
return;
+
+ collection.setEncoding(getEncoding());
+
SecurityCollection results[] =
new SecurityCollection[collections.length + 1];
for (int i = 0; i < collections.length; i++)
@@ -706,7 +728,7 @@ public class SecurityConstraint implemen
for (String method : methods) {
collection.addOmittedMethod(method);
}
- collection.addPattern(pattern);
+ collection.addPatternDecoded(pattern);
collection.setName("deny-uncovered-http-methods");
SecurityConstraint constraint = new SecurityConstraint();
constraint.setAuthConstraint(true);
@@ -759,7 +781,7 @@ public class SecurityConstraint implemen
for (String method : omittedMethods) {
collection.addMethod(method);
}
- collection.addPattern(pattern);
+ collection.addPatternDecoded(pattern);
collection.setName("deny-uncovered-http-methods");
SecurityConstraint constraint = new SecurityConstraint();
constraint.setAuthConstraint(true);
Modified: tomcat/trunk/java/org/apache/tomcat/util/descriptor/web/WebXml.java
URL:
http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/tomcat/util/descriptor/web/WebXml.java?rev=1758423&r1=1758422&r2=1758423&view=diff
==============================================================================
--- tomcat/trunk/java/org/apache/tomcat/util/descriptor/web/WebXml.java
(original)
+++ tomcat/trunk/java/org/apache/tomcat/util/descriptor/web/WebXml.java Tue Aug
30 16:41:12 2016
@@ -16,7 +16,9 @@
*/
package org.apache.tomcat.util.descriptor.web;
+import java.io.UnsupportedEncodingException;
import java.net.URL;
+import java.net.URLEncoder;
import java.util.ArrayList;
import java.util.Collection;
import java.util.EnumSet;
@@ -39,7 +41,9 @@ import javax.servlet.descriptor.TaglibDe
import org.apache.juli.logging.Log;
import org.apache.juli.logging.LogFactory;
+import org.apache.tomcat.util.buf.UDecoder;
import org.apache.tomcat.util.descriptor.XmlIdentifiers;
+import org.apache.tomcat.util.digester.DocumentProperties;
import org.apache.tomcat.util.res.StringManager;
/**
@@ -50,7 +54,7 @@ import org.apache.tomcat.util.res.String
* This class checks for invalid duplicates (eg filter/servlet names)
* StandardContext will check validity of values (eg URL formats etc)
*/
-public class WebXml {
+public class WebXml implements DocumentProperties.Encoding {
protected static final String ORDER_OTHERS =
"org.apache.catalina.order.others";
@@ -126,6 +130,24 @@ public class WebXml {
public Set<String> getBeforeOrdering() { return before; }
// Common elements and attributes
+ private String encoding = null;
+ @Override
+ public void setEncoding(String encoding) {
+ this.encoding = encoding;
+ }
+ /**
+ * Obtain the encoding of the XML source that was used to populated this
+ * object.
+ *
+ * @return The encoding of the associated XML source or <code>UTF-8</code>
+ * if the encoding could not be determined
+ */
+ public String getEncoding() {
+ if (encoding == null || encoding.length() == 0) {
+ return "UTF-8";
+ }
+ return encoding;
+ }
// Required attribute of web-app element
public String getVersion() {
@@ -309,9 +331,14 @@ public class WebXml {
public Map<String,ServletDef> getServlets() { return servlets; }
// servlet-mapping
+ // Note: URLPatterns from web.xml may be URL encoded
+ // (http://svn.apache.org/r285186)
private final Map<String,String> servletMappings = new HashMap<>();
private final Set<String> servletMappingNames = new HashSet<>();
public void addServletMapping(String urlPattern, String servletName) {
+ addServletMappingDecoded(UDecoder.URLDecode(urlPattern,
getEncoding()), servletName);
+ }
+ public void addServletMappingDecoded(String urlPattern, String
servletName) {
String oldServletName = servletMappings.put(urlPattern, servletName);
if (oldServletName != null) {
// Duplicate mapping. As per clarification from the Servlet EG,
@@ -394,6 +421,7 @@ public class WebXml {
// jsp-config/jsp-property-group
private final Set<JspPropertyGroup> jspPropertyGroups = new
LinkedHashSet<>();
public void addJspPropertyGroup(JspPropertyGroup propertyGroup) {
+ propertyGroup.setEncoding(getEncoding());
jspPropertyGroups.add(propertyGroup);
}
public Set<JspPropertyGroup> getJspPropertyGroups() {
@@ -405,6 +433,7 @@ public class WebXml {
// TODO: Should support multiple description elements with language
private final Set<SecurityConstraint> securityConstraints = new
HashSet<>();
public void addSecurityConstraint(SecurityConstraint securityConstraint) {
+ securityConstraint.setEncoding(getEncoding());
securityConstraints.add(securityConstraint);
}
public Set<SecurityConstraint> getSecurityConstraints() {
@@ -644,7 +673,6 @@ public class WebXml {
*/
public String toXml() {
StringBuilder sb = new StringBuilder(2048);
-
// TODO - Various, icon, description etc elements are skipped - mainly
// because they are ignored when web.xml is parsed - see above
@@ -771,7 +799,7 @@ public class WebXml {
sb.append(" <url-pattern>*</url-pattern>\n");
} else {
for (String urlPattern : filterMap.getURLPatterns()) {
- appendElement(sb, INDENT4, "url-pattern", urlPattern);
+ appendElement(sb, INDENT4, "url-pattern",
encodeUrl(urlPattern));
}
}
// dispatcher was added in Servlet 2.4
@@ -862,7 +890,7 @@ public class WebXml {
for (Map.Entry<String, String> entry : servletMappings.entrySet()) {
sb.append(" <servlet-mapping>\n");
appendElement(sb, INDENT4, "servlet-name", entry.getValue());
- appendElement(sb, INDENT4, "url-pattern", entry.getKey());
+ appendElement(sb, INDENT4, "url-pattern",
encodeUrl(entry.getKey()));
sb.append(" </servlet-mapping>\n");
}
sb.append('\n');
@@ -946,7 +974,7 @@ public class WebXml {
for (JspPropertyGroup jpg : jspPropertyGroups) {
sb.append(" <jsp-property-group>\n");
for (String urlPattern : jpg.getUrlPatterns()) {
- appendElement(sb, INDENT6, "url-pattern", urlPattern);
+ appendElement(sb, INDENT6, "url-pattern",
encodeUrl(urlPattern));
}
appendElement(sb, INDENT6, "el-ignored",
jpg.getElIgnored());
appendElement(sb, INDENT6, "page-encoding",
@@ -1041,7 +1069,7 @@ public class WebXml {
appendElement(sb, INDENT6, "description",
collection.getDescription());
for (String urlPattern : collection.findPatterns()) {
- appendElement(sb, INDENT6, "url-pattern", urlPattern);
+ appendElement(sb, INDENT6, "url-pattern",
encodeUrl(urlPattern));
}
for (String method : collection.findMethods()) {
appendElement(sb, INDENT6, "http-method", method);
@@ -1314,6 +1342,17 @@ public class WebXml {
return sb.toString();
}
+
+ private String encodeUrl(String input) {
+ try {
+ return URLEncoder.encode(input, "UTF-8");
+ } catch (UnsupportedEncodingException e) {
+ // Impossible. UTF-8 is a required character set
+ return null;
+ }
+ }
+
+
private static void appendElement(StringBuilder sb, String indent,
String elementName, String value) {
if (value == null) {
@@ -1628,7 +1667,7 @@ public class WebXml {
// Add fragment mappings
for (Map.Entry<String,String> mapping : servletMappingsToAdd) {
- addServletMapping(mapping.getKey(), mapping.getValue());
+ addServletMappingDecoded(mapping.getKey(), mapping.getValue());
}
for (WebXml fragment : fragments) {
Modified: tomcat/trunk/test/javax/servlet/http/TestHttpServlet.java
URL:
http://svn.apache.org/viewvc/tomcat/trunk/test/javax/servlet/http/TestHttpServlet.java?rev=1758423&r1=1758422&r2=1758423&view=diff
==============================================================================
--- tomcat/trunk/test/javax/servlet/http/TestHttpServlet.java (original)
+++ tomcat/trunk/test/javax/servlet/http/TestHttpServlet.java Tue Aug 30
16:41:12 2016
@@ -44,7 +44,7 @@ public class TestHttpServlet extends Tom
// Map the test Servlet
LargeBodyServlet largeBodyServlet = new LargeBodyServlet();
Tomcat.addServlet(ctx, "largeBodyServlet", largeBodyServlet);
- ctx.addServletMapping("/", "largeBodyServlet");
+ ctx.addServletMappingDecoded("/", "largeBodyServlet");
tomcat.start();
@@ -84,11 +84,11 @@ public class TestHttpServlet extends Tom
Bug57602ServletOuter outer = new Bug57602ServletOuter();
Tomcat.addServlet(ctx, "Bug57602ServletOuter", outer);
- ctx.addServletMapping("/outer", "Bug57602ServletOuter");
+ ctx.addServletMappingDecoded("/outer", "Bug57602ServletOuter");
Bug57602ServletInner inner = new Bug57602ServletInner();
Tomcat.addServlet(ctx, "Bug57602ServletInner", inner);
- ctx.addServletMapping("/inner", "Bug57602ServletInner");
+ ctx.addServletMappingDecoded("/inner", "Bug57602ServletInner");
tomcat.start();
Modified:
tomcat/trunk/test/org/apache/catalina/authenticator/TestDigestAuthenticator.java
URL:
http://svn.apache.org/viewvc/tomcat/trunk/test/org/apache/catalina/authenticator/TestDigestAuthenticator.java?rev=1758423&r1=1758422&r2=1758423&view=diff
==============================================================================
---
tomcat/trunk/test/org/apache/catalina/authenticator/TestDigestAuthenticator.java
(original)
+++
tomcat/trunk/test/org/apache/catalina/authenticator/TestDigestAuthenticator.java
Tue Aug 30 16:41:12 2016
@@ -275,9 +275,9 @@ public class TestDigestAuthenticator ext
// Add protected servlet
Tomcat.addServlet(ctxt, "TesterServlet", new TesterServlet());
- ctxt.addServletMapping(URI, "TesterServlet");
+ ctxt.addServletMappingDecoded(URI, "TesterServlet");
SecurityCollection collection = new SecurityCollection();
- collection.addPattern(URI);
+ collection.addPatternDecoded(URI);
SecurityConstraint sc = new SecurityConstraint();
sc.addAuthRole(ROLE);
sc.addCollection(collection);
Modified:
tomcat/trunk/test/org/apache/catalina/authenticator/TestFormAuthenticator.java
URL:
http://svn.apache.org/viewvc/tomcat/trunk/test/org/apache/catalina/authenticator/TestFormAuthenticator.java?rev=1758423&r1=1758422&r2=1758423&view=diff
==============================================================================
---
tomcat/trunk/test/org/apache/catalina/authenticator/TestFormAuthenticator.java
(original)
+++
tomcat/trunk/test/org/apache/catalina/authenticator/TestFormAuthenticator.java
Tue Aug 30 16:41:12 2016
@@ -700,19 +700,19 @@ public class TestFormAuthenticator exten
"", System.getProperty("java.io.tmpdir"));
Tomcat.addServlet(ctx, "SelectedMethods",
new SelectedMethodsServlet());
- ctx.addServletMapping("/test", "SelectedMethods");
+ ctx.addServletMappingDecoded("/test", "SelectedMethods");
// Login servlet just needs to respond "OK". Client will handle
// creating a valid response. No need for a form.
Tomcat.addServlet(ctx, "Login",
new TesterServlet());
- ctx.addServletMapping("/login", "Login");
+ ctx.addServletMappingDecoded("/login", "Login");
// Configure the security constraints
SecurityConstraint constraint = new SecurityConstraint();
SecurityCollection collection = new SecurityCollection();
collection.setName("Protect PUT");
collection.addMethod("PUT");
- collection.addPattern("/test");
+ collection.addPatternDecoded("/test");
constraint.addCollection(collection);
constraint.addAuthRole("tomcat");
ctx.addConstraint(constraint);
Modified:
tomcat/trunk/test/org/apache/catalina/authenticator/TestNonLoginAndBasicAuthenticator.java
URL:
http://svn.apache.org/viewvc/tomcat/trunk/test/org/apache/catalina/authenticator/TestNonLoginAndBasicAuthenticator.java?rev=1758423&r1=1758422&r2=1758423&view=diff
==============================================================================
---
tomcat/trunk/test/org/apache/catalina/authenticator/TestNonLoginAndBasicAuthenticator.java
(original)
+++
tomcat/trunk/test/org/apache/catalina/authenticator/TestNonLoginAndBasicAuthenticator.java
Tue Aug 30 16:41:12 2016
@@ -458,10 +458,10 @@ public class TestNonLoginAndBasicAuthent
// Add protected servlet to the context
Tomcat.addServlet(nonloginContext, "TesterServlet1", new
TesterServlet());
- nonloginContext.addServletMapping(URI_PROTECTED, "TesterServlet1");
+ nonloginContext.addServletMappingDecoded(URI_PROTECTED,
"TesterServlet1");
SecurityCollection collection1 = new SecurityCollection();
- collection1.addPattern(URI_PROTECTED);
+ collection1.addPatternDecoded(URI_PROTECTED);
SecurityConstraint sc1 = new SecurityConstraint();
sc1.addAuthRole(ROLE);
sc1.addCollection(collection1);
@@ -469,10 +469,10 @@ public class TestNonLoginAndBasicAuthent
// Add unprotected servlet to the context
Tomcat.addServlet(nonloginContext, "TesterServlet2", new
TesterServlet());
- nonloginContext.addServletMapping(URI_PUBLIC, "TesterServlet2");
+ nonloginContext.addServletMappingDecoded(URI_PUBLIC, "TesterServlet2");
SecurityCollection collection2 = new SecurityCollection();
- collection2.addPattern(URI_PUBLIC);
+ collection2.addPatternDecoded(URI_PUBLIC);
SecurityConstraint sc2 = new SecurityConstraint();
// do not add a role - which signals access permitted without one
sc2.addCollection(collection2);
@@ -494,9 +494,9 @@ public class TestNonLoginAndBasicAuthent
// Add protected servlet to the context
Tomcat.addServlet(basicContext, "TesterServlet3", new TesterServlet());
- basicContext.addServletMapping(URI_PROTECTED, "TesterServlet3");
+ basicContext.addServletMappingDecoded(URI_PROTECTED, "TesterServlet3");
SecurityCollection collection = new SecurityCollection();
- collection.addPattern(URI_PROTECTED);
+ collection.addPatternDecoded(URI_PROTECTED);
SecurityConstraint sc = new SecurityConstraint();
sc.addAuthRole(ROLE);
sc.addCollection(collection);
@@ -504,10 +504,10 @@ public class TestNonLoginAndBasicAuthent
// Add unprotected servlet to the context
Tomcat.addServlet(basicContext, "TesterServlet4", new TesterServlet());
- basicContext.addServletMapping(URI_PUBLIC, "TesterServlet4");
+ basicContext.addServletMappingDecoded(URI_PUBLIC, "TesterServlet4");
SecurityCollection collection2 = new SecurityCollection();
- collection2.addPattern(URI_PUBLIC);
+ collection2.addPatternDecoded(URI_PUBLIC);
SecurityConstraint sc2 = new SecurityConstraint();
// do not add a role - which signals access permitted without one
sc2.addCollection(collection2);
Modified:
tomcat/trunk/test/org/apache/catalina/authenticator/TestSSOnonLoginAndBasicAuthenticator.java
URL:
http://svn.apache.org/viewvc/tomcat/trunk/test/org/apache/catalina/authenticator/TestSSOnonLoginAndBasicAuthenticator.java?rev=1758423&r1=1758422&r2=1758423&view=diff
==============================================================================
---
tomcat/trunk/test/org/apache/catalina/authenticator/TestSSOnonLoginAndBasicAuthenticator.java
(original)
+++
tomcat/trunk/test/org/apache/catalina/authenticator/TestSSOnonLoginAndBasicAuthenticator.java
Tue Aug 30 16:41:12 2016
@@ -484,10 +484,10 @@ public class TestSSOnonLoginAndBasicAuth
// Add protected servlet to the context
Tomcat.addServlet(nonloginContext, "TesterServlet1",
new TesterServletEncodeUrl());
- nonloginContext.addServletMapping(URI_PROTECTED, "TesterServlet1");
+ nonloginContext.addServletMappingDecoded(URI_PROTECTED,
"TesterServlet1");
SecurityCollection collection1 = new SecurityCollection();
- collection1.addPattern(URI_PROTECTED);
+ collection1.addPatternDecoded(URI_PROTECTED);
SecurityConstraint sc1 = new SecurityConstraint();
sc1.addAuthRole(ROLE);
sc1.addCollection(collection1);
@@ -496,10 +496,10 @@ public class TestSSOnonLoginAndBasicAuth
// Add unprotected servlet to the context
Tomcat.addServlet(nonloginContext, "TesterServlet2",
new TesterServletEncodeUrl());
- nonloginContext.addServletMapping(URI_PUBLIC, "TesterServlet2");
+ nonloginContext.addServletMappingDecoded(URI_PUBLIC, "TesterServlet2");
SecurityCollection collection2 = new SecurityCollection();
- collection2.addPattern(URI_PUBLIC);
+ collection2.addPatternDecoded(URI_PUBLIC);
SecurityConstraint sc2 = new SecurityConstraint();
// do not add a role - which signals access permitted without one
sc2.addCollection(collection2);
@@ -523,9 +523,9 @@ public class TestSSOnonLoginAndBasicAuth
// Add protected servlet to the context
Tomcat.addServlet(basicContext, "TesterServlet3",
new TesterServletEncodeUrl());
- basicContext.addServletMapping(URI_PROTECTED, "TesterServlet3");
+ basicContext.addServletMappingDecoded(URI_PROTECTED, "TesterServlet3");
SecurityCollection collection = new SecurityCollection();
- collection.addPattern(URI_PROTECTED);
+ collection.addPatternDecoded(URI_PROTECTED);
SecurityConstraint sc = new SecurityConstraint();
sc.addAuthRole(ROLE);
sc.addCollection(collection);
@@ -534,9 +534,9 @@ public class TestSSOnonLoginAndBasicAuth
// Add unprotected servlet to the context
Tomcat.addServlet(basicContext, "TesterServlet4",
new TesterServletEncodeUrl());
- basicContext.addServletMapping(URI_PUBLIC, "TesterServlet4");
+ basicContext.addServletMappingDecoded(URI_PUBLIC, "TesterServlet4");
SecurityCollection collection2 = new SecurityCollection();
- collection2.addPattern(URI_PUBLIC);
+ collection2.addPatternDecoded(URI_PUBLIC);
SecurityConstraint sc2 = new SecurityConstraint();
// do not add a role - which signals access permitted without one
sc2.addCollection(collection2);
Modified:
tomcat/trunk/test/org/apache/catalina/authenticator/TestSSOnonLoginAndDigestAuthenticator.java
URL:
http://svn.apache.org/viewvc/tomcat/trunk/test/org/apache/catalina/authenticator/TestSSOnonLoginAndDigestAuthenticator.java?rev=1758423&r1=1758422&r2=1758423&view=diff
==============================================================================
---
tomcat/trunk/test/org/apache/catalina/authenticator/TestSSOnonLoginAndDigestAuthenticator.java
(original)
+++
tomcat/trunk/test/org/apache/catalina/authenticator/TestSSOnonLoginAndDigestAuthenticator.java
Tue Aug 30 16:41:12 2016
@@ -330,9 +330,9 @@ public class TestSSOnonLoginAndDigestAut
// Add protected servlet
Tomcat.addServlet(ctxt, "TesterServlet1", new TesterServlet());
- ctxt.addServletMapping(URI_PROTECTED, "TesterServlet1");
+ ctxt.addServletMappingDecoded(URI_PROTECTED, "TesterServlet1");
SecurityCollection collection1 = new SecurityCollection();
- collection1.addPattern(URI_PROTECTED);
+ collection1.addPatternDecoded(URI_PROTECTED);
SecurityConstraint sc1 = new SecurityConstraint();
sc1.addAuthRole(ROLE);
sc1.addCollection(collection1);
@@ -340,9 +340,9 @@ public class TestSSOnonLoginAndDigestAut
// Add unprotected servlet
Tomcat.addServlet(ctxt, "TesterServlet2", new TesterServlet());
- ctxt.addServletMapping(URI_PUBLIC, "TesterServlet2");
+ ctxt.addServletMappingDecoded(URI_PUBLIC, "TesterServlet2");
SecurityCollection collection2 = new SecurityCollection();
- collection2.addPattern(URI_PUBLIC);
+ collection2.addPatternDecoded(URI_PUBLIC);
SecurityConstraint sc2 = new SecurityConstraint();
// do not add a role - which signals access permitted without one
sc2.addCollection(collection2);
@@ -364,9 +364,9 @@ public class TestSSOnonLoginAndDigestAut
// Add protected servlet
Tomcat.addServlet(ctxt, "TesterServlet3", new TesterServlet());
- ctxt.addServletMapping(URI_PROTECTED, "TesterServlet3");
+ ctxt.addServletMappingDecoded(URI_PROTECTED, "TesterServlet3");
SecurityCollection collection = new SecurityCollection();
- collection.addPattern(URI_PROTECTED);
+ collection.addPatternDecoded(URI_PROTECTED);
SecurityConstraint sc = new SecurityConstraint();
sc.addAuthRole(ROLE);
sc.addCollection(collection);
Modified: tomcat/trunk/test/org/apache/catalina/connector/TestConnector.java
URL:
http://svn.apache.org/viewvc/tomcat/trunk/test/org/apache/catalina/connector/TestConnector.java?rev=1758423&r1=1758422&r2=1758423&view=diff
==============================================================================
--- tomcat/trunk/test/org/apache/catalina/connector/TestConnector.java
(original)
+++ tomcat/trunk/test/org/apache/catalina/connector/TestConnector.java Tue Aug
30 16:41:12 2016
@@ -43,7 +43,7 @@ public class TestConnector extends Tomca
Wrapper w =
Tomcat.addServlet(root, "tester", new TesterServlet());
w.setAsyncSupported(true);
- root.addServletMapping("/", "tester");
+ root.addServletMappingDecoded("/", "tester");
Connector connector = tomcat.getConnector();
Modified: tomcat/trunk/test/org/apache/catalina/connector/TestCoyoteAdapter.java
URL:
http://svn.apache.org/viewvc/tomcat/trunk/test/org/apache/catalina/connector/TestCoyoteAdapter.java?rev=1758423&r1=1758422&r2=1758423&view=diff
==============================================================================
--- tomcat/trunk/test/org/apache/catalina/connector/TestCoyoteAdapter.java
(original)
+++ tomcat/trunk/test/org/apache/catalina/connector/TestCoyoteAdapter.java Tue
Aug 30 16:41:12 2016
@@ -115,7 +115,7 @@ public class TestCoyoteAdapter extends T
Context ctx = tomcat.addContext("", docBase.getAbsolutePath());
Tomcat.addServlet(ctx, "servlet", new PathParamServlet());
- ctx.addServletMapping("/", "servlet");
+ ctx.addServletMappingDecoded("/", "servlet");
tomcat.start();
@@ -136,7 +136,7 @@ public class TestCoyoteAdapter extends T
Context ctx = tomcat.addContext("", null);
Tomcat.addServlet(ctx, "servlet", new PathParamServlet());
- ctx.addServletMapping("/", "servlet");
+ ctx.addServletMappingDecoded("/", "servlet");
tomcat.start();
@@ -189,7 +189,7 @@ public class TestCoyoteAdapter extends T
Context ctx = tomcat.addContext("/testapp", null);
Tomcat.addServlet(ctx, "servlet", new PathParamServlet());
- ctx.addServletMapping("*.txt", "servlet");
+ ctx.addServletMappingDecoded("*.txt", "servlet");
tomcat.start();
@@ -240,7 +240,7 @@ public class TestCoyoteAdapter extends T
PathInfoServlet servlet = new PathInfoServlet();
Tomcat.addServlet(ctx, "servlet", servlet);
- ctx.addServletMapping("/*", "servlet");
+ ctx.addServletMappingDecoded("/*", "servlet");
tomcat.start();
@@ -283,7 +283,7 @@ public class TestCoyoteAdapter extends T
AsyncServlet servlet = new AsyncServlet();
Wrapper w = Tomcat.addServlet(ctx, "async", servlet);
w.setAsyncSupported(true);
- ctx.addServletMapping("/async", "async");
+ ctx.addServletMappingDecoded("/async", "async");
tomcat.start();
Modified:
tomcat/trunk/test/org/apache/catalina/connector/TestCoyoteOutputStream.java
URL:
http://svn.apache.org/viewvc/tomcat/trunk/test/org/apache/catalina/connector/TestCoyoteOutputStream.java?rev=1758423&r1=1758422&r2=1758423&view=diff
==============================================================================
--- tomcat/trunk/test/org/apache/catalina/connector/TestCoyoteOutputStream.java
(original)
+++ tomcat/trunk/test/org/apache/catalina/connector/TestCoyoteOutputStream.java
Tue Aug 30 16:41:12 2016
@@ -108,7 +108,7 @@ public class TestCoyoteOutputStream exte
Context root = tomcat.addContext("", TEMP_DIR);
Tomcat.addServlet(root, "testServlet", new TestServlet());
- root.addServletMapping("/", "testServlet");
+ root.addServletMappingDecoded("/", "testServlet");
tomcat.start();
@@ -132,11 +132,11 @@ public class TestCoyoteOutputStream exte
Wrapper w = Tomcat.addServlet(root, "nbWrite",
new NonBlockingWriteServlet(asyncWriteTarget,
useContainerThreadToSetListener));
w.setAsyncSupported(true);
- root.addServletMapping("/nbWrite", "nbWrite");
+ root.addServletMappingDecoded("/nbWrite", "nbWrite");
Tomcat.addServlet(root, "write",
new BlockingWriteServlet(asyncWriteTarget, syncWriteTarget));
w.setAsyncSupported(true);
- root.addServletMapping("/write", "write");
+ root.addServletMappingDecoded("/write", "write");
tomcat.start();
Modified: tomcat/trunk/test/org/apache/catalina/connector/TestInputBuffer.java
URL:
http://svn.apache.org/viewvc/tomcat/trunk/test/org/apache/catalina/connector/TestInputBuffer.java?rev=1758423&r1=1758422&r2=1758423&view=diff
==============================================================================
--- tomcat/trunk/test/org/apache/catalina/connector/TestInputBuffer.java
(original)
+++ tomcat/trunk/test/org/apache/catalina/connector/TestInputBuffer.java Tue
Aug 30 16:41:12 2016
@@ -44,7 +44,7 @@ public class TestInputBuffer extends Tom
Tomcat tomcat = getTomcatInstance();
Context root = tomcat.addContext("", TEMP_DIR);
Tomcat.addServlet(root, "Echo", new Utf8Echo());
- root.addServletMapping("/test", "Echo");
+ root.addServletMappingDecoded("/test", "Echo");
tomcat.getConnector().setProperty("soTimeout", "300000");
tomcat.start();
Modified:
tomcat/trunk/test/org/apache/catalina/connector/TestKeepAliveCount.java
URL:
http://svn.apache.org/viewvc/tomcat/trunk/test/org/apache/catalina/connector/TestKeepAliveCount.java?rev=1758423&r1=1758422&r2=1758423&view=diff
==============================================================================
--- tomcat/trunk/test/org/apache/catalina/connector/TestKeepAliveCount.java
(original)
+++ tomcat/trunk/test/org/apache/catalina/connector/TestKeepAliveCount.java Tue
Aug 30 16:41:12 2016
@@ -58,7 +58,7 @@ public class TestKeepAliveCount extends
Tomcat tomcat = getTomcatInstance();
Context root = tomcat.addContext("", TEMP_DIR);
Tomcat.addServlet(root, "Simple", new SimpleServlet());
- root.addServletMapping("/test", "Simple");
+ root.addServletMappingDecoded("/test", "Simple");
tomcat.getConnector().setProperty("maxKeepAliveRequests", "5");
tomcat.getConnector().setProperty("soTimeout", "20000");
tomcat.getConnector().setProperty("keepAliveTimeout", "50000");
Modified:
tomcat/trunk/test/org/apache/catalina/connector/TestMaxConnections.java
URL:
http://svn.apache.org/viewvc/tomcat/trunk/test/org/apache/catalina/connector/TestMaxConnections.java?rev=1758423&r1=1758422&r2=1758423&view=diff
==============================================================================
--- tomcat/trunk/test/org/apache/catalina/connector/TestMaxConnections.java
(original)
+++ tomcat/trunk/test/org/apache/catalina/connector/TestMaxConnections.java Tue
Aug 30 16:41:12 2016
@@ -76,7 +76,7 @@ public class TestMaxConnections extends
StandardContext root = (StandardContext) tomcat.addContext("",
SimpleHttpClient.TEMP_DIR);
root.setUnloadDelay(soTimeout);
Tomcat.addServlet(root, "Simple", new SimpleServlet());
- root.addServletMapping("/test", "Simple");
+ root.addServletMappingDecoded("/test", "Simple");
tomcat.getConnector().setProperty("maxKeepAliveRequests", "1");
tomcat.getConnector().setProperty("maxThreads", "10");
tomcat.getConnector().setProperty("soTimeout", "20000");
Modified: tomcat/trunk/test/org/apache/catalina/connector/TestOutputBuffer.java
URL:
http://svn.apache.org/viewvc/tomcat/trunk/test/org/apache/catalina/connector/TestOutputBuffer.java?rev=1758423&r1=1758422&r2=1758423&view=diff
==============================================================================
--- tomcat/trunk/test/org/apache/catalina/connector/TestOutputBuffer.java
(original)
+++ tomcat/trunk/test/org/apache/catalina/connector/TestOutputBuffer.java Tue
Aug 30 16:41:12 2016
@@ -50,7 +50,7 @@ public class TestOutputBuffer extends To
for (int i = 1; i <= WritingServlet.EXPECTED_CONTENT_LENGTH; i*=10) {
WritingServlet servlet = new WritingServlet(i);
Tomcat.addServlet(root, "servlet" + i, servlet);
- root.addServletMapping("/servlet" + i, "servlet" + i);
+ root.addServletMappingDecoded("/servlet" + i, "servlet" + i);
}
tomcat.start();
@@ -84,7 +84,7 @@ public class TestOutputBuffer extends To
Bug52577Servlet bug52577 = new Bug52577Servlet();
Tomcat.addServlet(root, "bug52577", bug52577);
- root.addServletMapping("/", "bug52577");
+ root.addServletMappingDecoded("/", "bug52577");
tomcat.start();
Modified: tomcat/trunk/test/org/apache/catalina/connector/TestRequest.java
URL:
http://svn.apache.org/viewvc/tomcat/trunk/test/org/apache/catalina/connector/TestRequest.java?rev=1758423&r1=1758422&r2=1758423&view=diff
==============================================================================
--- tomcat/trunk/test/org/apache/catalina/connector/TestRequest.java (original)
+++ tomcat/trunk/test/org/apache/catalina/connector/TestRequest.java Tue Aug 30
16:41:12 2016
@@ -177,7 +177,7 @@ public class TestRequest extends TomcatB
Tomcat tomcat = getTomcatInstance();
Context root = tomcat.addContext("", TEMP_DIR);
Tomcat.addServlet(root, "Bug37794", new Bug37794Servlet());
- root.addServletMapping("/test", "Bug37794");
+ root.addServletMappingDecoded("/test", "Bug37794");
if (createFilter) {
FilterDef failedRequestFilter = new FilterDef();
@@ -186,7 +186,7 @@ public class TestRequest extends TomcatB
FailedRequestFilter.class.getName());
FilterMap failedRequestFilterMap = new FilterMap();
failedRequestFilterMap.setFilterName("failedRequestFilter");
- failedRequestFilterMap.addURLPattern("/*");
+ failedRequestFilterMap.addURLPatternDecoded("/*");
root.addFilterDef(failedRequestFilter);
root.addFilterMap(failedRequestFilterMap);
}
@@ -277,7 +277,7 @@ public class TestRequest extends TomcatB
// Add the Servlet
Tomcat.addServlet(ctx, "servlet", new EchoQueryStringServlet());
- ctx.addServletMapping("/", "servlet");
+ ctx.addServletMappingDecoded("/", "servlet");
tomcat.start();
@@ -325,7 +325,7 @@ public class TestRequest extends TomcatB
ctx.getPipeline().addValve(new BasicAuthenticator());
Tomcat.addServlet(ctx, "servlet", new LoginLogoutServlet());
- ctx.addServletMapping("/", "servlet");
+ ctx.addServletMappingDecoded("/", "servlet");
TesterMapRealm realm = new TesterMapRealm();
realm.addUser(LoginLogoutServlet.USER, LoginLogoutServlet.PWD);
@@ -372,7 +372,7 @@ public class TestRequest extends TomcatB
Context root = tomcat.addContext("",
System.getProperty("java.io.tmpdir"));
Tomcat.addServlet(root, "Bug37794", new Bug37794Servlet());
- root.addServletMapping("/", "Bug37794");
+ root.addServletMappingDecoded("/", "Bug37794");
tomcat.start();
HttpURLConnection conn = getConnection("http://localhost:"; + getPort()
+ "/");
@@ -386,7 +386,7 @@ public class TestRequest extends TomcatB
Context root = tomcat.addContext("",
System.getProperty("java.io.tmpdir"));
Tomcat.addServlet(root, "Bug37794", new Bug37794Servlet());
- root.addServletMapping("/", "Bug37794");
+ root.addServletMappingDecoded("/", "Bug37794");
tomcat.start();
HttpURLConnection conn = getConnection("http://localhost:"; + getPort()
+ "/");
@@ -479,7 +479,7 @@ public class TestRequest extends TomcatB
System.getProperty("java.io.tmpdir"));
root.setAllowCasualMultipartParsing(true);
Tomcat.addServlet(root, "Bug54984", new Bug54984Servlet());
- root.addServletMapping("/", "Bug54984");
+ root.addServletMappingDecoded("/", "Bug54984");
tomcat.start();
HttpURLConnection conn = getConnection("http://localhost:"; + getPort()
@@ -556,7 +556,7 @@ public class TestRequest extends TomcatB
Tomcat tomcat = getTomcatInstance();
Context root = tomcat.addContext("", TEMP_DIR);
Tomcat.addServlet(root, "EchoParameters", new
EchoParametersServlet());
- root.addServletMapping("/echo", "EchoParameters");
+ root.addServletMappingDecoded("/echo", "EchoParameters");
tomcat.start();
setPort(tomcat.getConnector().getLocalPort());
@@ -829,7 +829,7 @@ public class TestRequest extends TomcatB
Context ctx = tomcat.addContext(deployPath, null);
Tomcat.addServlet(ctx, "servlet", new Bug56501Servelet());
- ctx.addServletMapping("/*", "servlet");
+ ctx.addServletMappingDecoded("/*", "servlet");
tomcat.start();
Modified: tomcat/trunk/test/org/apache/catalina/connector/TestResponse.java
URL:
http://svn.apache.org/viewvc/tomcat/trunk/test/org/apache/catalina/connector/TestResponse.java?rev=1758423&r1=1758422&r2=1758423&view=diff
==============================================================================
--- tomcat/trunk/test/org/apache/catalina/connector/TestResponse.java (original)
+++ tomcat/trunk/test/org/apache/catalina/connector/TestResponse.java Tue Aug
30 16:41:12 2016
@@ -56,7 +56,7 @@ public class TestResponse extends Tomcat
Context ctx = tomcat.addContext("", null);
Tomcat.addServlet(ctx, "servlet", new Bug49598Servlet());
- ctx.addServletMapping("/", "servlet");
+ ctx.addServletMappingDecoded("/", "servlet");
tomcat.start();
@@ -112,7 +112,7 @@ public class TestResponse extends Tomcat
Context ctx = tomcat.addContext("", null);
Tomcat.addServlet(ctx, "servlet", new CharsetServlet());
- ctx.addServletMapping("/", "servlet");
+ ctx.addServletMappingDecoded("/", "servlet");
tomcat.start();
@@ -151,7 +151,7 @@ public class TestResponse extends Tomcat
Context ctx = tomcat.addContext("", null);
Tomcat.addServlet(ctx, "servlet", new Bug52811Servlet());
- ctx.addServletMapping("/", "servlet");
+ ctx.addServletMappingDecoded("/", "servlet");
tomcat.start();
Modified: tomcat/trunk/test/org/apache/catalina/connector/TestSendFile.java
URL:
http://svn.apache.org/viewvc/tomcat/trunk/test/org/apache/catalina/connector/TestSendFile.java?rev=1758423&r1=1758422&r2=1758423&view=diff
==============================================================================
--- tomcat/trunk/test/org/apache/catalina/connector/TestSendFile.java (original)
+++ tomcat/trunk/test/org/apache/catalina/connector/TestSendFile.java Tue Aug
30 16:41:12 2016
@@ -62,7 +62,7 @@ public class TestSendFile extends Tomcat
for (int i=0; i<ITERATIONS; i++) {
WritingServlet servlet = new WritingServlet(files[i]);
Tomcat.addServlet(root, "servlet" + i, servlet);
- root.addServletMapping("/servlet" + i, "servlet" + i);
+ root.addServletMappingDecoded("/servlet" + i, "servlet" + i);
}
tomcat.start();
Modified: tomcat/trunk/test/org/apache/catalina/core/TestApplicationContext.java
URL:
http://svn.apache.org/viewvc/tomcat/trunk/test/org/apache/catalina/core/TestApplicationContext.java?rev=1758423&r1=1758422&r2=1758423&view=diff
==============================================================================
--- tomcat/trunk/test/org/apache/catalina/core/TestApplicationContext.java
(original)
+++ tomcat/trunk/test/org/apache/catalina/core/TestApplicationContext.java Tue
Aug 30 16:41:12 2016
@@ -189,7 +189,7 @@ public class TestApplicationContext exte
ctx.setCrossContext(true);
Tomcat.addServlet(ctx, "Bug57190Servlet", new Bug57190Servlet());
- ctx.addServletMapping("/", "Bug57190Servlet");
+ ctx.addServletMappingDecoded("/", "Bug57190Servlet");
tomcat.start();
@@ -262,14 +262,14 @@ public class TestApplicationContext exte
Context ctx2 = tomcat.addContext("/second", null);
GetAttributeServlet getAttributeServlet = new GetAttributeServlet();
Tomcat.addServlet(ctx2, "getAttributeServlet", getAttributeServlet);
- ctx2.addServletMapping("/test", "getAttributeServlet");
+ ctx2.addServletMappingDecoded("/test", "getAttributeServlet");
// No file system docBase required
Context ctx1 = tomcat.addContext("/first", null);
ctx1.setCrossContext(true);
SetAttributeServlet setAttributeServlet = new
SetAttributeServlet("/test", "/second");
Tomcat.addServlet(ctx1, "setAttributeServlet", setAttributeServlet);
- ctx1.addServletMapping("/test", "setAttributeServlet");
+ ctx1.addServletMappingDecoded("/test", "setAttributeServlet");
tomcat.start();
Modified:
tomcat/trunk/test/org/apache/catalina/core/TestApplicationContextGetRequestDispatcher.java
URL:
http://svn.apache.org/viewvc/tomcat/trunk/test/org/apache/catalina/core/TestApplicationContextGetRequestDispatcher.java?rev=1758423&r1=1758422&r2=1758423&view=diff
==============================================================================
---
tomcat/trunk/test/org/apache/catalina/core/TestApplicationContextGetRequestDispatcher.java
(original)
+++
tomcat/trunk/test/org/apache/catalina/core/TestApplicationContextGetRequestDispatcher.java
Tue Aug 30 16:41:12 2016
@@ -39,6 +39,7 @@ import org.apache.catalina.startup.Tomca
import org.apache.catalina.startup.TomcatBaseTest;
import org.apache.catalina.util.URLEncoder;
import org.apache.tomcat.util.buf.ByteChunk;
+import org.apache.tomcat.util.buf.UDecoder;
@RunWith(value = Parameterized.class)
public class TestApplicationContextGetRequestDispatcher extends TomcatBaseTest
{
@@ -364,12 +365,11 @@ public class TestApplicationContextGetRe
// Add a default servlet to return 404 for not found resources
Tomcat.addServlet(ctx, "Default", new Default404Servlet());
- ctx.addServletMapping("/*", "Default");
+ ctx.addServletMappingDecoded("/*", "Default");
// Add a target servlet to dispatch to
Tomcat.addServlet(ctx, "target", new TargetServlet());
- // Note: This will decode the provided path
- ctx.addServletMapping(targetPath, "target");
+ ctx.addServletMappingDecoded(UDecoder.URLDecode(targetPath, "UTF-8"),
"target");
if (useAsync) {
Wrapper w = Tomcat.addServlet(
@@ -378,8 +378,7 @@ public class TestApplicationContextGetRe
} else {
Tomcat.addServlet(ctx, "rd", new DispatcherServlet(dispatchPath));
}
- // Note: This will decode the provided path
- ctx.addServletMapping(startPath, "rd");
+ ctx.addServletMappingDecoded(UDecoder.URLDecode(startPath, "UTF-8"),
"rd");
tomcat.start();
Modified:
tomcat/trunk/test/org/apache/catalina/core/TestApplicationFilterConfig.java
URL:
http://svn.apache.org/viewvc/tomcat/trunk/test/org/apache/catalina/core/TestApplicationFilterConfig.java?rev=1758423&r1=1758422&r2=1758423&view=diff
==============================================================================
--- tomcat/trunk/test/org/apache/catalina/core/TestApplicationFilterConfig.java
(original)
+++ tomcat/trunk/test/org/apache/catalina/core/TestApplicationFilterConfig.java
Tue Aug 30 16:41:12 2016
@@ -41,7 +41,7 @@ public class TestApplicationFilterConfig
Context ctx = tomcat.addContext("", null);
Tomcat.addServlet(ctx, "HelloWorld", new HelloWorldServlet());
- ctx.addServletMapping("/", "HelloWorld");
+ ctx.addServletMappingDecoded("/", "HelloWorld");
// Add a filter with a name that should be escaped if used in a JMX
// object name
Modified:
tomcat/trunk/test/org/apache/catalina/core/TestApplicationHttpRequest.java
URL:
http://svn.apache.org/viewvc/tomcat/trunk/test/org/apache/catalina/core/TestApplicationHttpRequest.java?rev=1758423&r1=1758422&r2=1758423&view=diff
==============================================================================
--- tomcat/trunk/test/org/apache/catalina/core/TestApplicationHttpRequest.java
(original)
+++ tomcat/trunk/test/org/apache/catalina/core/TestApplicationHttpRequest.java
Tue Aug 30 16:41:12 2016
@@ -189,10 +189,10 @@ public class TestApplicationHttpRequest
} else {
Tomcat.addServlet(ctx, "forward", new ForwardServlet("/display?" +
forwardQueryString));
}
- ctx.addServletMapping("/forward", "forward");
+ ctx.addServletMappingDecoded("/forward", "forward");
Tomcat.addServlet(ctx, "display", new
DisplayParameterServlet(expected));
- ctx.addServletMapping("/display", "display");
+ ctx.addServletMappingDecoded("/display", "display");
tomcat.start();
@@ -219,10 +219,10 @@ public class TestApplicationHttpRequest
Context ctx = tomcat.addContext("", null);
Tomcat.addServlet(ctx, "forward", new ForwardServlet("/modify"));
- ctx.addServletMapping("/forward", "forward");
+ ctx.addServletMappingDecoded("/forward", "forward");
Tomcat.addServlet(ctx, "modify", new ModifyParameterServlet());
- ctx.addServletMapping("/modify", "modify");
+ ctx.addServletMappingDecoded("/modify", "modify");
tomcat.start();
Modified: tomcat/trunk/test/org/apache/catalina/core/TestApplicationMapping.java
URL:
http://svn.apache.org/viewvc/tomcat/trunk/test/org/apache/catalina/core/TestApplicationMapping.java?rev=1758423&r1=1758422&r2=1758423&view=diff
==============================================================================
--- tomcat/trunk/test/org/apache/catalina/core/TestApplicationMapping.java
(original)
+++ tomcat/trunk/test/org/apache/catalina/core/TestApplicationMapping.java Tue
Aug 30 16:41:12 2016
@@ -111,7 +111,7 @@ public class TestApplicationMapping exte
Context ctx = tomcat.addContext(contextPath, null);
Tomcat.addServlet(ctx, "Mapping", new MappingServlet());
- ctx.addServletMapping(mapping, "Mapping");
+ ctx.addServletMappingDecoded(mapping, "Mapping");
tomcat.start();
@@ -132,9 +132,9 @@ public class TestApplicationMapping exte
Context ctx = tomcat.addContext(contextPath, null);
Tomcat.addServlet(ctx, "Include", new IncludeServlet());
- ctx.addServletMapping(mapping, "Include");
+ ctx.addServletMappingDecoded(mapping, "Include");
Tomcat.addServlet(ctx, "Mapping", new MappingServlet());
- ctx.addServletMapping("/mapping", "Mapping");
+ ctx.addServletMappingDecoded("/mapping", "Mapping");
tomcat.start();
@@ -160,9 +160,9 @@ public class TestApplicationMapping exte
Context ctx = tomcat.addContext(contextPath, null);
Tomcat.addServlet(ctx, "Include", new NamedIncludeServlet());
- ctx.addServletMapping(mapping, "Include");
+ ctx.addServletMappingDecoded(mapping, "Include");
Tomcat.addServlet(ctx, "Mapping", new MappingServlet());
- ctx.addServletMapping("/mapping", "Mapping");
+ ctx.addServletMappingDecoded("/mapping", "Mapping");
tomcat.start();
@@ -183,9 +183,9 @@ public class TestApplicationMapping exte
Context ctx = tomcat.addContext(contextPath, null);
Tomcat.addServlet(ctx, "Forward", new ForwardServlet());
- ctx.addServletMapping(mapping, "Forward");
+ ctx.addServletMappingDecoded(mapping, "Forward");
Tomcat.addServlet(ctx, "Mapping", new MappingServlet());
- ctx.addServletMapping("/mapping", "Mapping");
+ ctx.addServletMappingDecoded("/mapping", "Mapping");
tomcat.start();
@@ -211,9 +211,9 @@ public class TestApplicationMapping exte
Context ctx = tomcat.addContext(contextPath, null);
Tomcat.addServlet(ctx, "Forward", new NamedForwardServlet());
- ctx.addServletMapping(mapping, "Forward");
+ ctx.addServletMappingDecoded(mapping, "Forward");
Tomcat.addServlet(ctx, "Mapping", new MappingServlet());
- ctx.addServletMapping("/mapping", "Mapping");
+ ctx.addServletMappingDecoded("/mapping", "Mapping");
tomcat.start();
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org
