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/02/27 17:05:18 UTC
svn commit: r1784599 - in /tomcat/tc8.5.x/trunk:
java/org/apache/catalina/connector/ java/org/apache/catalina/core/
java/org/apache/catalina/servlet4preview/http/
test/org/apache/catalina/core/ webapps/docs/
Author: markt
Date: Mon Feb 27 17:05:17 2017
New Revision: 1784599
URL: http://svn.apache.org/viewvc?rev=1784599&view=rev
Log:
Rename Mapping to ServletMapping
Added:
tomcat/tc8.5.x/trunk/java/org/apache/catalina/servlet4preview/http/ServletMapping.java
- copied, changed from r1784598, tomcat/tc8.5.x/trunk/java/org/apache/catalina/servlet4preview/http/Mapping.java
Removed:
tomcat/tc8.5.x/trunk/java/org/apache/catalina/servlet4preview/http/Mapping.java
Modified:
tomcat/tc8.5.x/trunk/java/org/apache/catalina/connector/Request.java
tomcat/tc8.5.x/trunk/java/org/apache/catalina/connector/RequestFacade.java
tomcat/tc8.5.x/trunk/java/org/apache/catalina/core/ApplicationContext.java
tomcat/tc8.5.x/trunk/java/org/apache/catalina/core/ApplicationDispatcher.java
tomcat/tc8.5.x/trunk/java/org/apache/catalina/core/ApplicationHttpRequest.java
tomcat/tc8.5.x/trunk/java/org/apache/catalina/core/ApplicationMapping.java
tomcat/tc8.5.x/trunk/java/org/apache/catalina/servlet4preview/http/HttpServletRequest.java
tomcat/tc8.5.x/trunk/java/org/apache/catalina/servlet4preview/http/HttpServletRequestWrapper.java
tomcat/tc8.5.x/trunk/test/org/apache/catalina/core/TestApplicationMapping.java
tomcat/tc8.5.x/trunk/webapps/docs/changelog.xml
Modified: tomcat/tc8.5.x/trunk/java/org/apache/catalina/connector/Request.java
URL: http://svn.apache.org/viewvc/tomcat/tc8.5.x/trunk/java/org/apache/catalina/connector/Request.java?rev=1784599&r1=1784598&r2=1784599&view=diff
==============================================================================
--- tomcat/tc8.5.x/trunk/java/org/apache/catalina/connector/Request.java (original)
+++ tomcat/tc8.5.x/trunk/java/org/apache/catalina/connector/Request.java Mon Feb 27 17:05:17 2017
@@ -80,8 +80,8 @@ import org.apache.catalina.core.Applicat
import org.apache.catalina.core.ApplicationSessionCookieConfig;
import org.apache.catalina.core.AsyncContextImpl;
import org.apache.catalina.mapper.MappingData;
-import org.apache.catalina.servlet4preview.http.Mapping;
import org.apache.catalina.servlet4preview.http.PushBuilder;
+import org.apache.catalina.servlet4preview.http.ServletMapping;
import org.apache.catalina.util.ParameterMap;
import org.apache.catalina.util.URLEncoder;
import org.apache.coyote.ActionCode;
@@ -2195,8 +2195,8 @@ public class Request implements org.apac
@Override
- public Mapping getMapping() {
- return applicationMapping.getMapping();
+ public ServletMapping getServletMapping() {
+ return applicationMapping.getServletMapping();
}
Modified: tomcat/tc8.5.x/trunk/java/org/apache/catalina/connector/RequestFacade.java
URL: http://svn.apache.org/viewvc/tomcat/tc8.5.x/trunk/java/org/apache/catalina/connector/RequestFacade.java?rev=1784599&r1=1784598&r2=1784599&view=diff
==============================================================================
--- tomcat/tc8.5.x/trunk/java/org/apache/catalina/connector/RequestFacade.java (original)
+++ tomcat/tc8.5.x/trunk/java/org/apache/catalina/connector/RequestFacade.java Mon Feb 27 17:05:17 2017
@@ -42,8 +42,8 @@ import javax.servlet.http.Part;
import org.apache.catalina.Globals;
import org.apache.catalina.security.SecurityUtil;
import org.apache.catalina.servlet4preview.http.HttpServletRequest;
-import org.apache.catalina.servlet4preview.http.Mapping;
import org.apache.catalina.servlet4preview.http.PushBuilder;
+import org.apache.catalina.servlet4preview.http.ServletMapping;
import org.apache.tomcat.util.res.StringManager;
/**
@@ -1123,8 +1123,8 @@ public class RequestFacade implements Ht
* removed or replaced at any time until Servlet 4.0 becomes final.
*/
@Override
- public Mapping getMapping() {
- return request.getMapping();
+ public ServletMapping getServletMapping() {
+ return request.getServletMapping();
}
Modified: tomcat/tc8.5.x/trunk/java/org/apache/catalina/core/ApplicationContext.java
URL: http://svn.apache.org/viewvc/tomcat/tc8.5.x/trunk/java/org/apache/catalina/core/ApplicationContext.java?rev=1784599&r1=1784598&r2=1784599&view=diff
==============================================================================
--- tomcat/tc8.5.x/trunk/java/org/apache/catalina/core/ApplicationContext.java (original)
+++ tomcat/tc8.5.x/trunk/java/org/apache/catalina/core/ApplicationContext.java Mon Feb 27 17:05:17 2017
@@ -66,7 +66,7 @@ import org.apache.catalina.WebResourceRo
import org.apache.catalina.Wrapper;
import org.apache.catalina.connector.Connector;
import org.apache.catalina.mapper.MappingData;
-import org.apache.catalina.servlet4preview.http.Mapping;
+import org.apache.catalina.servlet4preview.http.ServletMapping;
import org.apache.catalina.util.ServerInfo;
import org.apache.catalina.util.URLEncoder;
import org.apache.tomcat.util.ExceptionUtils;
@@ -482,7 +482,7 @@ public class ApplicationContext implemen
Wrapper wrapper = mappingData.wrapper;
String wrapperPath = mappingData.wrapperPath.toString();
String pathInfo = mappingData.pathInfo.toString();
- Mapping mapping = (new ApplicationMapping(mappingData)).getMapping();
+ ServletMapping mapping = (new ApplicationMapping(mappingData)).getServletMapping();
mappingData.recycle();
Modified: tomcat/tc8.5.x/trunk/java/org/apache/catalina/core/ApplicationDispatcher.java
URL: http://svn.apache.org/viewvc/tomcat/tc8.5.x/trunk/java/org/apache/catalina/core/ApplicationDispatcher.java?rev=1784599&r1=1784598&r2=1784599&view=diff
==============================================================================
--- tomcat/tc8.5.x/trunk/java/org/apache/catalina/core/ApplicationDispatcher.java (original)
+++ tomcat/tc8.5.x/trunk/java/org/apache/catalina/core/ApplicationDispatcher.java Mon Feb 27 17:05:17 2017
@@ -44,7 +44,7 @@ import org.apache.catalina.connector.Req
import org.apache.catalina.connector.RequestFacade;
import org.apache.catalina.connector.Response;
import org.apache.catalina.connector.ResponseFacade;
-import org.apache.catalina.servlet4preview.http.Mapping;
+import org.apache.catalina.servlet4preview.http.ServletMapping;
import org.apache.tomcat.util.ExceptionUtils;
import org.apache.tomcat.util.res.StringManager;
@@ -206,7 +206,7 @@ final class ApplicationDispatcher implem
*/
public ApplicationDispatcher
(Wrapper wrapper, String requestURI, String servletPath,
- String pathInfo, String queryString, Mapping mapping, String name) {
+ String pathInfo, String queryString, ServletMapping mapping, String name) {
super();
@@ -263,7 +263,7 @@ final class ApplicationDispatcher implem
/**
* The mapping for this RequestDispatcher.
*/
- private final Mapping mapping;
+ private final ServletMapping mapping;
/**
@@ -369,12 +369,12 @@ final class ApplicationDispatcher implem
hrequest.getPathInfo());
wrequest.setAttribute(RequestDispatcher.FORWARD_QUERY_STRING,
hrequest.getQueryString());
- Mapping mapping;
+ ServletMapping mapping;
if (hrequest instanceof org.apache.catalina.servlet4preview.http.HttpServletRequest) {
mapping = ((org.apache.catalina.servlet4preview.http.HttpServletRequest)
- hrequest).getMapping();
+ hrequest).getServletMapping();
} else {
- mapping = (new ApplicationMapping(null)).getMapping();
+ mapping = (new ApplicationMapping(null)).getServletMapping();
}
wrequest.setAttribute(
org.apache.catalina.servlet4preview.RequestDispatcher.FORWARD_MAPPING,
Modified: tomcat/tc8.5.x/trunk/java/org/apache/catalina/core/ApplicationHttpRequest.java
URL: http://svn.apache.org/viewvc/tomcat/tc8.5.x/trunk/java/org/apache/catalina/core/ApplicationHttpRequest.java?rev=1784599&r1=1784598&r2=1784599&view=diff
==============================================================================
--- tomcat/tc8.5.x/trunk/java/org/apache/catalina/core/ApplicationHttpRequest.java (original)
+++ tomcat/tc8.5.x/trunk/java/org/apache/catalina/core/ApplicationHttpRequest.java Mon Feb 27 17:05:17 2017
@@ -37,8 +37,8 @@ import org.apache.catalina.Context;
import org.apache.catalina.Globals;
import org.apache.catalina.Manager;
import org.apache.catalina.Session;
-import org.apache.catalina.servlet4preview.http.Mapping;
import org.apache.catalina.servlet4preview.http.PushBuilder;
+import org.apache.catalina.servlet4preview.http.ServletMapping;
import org.apache.catalina.util.ParameterMap;
import org.apache.tomcat.util.buf.B2CConverter;
import org.apache.tomcat.util.buf.MessageBytes;
@@ -189,7 +189,7 @@ class ApplicationHttpRequest
/**
* The mapping for this request.
*/
- private Mapping mapping = null;
+ private ServletMapping mapping = null;
/**
@@ -523,7 +523,7 @@ class ApplicationHttpRequest
@Override
- public Mapping getMapping() {
+ public ServletMapping getServletMapping() {
return mapping;
}
@@ -707,9 +707,9 @@ class ApplicationHttpRequest
requestURI = request.getRequestURI();
servletPath = request.getServletPath();
if (request instanceof org.apache.catalina.servlet4preview.http.HttpServletRequest) {
- mapping = ((org.apache.catalina.servlet4preview.http.HttpServletRequest) request).getMapping();
+ mapping = ((org.apache.catalina.servlet4preview.http.HttpServletRequest) request).getServletMapping();
} else {
- mapping = (new ApplicationMapping(null)).getMapping();
+ mapping = (new ApplicationMapping(null)).getServletMapping();
}
}
@@ -769,7 +769,7 @@ class ApplicationHttpRequest
}
- void setMapping(Mapping mapping) {
+ void setMapping(ServletMapping mapping) {
this.mapping = mapping;
}
Modified: tomcat/tc8.5.x/trunk/java/org/apache/catalina/core/ApplicationMapping.java
URL: http://svn.apache.org/viewvc/tomcat/tc8.5.x/trunk/java/org/apache/catalina/core/ApplicationMapping.java?rev=1784599&r1=1784598&r2=1784599&view=diff
==============================================================================
--- tomcat/tc8.5.x/trunk/java/org/apache/catalina/core/ApplicationMapping.java (original)
+++ tomcat/tc8.5.x/trunk/java/org/apache/catalina/core/ApplicationMapping.java Mon Feb 27 17:05:17 2017
@@ -17,20 +17,20 @@
package org.apache.catalina.core;
import org.apache.catalina.mapper.MappingData;
-import org.apache.catalina.servlet4preview.http.Mapping;
import org.apache.catalina.servlet4preview.http.MappingMatch;
+import org.apache.catalina.servlet4preview.http.ServletMapping;
public class ApplicationMapping {
private final MappingData mappingData;
- private volatile Mapping mapping = null;
+ private volatile ServletMapping mapping = null;
public ApplicationMapping(MappingData mappingData) {
this.mappingData = mappingData;
}
- public Mapping getMapping() {
+ public ServletMapping getServletMapping() {
if (mapping == null) {
if (mappingData == null) {
mapping = new MappingImpl("", "", MappingMatch.UNKNOWN, "");
@@ -77,7 +77,7 @@ public class ApplicationMapping {
mapping = null;
}
- private static class MappingImpl implements Mapping {
+ private static class MappingImpl implements ServletMapping {
private final String matchValue;
private final String pattern;
Modified: tomcat/tc8.5.x/trunk/java/org/apache/catalina/servlet4preview/http/HttpServletRequest.java
URL: http://svn.apache.org/viewvc/tomcat/tc8.5.x/trunk/java/org/apache/catalina/servlet4preview/http/HttpServletRequest.java?rev=1784599&r1=1784598&r2=1784599&view=diff
==============================================================================
--- tomcat/tc8.5.x/trunk/java/org/apache/catalina/servlet4preview/http/HttpServletRequest.java (original)
+++ tomcat/tc8.5.x/trunk/java/org/apache/catalina/servlet4preview/http/HttpServletRequest.java Mon Feb 27 17:05:17 2017
@@ -21,7 +21,7 @@ package org.apache.catalina.servlet4prev
*/
public interface HttpServletRequest extends javax.servlet.http.HttpServletRequest {
- public Mapping getMapping();
+ public ServletMapping getServletMapping();
/**
* Does the current request allow push requests. This will return {@code
Modified: tomcat/tc8.5.x/trunk/java/org/apache/catalina/servlet4preview/http/HttpServletRequestWrapper.java
URL: http://svn.apache.org/viewvc/tomcat/tc8.5.x/trunk/java/org/apache/catalina/servlet4preview/http/HttpServletRequestWrapper.java?rev=1784599&r1=1784598&r2=1784599&view=diff
==============================================================================
--- tomcat/tc8.5.x/trunk/java/org/apache/catalina/servlet4preview/http/HttpServletRequestWrapper.java (original)
+++ tomcat/tc8.5.x/trunk/java/org/apache/catalina/servlet4preview/http/HttpServletRequestWrapper.java Mon Feb 27 17:05:17 2017
@@ -43,13 +43,13 @@ public class HttpServletRequestWrapper e
* {@inheritDoc}
* <p>
* The default behavior of this method is to return
- * {@link HttpServletRequest#getMapping()} on the wrapped request object.
+ * {@link HttpServletRequest#getServletMapping()} on the wrapped request object.
*
* @since Servlet 4.0
*/
@Override
- public Mapping getMapping() {
- return this._getHttpServletRequest().getMapping();
+ public ServletMapping getServletMapping() {
+ return this._getHttpServletRequest().getServletMapping();
}
/**
Copied: tomcat/tc8.5.x/trunk/java/org/apache/catalina/servlet4preview/http/ServletMapping.java (from r1784598, tomcat/tc8.5.x/trunk/java/org/apache/catalina/servlet4preview/http/Mapping.java)
URL: http://svn.apache.org/viewvc/tomcat/tc8.5.x/trunk/java/org/apache/catalina/servlet4preview/http/ServletMapping.java?p2=tomcat/tc8.5.x/trunk/java/org/apache/catalina/servlet4preview/http/ServletMapping.java&p1=tomcat/tc8.5.x/trunk/java/org/apache/catalina/servlet4preview/http/Mapping.java&r1=1784598&r2=1784599&rev=1784599&view=diff
==============================================================================
--- tomcat/tc8.5.x/trunk/java/org/apache/catalina/servlet4preview/http/Mapping.java (original)
+++ tomcat/tc8.5.x/trunk/java/org/apache/catalina/servlet4preview/http/ServletMapping.java Mon Feb 27 17:05:17 2017
@@ -22,7 +22,7 @@ package org.apache.catalina.servlet4prev
*
* @since 4.0
*/
-public interface Mapping {
+public interface ServletMapping {
/**
* @return The value that was matched or the empty String if not known.
Modified: tomcat/tc8.5.x/trunk/test/org/apache/catalina/core/TestApplicationMapping.java
URL: http://svn.apache.org/viewvc/tomcat/tc8.5.x/trunk/test/org/apache/catalina/core/TestApplicationMapping.java?rev=1784599&r1=1784598&r2=1784599&view=diff
==============================================================================
--- tomcat/tc8.5.x/trunk/test/org/apache/catalina/core/TestApplicationMapping.java (original)
+++ tomcat/tc8.5.x/trunk/test/org/apache/catalina/core/TestApplicationMapping.java Mon Feb 27 17:05:17 2017
@@ -29,7 +29,7 @@ import org.junit.Assert;
import org.junit.Test;
import org.apache.catalina.Context;
-import org.apache.catalina.servlet4preview.http.Mapping;
+import org.apache.catalina.servlet4preview.http.ServletMapping;
import org.apache.catalina.startup.Tomcat;
import org.apache.catalina.startup.TomcatBaseTest;
import org.apache.tomcat.util.buf.ByteChunk;
@@ -284,13 +284,13 @@ public class TestApplicationMapping exte
throws ServletException, IOException {
resp.setContentType("text/plain;charset=UTF-8");
PrintWriter pw = resp.getWriter();
- Mapping mapping = ((org.apache.catalina.servlet4preview.http.HttpServletRequest)
- req).getMapping();
+ ServletMapping mapping = ((org.apache.catalina.servlet4preview.http.HttpServletRequest)
+ req).getServletMapping();
pw.println("MatchValue=[" + mapping.getMatchValue() + "]");
pw.println("Pattern=[" + mapping.getPattern() + "]");
pw.println("MatchType=[" + mapping.getMappingMatch() + "]");
pw.println("ServletName=[" + mapping.getServletName() + "]");
- Mapping includeMapping = (Mapping) req.getAttribute(
+ ServletMapping includeMapping = (ServletMapping) req.getAttribute(
org.apache.catalina.servlet4preview.RequestDispatcher.INCLUDE_MAPPING);
if (includeMapping != null) {
pw.println("IncludeMatchValue=[" + includeMapping.getMatchValue() + "]");
@@ -299,7 +299,7 @@ public class TestApplicationMapping exte
pw.println("IncludeServletName=[" + includeMapping.getServletName() + "]");
}
- Mapping forwardMapping = (Mapping) req.getAttribute(
+ ServletMapping forwardMapping = (ServletMapping) req.getAttribute(
org.apache.catalina.servlet4preview.RequestDispatcher.FORWARD_MAPPING);
if (forwardMapping != null) {
pw.println("ForwardMatchValue=[" + forwardMapping.getMatchValue() + "]");
Modified: tomcat/tc8.5.x/trunk/webapps/docs/changelog.xml
URL: http://svn.apache.org/viewvc/tomcat/tc8.5.x/trunk/webapps/docs/changelog.xml?rev=1784599&r1=1784598&r2=1784599&view=diff
==============================================================================
--- tomcat/tc8.5.x/trunk/webapps/docs/changelog.xml (original)
+++ tomcat/tc8.5.x/trunk/webapps/docs/changelog.xml Mon Feb 27 17:05:17 2017
@@ -126,6 +126,13 @@
character in <code>jar:file:...</code> URLs. The default separator of
<code>*</code> remains unchanged. (markt)
</fix>
+ <update>
+ Update the <code>org.apache.catalina.servlet4preview</code> package that
+ can be used to gain early access to Servlet 4.0 features to align with
+ the latest proposals from the Servlet 4.0 expert group. Note that the
+ Servlet 4.0 API is still a work in progress and further changes are
+ likely. (markt)
+ </update>
</changelog>
</subsection>
<subsection name="Coyote">
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@tomcat.apache.org
For additional commands, e-mail: dev-help@tomcat.apache.org