diff options
author | Jan Bartel | 2012-09-07 03:46:51 +0000 |
---|---|---|
committer | Jan Bartel | 2012-10-27 06:38:43 +0000 |
commit | 958d0c6949b7d3d188a6eef51318f099f0928cad (patch) | |
tree | dd8063ae6d68b17c7c6ed2ff941f49441c833390 | |
parent | 605b0360e18b4b99341a207a22f8b547989af8ae (diff) | |
download | org.eclipse.jetty.project-958d0c6949b7d3d188a6eef51318f099f0928cad.tar.gz org.eclipse.jetty.project-958d0c6949b7d3d188a6eef51318f099f0928cad.tar.xz org.eclipse.jetty.project-958d0c6949b7d3d188a6eef51318f099f0928cad.zip |
Start incorporating (EDR version) servlet 3.1 spec
29 files changed, 178 insertions, 34 deletions
diff --git a/examples/async-rest/async-rest-jar/pom.xml b/examples/async-rest/async-rest-jar/pom.xml index 919bcd955d..22e2936844 100644 --- a/examples/async-rest/async-rest-jar/pom.xml +++ b/examples/async-rest/async-rest-jar/pom.xml @@ -21,8 +21,8 @@ <version>${project.version}</version> </dependency> <dependency> - <groupId>org.eclipse.jetty.orbit</groupId> - <artifactId>javax.servlet</artifactId> + <groupId>javax.servlet</groupId> + <artifactId>javax.servlet-api</artifactId> <scope>provided</scope> </dependency> </dependencies> diff --git a/examples/async-rest/async-rest-webapp/pom.xml b/examples/async-rest/async-rest-webapp/pom.xml index 4544b19ade..3f100f2944 100644 --- a/examples/async-rest/async-rest-webapp/pom.xml +++ b/examples/async-rest/async-rest-webapp/pom.xml @@ -25,8 +25,8 @@ <scope>test</scope> </dependency> <dependency> - <groupId>org.eclipse.jetty.orbit</groupId> - <artifactId>javax.servlet</artifactId> + <groupId>javax.servlet</groupId> + <artifactId>javax.servlet-api</artifactId> <scope>provided</scope> </dependency> </dependencies> diff --git a/jetty-continuation/pom.xml b/jetty-continuation/pom.xml index 55df520021..ee13619a11 100644 --- a/jetty-continuation/pom.xml +++ b/jetty-continuation/pom.xml @@ -53,9 +53,9 @@ </build> <dependencies> <dependency> - <groupId>org.eclipse.jetty.orbit</groupId> - <artifactId>javax.servlet</artifactId> + <groupId>javax.servlet</groupId> + <artifactId>javax.servlet-api</artifactId> <scope>provided</scope> - </dependency> + </dependency> </dependencies> </project> diff --git a/jetty-osgi/jetty-osgi-boot-jsp/pom.xml b/jetty-osgi/jetty-osgi-boot-jsp/pom.xml index 9cf70d9efe..04136a5c49 100644 --- a/jetty-osgi/jetty-osgi-boot-jsp/pom.xml +++ b/jetty-osgi/jetty-osgi-boot-jsp/pom.xml @@ -33,8 +33,8 @@ </dependency> <!-- Orbit Servlet Deps --> <dependency> - <groupId>org.eclipse.jetty.orbit</groupId> - <artifactId>javax.servlet</artifactId> + <groupId>javax.servlet</groupId> + <artifactId>javax.servlet-api</artifactId> </dependency> <!-- Orbit JSP Deps --> <dependency> diff --git a/jetty-osgi/jetty-osgi-httpservice/pom.xml b/jetty-osgi/jetty-osgi-httpservice/pom.xml index d308410fc8..fd6b170b6b 100644 --- a/jetty-osgi/jetty-osgi-httpservice/pom.xml +++ b/jetty-osgi/jetty-osgi-httpservice/pom.xml @@ -30,8 +30,8 @@ <artifactId>org.eclipse.osgi</artifactId> </dependency> <dependency> - <groupId>org.eclipse.jetty.orbit</groupId> - <artifactId>javax.servlet</artifactId> + <groupId>javax.servlet</groupId> + <artifactId>javax.servlet-api</artifactId> </dependency> </dependencies> diff --git a/jetty-osgi/pom.xml b/jetty-osgi/pom.xml index 24d1b7c9dd..53327a7004 100644 --- a/jetty-osgi/pom.xml +++ b/jetty-osgi/pom.xml @@ -23,7 +23,9 @@ <module>jetty-osgi-boot-jsp</module> <module>jetty-osgi-boot-warurl</module> <module>jetty-osgi-httpservice</module> +<!-- <module>test-jetty-osgi</module> +--> </modules> <build> <resources> diff --git a/jetty-osgi/test-jetty-osgi/pom.xml b/jetty-osgi/test-jetty-osgi/pom.xml index 63d9c59f9b..c0f3fc2d83 100644 --- a/jetty-osgi/test-jetty-osgi/pom.xml +++ b/jetty-osgi/test-jetty-osgi/pom.xml @@ -154,8 +154,8 @@ </dependency> <!-- Orbit Servlet Deps --> <dependency> - <groupId>org.eclipse.jetty.orbit</groupId> - <artifactId>javax.servlet</artifactId> + <groupId>javax.servlet</groupId> + <artifactId>javax.servlet-api</artifactId> <scope>test</scope> </dependency> <!-- Orbit JSP Deps --> @@ -384,4 +384,4 @@ </plugins> </pluginManagement> </build> -</project>
\ No newline at end of file +</project> diff --git a/jetty-proxy/pom.xml b/jetty-proxy/pom.xml index a2ebe53282..c5bcf1d13d 100644 --- a/jetty-proxy/pom.xml +++ b/jetty-proxy/pom.xml @@ -80,8 +80,8 @@ <version>${project.version}</version> </dependency> <dependency> - <groupId>org.eclipse.jetty.orbit</groupId> - <artifactId>javax.servlet</artifactId> + <groupId>javax.servlet</groupId> + <artifactId>javax.servlet-api</artifactId> <scope>provided</scope> </dependency> </dependencies> diff --git a/jetty-rewrite/pom.xml b/jetty-rewrite/pom.xml index 69914e4d10..7643069a4c 100644 --- a/jetty-rewrite/pom.xml +++ b/jetty-rewrite/pom.xml @@ -85,8 +85,8 @@ <version>${project.version}</version> </dependency> <dependency> - <groupId>org.eclipse.jetty.orbit</groupId> - <artifactId>javax.servlet</artifactId> + <groupId>javax.servlet</groupId> + <artifactId>javax.servlet-api</artifactId> </dependency> </dependencies> </project> diff --git a/jetty-rhttp/jetty-rhttp-connector/pom.xml b/jetty-rhttp/jetty-rhttp-connector/pom.xml index 95b8d9c8ad..b248c48210 100644 --- a/jetty-rhttp/jetty-rhttp-connector/pom.xml +++ b/jetty-rhttp/jetty-rhttp-connector/pom.xml @@ -88,7 +88,7 @@ </dependency> <dependency> <groupId>javax.servlet</groupId> - <artifactId>servlet-api</artifactId> + <artifactId>javax.servlet-api</artifactId> <scope>test</scope> </dependency> </dependencies> diff --git a/jetty-rhttp/jetty-rhttp-gateway/pom.xml b/jetty-rhttp/jetty-rhttp-gateway/pom.xml index 1243cd83e2..054bbd55bf 100644 --- a/jetty-rhttp/jetty-rhttp-gateway/pom.xml +++ b/jetty-rhttp/jetty-rhttp-gateway/pom.xml @@ -66,7 +66,7 @@ </dependency> <dependency> <groupId>javax.servlet</groupId> - <artifactId>servlet-api</artifactId> + <artifactId>javax.servlet-api</artifactId> </dependency> <dependency> <groupId>org.eclipse.jetty</groupId> diff --git a/jetty-security/src/main/java/org/eclipse/jetty/security/authentication/DeferredAuthentication.java b/jetty-security/src/main/java/org/eclipse/jetty/security/authentication/DeferredAuthentication.java index 04fd15afcc..cf2bfdc148 100644 --- a/jetty-security/src/main/java/org/eclipse/jetty/security/authentication/DeferredAuthentication.java +++ b/jetty-security/src/main/java/org/eclipse/jetty/security/authentication/DeferredAuthentication.java @@ -27,6 +27,7 @@ import java.util.Locale; import javax.servlet.ServletOutputStream; import javax.servlet.ServletRequest; import javax.servlet.ServletResponse; +import javax.servlet.WriteListener; import javax.servlet.http.Cookie; import javax.servlet.http.HttpServletRequest; import javax.servlet.http.HttpServletResponse; @@ -311,6 +312,11 @@ public class DeferredAuthentication implements Authentication.Deferred public void setContentLength(int len) { } + + public void setContentLengthLong(long len) + { + + } @Override public void setContentType(String type) @@ -346,6 +352,7 @@ public class DeferredAuthentication implements Authentication.Deferred return 0; } + }; /* ------------------------------------------------------------ */ @@ -364,6 +371,16 @@ public class DeferredAuthentication implements Authentication.Deferred public void println(String s) throws IOException { } + + public boolean canWrite() + { + return false; + } + + public void setWriteListener(WriteListener writeListener) + { + + } }; diff --git a/jetty-server/pom.xml b/jetty-server/pom.xml index f24e7daad2..0dcd7c8afd 100644 --- a/jetty-server/pom.xml +++ b/jetty-server/pom.xml @@ -88,8 +88,12 @@ <scope>test</scope> </dependency> <dependency> + <groupId>javax.servlet</groupId> + <artifactId>javax.servlet-api</artifactId> +<!-- <groupId>org.eclipse.jetty.orbit</groupId> <artifactId>javax.servlet</artifactId> +--> </dependency> <dependency> <groupId>org.eclipse.jetty</groupId> diff --git a/jetty-server/src/main/java/org/eclipse/jetty/server/ByteBufferHttpInput.java b/jetty-server/src/main/java/org/eclipse/jetty/server/ByteBufferHttpInput.java index 8d81c83c7b..987063016e 100644 --- a/jetty-server/src/main/java/org/eclipse/jetty/server/ByteBufferHttpInput.java +++ b/jetty-server/src/main/java/org/eclipse/jetty/server/ByteBufferHttpInput.java @@ -20,6 +20,8 @@ package org.eclipse.jetty.server; import java.nio.ByteBuffer; +import javax.servlet.ReadListener; + /** * <p>An implementation of HttpInput using {@link ByteBuffer} as items.</p> */ diff --git a/jetty-server/src/main/java/org/eclipse/jetty/server/HttpInput.java b/jetty-server/src/main/java/org/eclipse/jetty/server/HttpInput.java index 5484a62253..446135d11b 100644 --- a/jetty-server/src/main/java/org/eclipse/jetty/server/HttpInput.java +++ b/jetty-server/src/main/java/org/eclipse/jetty/server/HttpInput.java @@ -22,6 +22,7 @@ import java.io.IOException; import java.io.InterruptedIOException; import javax.servlet.ServletInputStream; +import javax.servlet.ReadListener; import org.eclipse.jetty.io.EofException; import org.eclipse.jetty.io.RuntimeIOException; @@ -243,4 +244,25 @@ public abstract class HttpInput<T> extends ServletInputStream } } } + + @Override + public boolean isFinished() + { + // TODO Auto-generated method stub + return false; + } + + @Override + public boolean isReady() + { + // TODO Auto-generated method stub + return false; + } + + @Override + public void setReadListener(ReadListener readListener) + { + // TODO Auto-generated method stub + + } } diff --git a/jetty-server/src/main/java/org/eclipse/jetty/server/HttpOutput.java b/jetty-server/src/main/java/org/eclipse/jetty/server/HttpOutput.java index 05a80f1282..101bb91f2f 100644 --- a/jetty-server/src/main/java/org/eclipse/jetty/server/HttpOutput.java +++ b/jetty-server/src/main/java/org/eclipse/jetty/server/HttpOutput.java @@ -28,6 +28,7 @@ import javax.servlet.RequestDispatcher; import javax.servlet.ServletOutputStream; import javax.servlet.ServletRequest; import javax.servlet.ServletResponse; +import javax.servlet.WriteListener; import org.eclipse.jetty.http.HttpContent; import org.eclipse.jetty.http.HttpHeader; @@ -55,6 +56,8 @@ public class HttpOutput extends ServletOutputStream private long _written; private ByteBuffer _aggregate; private int _bufferSize; + private WriteListener _writeListener; + public HttpOutput(HttpChannel<?> channel) { @@ -324,4 +327,24 @@ public class HttpOutput extends ServletOutputStream if (BufferUtil.hasContent(_aggregate)) BufferUtil.clear(_aggregate); } + + @Override + public boolean canWrite() + { + // TODO Auto-generated method stub + return false; + } + + @Override + public void setWriteListener(WriteListener writeListener) + { + _writeListener = writeListener; + + // TODO implement behaviour + + /* WriteListener.onWritePossible() will be called IFF (if and only if) canWrite has been + called AND has returned false AND a writeListener has previously been + set. + */ + } } diff --git a/jetty-server/src/main/java/org/eclipse/jetty/server/Request.java b/jetty-server/src/main/java/org/eclipse/jetty/server/Request.java index 583c7c81b1..df82e9f9c5 100644 --- a/jetty-server/src/main/java/org/eclipse/jetty/server/Request.java +++ b/jetty-server/src/main/java/org/eclipse/jetty/server/Request.java @@ -57,6 +57,7 @@ import javax.servlet.http.HttpServletRequest; import javax.servlet.http.HttpServletResponse; import javax.servlet.http.HttpSession; import javax.servlet.http.Part; +import javax.servlet.http.ProtocolHandler; import org.eclipse.jetty.http.HttpCookie; import org.eclipse.jetty.http.HttpFields; @@ -444,6 +445,16 @@ public class Request implements HttpServletRequest { return (int)_fields.getLongField(HttpHeader.CONTENT_LENGTH.toString()); } + + /* ------------------------------------------------------------ */ + /* + * @see javax.servlet.ServletRequest.getContentLengthLong() + */ + @Override + public long getContentLengthLong() + { + return _fields.getLongField(HttpHeader.CONTENT_LENGTH.toString()); + } /* ------------------------------------------------------------ */ /* @@ -2143,4 +2154,12 @@ public class Request implements HttpServletRequest setParameters(parameters); setQueryString(query); } + + + /* ------------------------------------------------------------ */ + @Override + public void upgrade(ProtocolHandler arg0) throws IOException + { + // TODO Auto-generated method stub + } } diff --git a/jetty-server/src/main/java/org/eclipse/jetty/server/Response.java b/jetty-server/src/main/java/org/eclipse/jetty/server/Response.java index 99da268d1d..8400255b42 100644 --- a/jetty-server/src/main/java/org/eclipse/jetty/server/Response.java +++ b/jetty-server/src/main/java/org/eclipse/jetty/server/Response.java @@ -440,6 +440,7 @@ public class Response implements HttpServletResponse if (!URIUtil.hasScheme(location)) { StringBuilder buf = _channel.getRequest().getRootURL(); + if (location.startsWith("/")) buf.append(location); else @@ -758,6 +759,12 @@ public class Response implements HttpServletResponse _contentLength = len; _fields.putLongField(HttpHeader.CONTENT_LENGTH.toString(), len); } + + @Override + public void setContentLengthLong(long length) + { + setLongContentLength(length); + } @Override public void setCharacterEncoding(String encoding) @@ -1016,4 +1023,6 @@ public class Response implements HttpServletResponse { return String.format("%s %d %s%n%s", _channel.getRequest().getHttpVersion(), _status, _reason == null ? "" : _reason, _fields); } + + } diff --git a/jetty-server/src/main/java/org/eclipse/jetty/server/ServletRequestHttpWrapper.java b/jetty-server/src/main/java/org/eclipse/jetty/server/ServletRequestHttpWrapper.java index 242a27b441..5341fb4be6 100644 --- a/jetty-server/src/main/java/org/eclipse/jetty/server/ServletRequestHttpWrapper.java +++ b/jetty-server/src/main/java/org/eclipse/jetty/server/ServletRequestHttpWrapper.java @@ -32,9 +32,13 @@ import javax.servlet.http.HttpServletRequest; import javax.servlet.http.HttpServletResponse; import javax.servlet.http.HttpSession; import javax.servlet.http.Part; +import javax.servlet.http.ProtocolHandler; /* ------------------------------------------------------------ */ -/** Class to tunnel a ServletRequest via a HttpServletRequest +/** + * ServletRequestHttpWrapper + * + * Class to tunnel a ServletRequest via a HttpServletRequest */ public class ServletRequestHttpWrapper extends ServletRequestWrapper implements HttpServletRequest { @@ -208,5 +212,13 @@ public class ServletRequestHttpWrapper extends ServletRequestWrapper implements } + + @Override + public void upgrade(ProtocolHandler handler) throws IOException + { + // TODO Auto-generated method stub + + } + } diff --git a/jetty-server/src/main/java/org/eclipse/jetty/server/ServletResponseHttpWrapper.java b/jetty-server/src/main/java/org/eclipse/jetty/server/ServletResponseHttpWrapper.java index 91278460b7..3e3263442b 100644 --- a/jetty-server/src/main/java/org/eclipse/jetty/server/ServletResponseHttpWrapper.java +++ b/jetty-server/src/main/java/org/eclipse/jetty/server/ServletResponseHttpWrapper.java @@ -28,7 +28,10 @@ import javax.servlet.http.HttpServletResponse; /* ------------------------------------------------------------ */ -/** Wrapper to tunnel a ServletResponse via a HttpServletResponse +/** + * ServletResponseHttpWrapper + * + * Wrapper to tunnel a ServletResponse via a HttpServletResponse */ public class ServletResponseHttpWrapper extends ServletResponseWrapper implements HttpServletResponse { diff --git a/jetty-servlets/pom.xml b/jetty-servlets/pom.xml index d71af33fd2..9ff8175719 100644 --- a/jetty-servlets/pom.xml +++ b/jetty-servlets/pom.xml @@ -75,8 +75,8 @@ <version>${project.version}</version> </dependency> <dependency> - <groupId>org.eclipse.jetty.orbit</groupId> - <artifactId>javax.servlet</artifactId> + <groupId>javax.servlet</groupId> + <artifactId>javax.servlet-api</artifactId> <scope>provided</scope> </dependency> </dependencies> diff --git a/jetty-servlets/src/main/java/org/eclipse/jetty/servlets/gzip/AbstractCompressedStream.java b/jetty-servlets/src/main/java/org/eclipse/jetty/servlets/gzip/AbstractCompressedStream.java index 524cf6022d..477c258cd6 100644 --- a/jetty-servlets/src/main/java/org/eclipse/jetty/servlets/gzip/AbstractCompressedStream.java +++ b/jetty-servlets/src/main/java/org/eclipse/jetty/servlets/gzip/AbstractCompressedStream.java @@ -25,6 +25,7 @@ import java.io.PrintWriter; import java.io.UnsupportedEncodingException; import java.util.zip.DeflaterOutputStream; import javax.servlet.ServletOutputStream; +import javax.servlet.WriteListener; import javax.servlet.http.HttpServletRequest; import javax.servlet.http.HttpServletResponse; @@ -337,6 +338,20 @@ public abstract class AbstractCompressedStream extends ServletOutputStream _response.setHeader(name, value); } + @Override + public boolean canWrite() + { + // TODO Auto-generated method stub + return false; + } + + @Override + public void setWriteListener(WriteListener writeListener) + { + // TODO Auto-generated method stub + + } + /** * Create the stream fitting to the underlying compression type. * diff --git a/jetty-util-ajax/pom.xml b/jetty-util-ajax/pom.xml index c08c077e2c..48264bb072 100644 --- a/jetty-util-ajax/pom.xml +++ b/jetty-util-ajax/pom.xml @@ -77,8 +77,8 @@ <version>${project.version}</version> </dependency> <dependency> - <groupId>org.eclipse.jetty.orbit</groupId> - <artifactId>javax.servlet</artifactId> + <groupId>javax.servlet</groupId> + <artifactId>javax.servlet-api</artifactId> <scope>provided</scope> </dependency> <dependency> diff --git a/jetty-util/pom.xml b/jetty-util/pom.xml index 4099da99d6..0aa14a259e 100644 --- a/jetty-util/pom.xml +++ b/jetty-util/pom.xml @@ -70,8 +70,8 @@ </build> <dependencies> <dependency> - <groupId>org.eclipse.jetty.orbit</groupId> - <artifactId>javax.servlet</artifactId> + <groupId>javax.servlet</groupId> + <artifactId>javax.servlet-api</artifactId> <scope>provided</scope> </dependency> <dependency> diff --git a/jetty-websocket/websocket-server/pom.xml b/jetty-websocket/websocket-server/pom.xml index ce0893b159..02812c5941 100644 --- a/jetty-websocket/websocket-server/pom.xml +++ b/jetty-websocket/websocket-server/pom.xml @@ -22,8 +22,8 @@ <scope>compile</scope> </dependency> <dependency> - <groupId>org.eclipse.jetty.orbit</groupId> - <artifactId>javax.servlet</artifactId> + <groupId>javax.servlet</groupId> + <artifactId>javax.servlet-api</artifactId> <scope>provided</scope> </dependency> <dependency> @@ -147,7 +147,9 @@ <!-- Banned Dependencies (should use Orbit based versions now) --> <bannedDependencies> <excludes> +<!-- <exclude>javax.servlet</exclude> +--> <exclude>javax.servlet.jsp</exclude> <exclude>org.apache.geronimo.specs</exclude> <exclude>javax.mail</exclude> @@ -156,8 +158,16 @@ <!-- allowed combinations --> <includes> <include>org.apache.geronimo.specs:geronimo-atinject_1.0_spec:jar:*</include> +<!-- <include>javax.servlet:*:*:*:provided</include> +--> <include>javax.servlet.jsp:*:*:*:provided</include> + <include>javax.servlet</include> +<!-- + <include>org.apache.geronimo.specs</include> + <include>javax.mail</include> + <include>javax.activation</include> +--> </includes> <searchTransitive>true</searchTransitive> <message>This dependency is banned, use the ORBIT provided dependency instead.</message> @@ -430,9 +440,14 @@ <dependencies> <!-- Orbit Deps --> <dependency> + <groupId>javax.servlet</groupId> + <artifactId>javax.servlet-api</artifactId> + <version>3.1-b01</version> +<!-- <groupId>org.eclipse.jetty.orbit</groupId> <artifactId>javax.servlet</artifactId> <version>3.0.0.v201112011016</version> +--> </dependency> <dependency> <groupId>org.eclipse.jetty.orbit</groupId> diff --git a/test-jetty-webapp/pom.xml b/test-jetty-webapp/pom.xml index df56c84db8..c887a0dfc1 100644 --- a/test-jetty-webapp/pom.xml +++ b/test-jetty-webapp/pom.xml @@ -163,8 +163,8 @@ <version>${project.version}</version> </dependency> <dependency> - <groupId>org.eclipse.jetty.orbit</groupId> - <artifactId>javax.servlet</artifactId> + <groupId>javax.servlet</groupId> + <artifactId>javax.servlet-api</artifactId> <scope>provided</scope> </dependency> <dependency> diff --git a/tests/test-sessions/test-mongodb-sessions/src/test/java/org/eclipse/jetty/nosql/mongodb/ServerCrossContextSessionTest.java b/tests/test-sessions/test-mongodb-sessions/src/test/java/org/eclipse/jetty/nosql/mongodb/ServerCrossContextSessionTest.java index 98cf84c34e..e8102f151b 100644 --- a/tests/test-sessions/test-mongodb-sessions/src/test/java/org/eclipse/jetty/nosql/mongodb/ServerCrossContextSessionTest.java +++ b/tests/test-sessions/test-mongodb-sessions/src/test/java/org/eclipse/jetty/nosql/mongodb/ServerCrossContextSessionTest.java @@ -23,6 +23,7 @@ import org.eclipse.jetty.server.session.AbstractTestServer; import org.junit.Ignore; import org.junit.Test; + public class ServerCrossContextSessionTest extends AbstractServerCrossContextSessionTest { public AbstractTestServer createServer(int port) diff --git a/tests/test-webapps/test-webapp-rfc2616/pom.xml b/tests/test-webapps/test-webapp-rfc2616/pom.xml index e1aa6438ce..29f6e0f2cd 100644 --- a/tests/test-webapps/test-webapp-rfc2616/pom.xml +++ b/tests/test-webapps/test-webapp-rfc2616/pom.xml @@ -45,8 +45,8 @@ <version>${project.version}</version> </dependency> <dependency> - <groupId>org.eclipse.jetty.orbit</groupId> - <artifactId>javax.servlet</artifactId> + <groupId>javax.servlet</groupId> + <artifactId>javax.servlet-api</artifactId> <scope>provided</scope> </dependency> </dependencies> |