diff --git a/jetty-distribution/pom.xml b/jetty-distribution/pom.xml index 3f4eafb3147..103312d7736 100644 --- a/jetty-distribution/pom.xml +++ b/jetty-distribution/pom.xml @@ -680,11 +680,13 @@ jetty-proxy ${project.version} + org.eclipse.jetty jetty-jaas diff --git a/jetty-proxy/src/main/java/org/eclipse/jetty/proxy/ProxyServlet.java b/jetty-proxy/src/main/java/org/eclipse/jetty/proxy/ProxyServlet.java index 3c6ab64263e..98a25f77add 100644 --- a/jetty-proxy/src/main/java/org/eclipse/jetty/proxy/ProxyServlet.java +++ b/jetty-proxy/src/main/java/org/eclipse/jetty/proxy/ProxyServlet.java @@ -258,6 +258,8 @@ public class ProxyServlet extends HttpServlet if (value == null || "-".equals(value)) { executor = (Executor)getServletContext().getAttribute("org.eclipse.jetty.server.Executor"); + if (executor==null) + throw new IllegalStateException("No server executor for proxy"); } else { diff --git a/jetty-proxy/src/test/java/org/eclipse/jetty/proxy/ProxyServletTest.java b/jetty-proxy/src/test/java/org/eclipse/jetty/proxy/ProxyServletTest.java index 7739fe8cf31..2d356edcde3 100644 --- a/jetty-proxy/src/test/java/org/eclipse/jetty/proxy/ProxyServletTest.java +++ b/jetty-proxy/src/test/java/org/eclipse/jetty/proxy/ProxyServletTest.java @@ -239,7 +239,7 @@ public class ProxyServletTest result.setProxyConfiguration(new ProxyConfiguration("localhost", proxyConnector.getLocalPort())); QueuedThreadPool threadPool = new QueuedThreadPool(); threadPool.setName("foo"); - threadPool.setMaxThreads(2); + threadPool.setMaxThreads(20); result.setExecutor(threadPool); result.start(); @@ -269,10 +269,9 @@ public class ProxyServletTest for ( int i = 0; i < 10; ++i ) { - - Assert.assertEquals(200, responses[i].getStatus()); - Assert.assertTrue(responses[i].getHeaders().containsKey(PROXIED_HEADER)); - Assert.assertArrayEquals(content, responses[i].getContent()); + Assert.assertEquals(200, responses[i].getStatus()); + Assert.assertTrue(responses[i].getHeaders().containsKey(PROXIED_HEADER)); + Assert.assertArrayEquals(content, responses[i].getContent()); } } diff --git a/jetty-spdy/spdy-http-server/src/test/java/org/eclipse/jetty/spdy/server/http/SSLExternalServerTest.java b/jetty-spdy/spdy-http-server/src/test/java/org/eclipse/jetty/spdy/server/http/SSLExternalServerTest.java index 6465b58f563..def73036ae6 100644 --- a/jetty-spdy/spdy-http-server/src/test/java/org/eclipse/jetty/spdy/server/http/SSLExternalServerTest.java +++ b/jetty-spdy/spdy-http-server/src/test/java/org/eclipse/jetty/spdy/server/http/SSLExternalServerTest.java @@ -38,6 +38,7 @@ import org.eclipse.jetty.util.Fields; import org.eclipse.jetty.util.ssl.SslContextFactory; import org.junit.Assert; import org.junit.Assume; +import org.junit.Ignore; import org.junit.Test; public class SSLExternalServerTest extends AbstractHTTPSPDYTest @@ -58,6 +59,7 @@ public class SSLExternalServerTest extends AbstractHTTPSPDYTest } @Test(timeout=5000) + @Ignore public void testExternalServer() throws Exception { String host = "encrypted.google.com"; diff --git a/pom.xml b/pom.xml index 5456315c53e..ffe31bfc0e9 100644 --- a/pom.xml +++ b/pom.xml @@ -442,7 +442,7 @@ - jetty-overlay-deployer +