Repository: knox
Updated Branches:
  refs/heads/master cf53d7749 -> d5d95e329


KNOX-1516 - Upgrade Jetty to 9.4.12

Signed-off-by: Kevin Risden <kris...@apache.org>


Project: http://git-wip-us.apache.org/repos/asf/knox/repo
Commit: http://git-wip-us.apache.org/repos/asf/knox/commit/d5d95e32
Tree: http://git-wip-us.apache.org/repos/asf/knox/tree/d5d95e32
Diff: http://git-wip-us.apache.org/repos/asf/knox/diff/d5d95e32

Branch: refs/heads/master
Commit: d5d95e329b2c2f81f87da793fee99c7a4febcd24
Parents: cf53d77
Author: Kevin Risden <kris...@apache.org>
Authored: Fri Oct 5 10:26:39 2018 -0400
Committer: Kevin Risden <kris...@apache.org>
Committed: Fri Nov 9 18:41:16 2018 -0500

----------------------------------------------------------------------
 .../pom.xml                                     | 13 ++++++
 .../function/UsernameFunctionProcessorTest.java |  7 +--
 .../pom.xml                                     | 13 ++++++
 .../impl/ServiceRegistryFunctionsTest.java      |  9 ++--
 gateway-provider-rewrite/pom.xml                | 12 ++++++
 .../UrlRewriteServletContextListenerTest.java   |  7 +--
 .../api/UrlRewriteServletFilterTest.java        |  7 +--
 .../impl/FrontendFunctionProcessorTest.java     |  9 ++--
 .../gateway/filter/RequestUpdateHandler.java    |  3 +-
 .../knox/gateway/trace/AccessHandler.java       |  2 +-
 gateway-test-release/pom.xml                    | 40 +++++++++++++++--
 gateway-test-utils/pom.xml                      | 12 +++++-
 .../knox/gateway/GatewayMultiFuncTest.java      |  6 +--
 .../SimpleDescriptorHandlerFuncTest.java        |  1 +
 pom.xml                                         | 45 +++++++++++++++++++-
 15 files changed, 157 insertions(+), 29 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/knox/blob/d5d95e32/gateway-provider-identity-assertion-common/pom.xml
----------------------------------------------------------------------
diff --git a/gateway-provider-identity-assertion-common/pom.xml 
b/gateway-provider-identity-assertion-common/pom.xml
index 946c550..65c6de5 100644
--- a/gateway-provider-identity-assertion-common/pom.xml
+++ b/gateway-provider-identity-assertion-common/pom.xml
@@ -89,6 +89,19 @@
             <scope>test</scope>
         </dependency>
         <dependency>
+            <groupId>org.eclipse.jetty</groupId>
+            <artifactId>jetty-servlet</artifactId>
+            <classifier>tests</classifier>
+            <scope>test</scope>
+        </dependency>
+        <dependency>
+            <groupId>org.eclipse.jetty</groupId>
+            <artifactId>jetty-http</artifactId>
+            <classifier>tests</classifier>
+            <scope>test</scope>
+        </dependency>
+
+        <dependency>
             <groupId>org.apache.velocity</groupId>
             <artifactId>velocity</artifactId>
             <scope>test</scope>

http://git-wip-us.apache.org/repos/asf/knox/blob/d5d95e32/gateway-provider-identity-assertion-common/src/test/java/org/apache/knox/gateway/identityasserter/function/UsernameFunctionProcessorTest.java
----------------------------------------------------------------------
diff --git 
a/gateway-provider-identity-assertion-common/src/test/java/org/apache/knox/gateway/identityasserter/function/UsernameFunctionProcessorTest.java
 
b/gateway-provider-identity-assertion-common/src/test/java/org/apache/knox/gateway/identityasserter/function/UsernameFunctionProcessorTest.java
index 75a00e5..8f81515 100644
--- 
a/gateway-provider-identity-assertion-common/src/test/java/org/apache/knox/gateway/identityasserter/function/UsernameFunctionProcessorTest.java
+++ 
b/gateway-provider-identity-assertion-common/src/test/java/org/apache/knox/gateway/identityasserter/function/UsernameFunctionProcessorTest.java
@@ -33,7 +33,6 @@ import org.eclipse.jetty.servlet.FilterHolder;
 import org.eclipse.jetty.servlet.ServletHolder;
 import org.eclipse.jetty.http.HttpTester;
 import org.eclipse.jetty.servlet.ServletTester;
-import org.eclipse.jetty.util.ArrayQueue;
 import org.eclipse.jetty.util.log.Log;
 import org.hamcrest.core.Is;
 import org.junit.After;
@@ -54,11 +53,13 @@ import java.net.URL;
 import java.nio.charset.StandardCharsets;
 import java.security.PrivilegedActionException;
 import java.security.PrivilegedExceptionAction;
+import java.util.ArrayDeque;
 import java.util.Arrays;
 import java.util.EnumSet;
 import java.util.HashMap;
 import java.util.Iterator;
 import java.util.Map;
+import java.util.Queue;
 import java.util.ServiceLoader;
 
 import static org.hamcrest.MatcherAssert.assertThat;
@@ -71,7 +72,7 @@ public class UsernameFunctionProcessorTest {
   private ServletTester server;
   private HttpTester.Request request;
   private HttpTester.Response response;
-  private ArrayQueue<MockInteraction> interactions;
+  private Queue<MockInteraction> interactions;
   private MockInteraction interaction;
 
   private static URL getTestResource( String name ) {
@@ -101,7 +102,7 @@ public class UsernameFunctionProcessorTest {
     }
     rewriteFilter.setFilter( new UrlRewriteServletFilter() );
 
-    interactions = new ArrayQueue<>();
+    interactions = new ArrayDeque<>();
 
     ServletHolder servlet = server.addServlet( MockServlet.class, "/" );
     servlet.setServlet( new MockServlet( "mock-servlet", interactions ) );

http://git-wip-us.apache.org/repos/asf/knox/blob/d5d95e32/gateway-provider-rewrite-func-service-registry/pom.xml
----------------------------------------------------------------------
diff --git a/gateway-provider-rewrite-func-service-registry/pom.xml 
b/gateway-provider-rewrite-func-service-registry/pom.xml
index 0216a82..c88f7f5 100644
--- a/gateway-provider-rewrite-func-service-registry/pom.xml
+++ b/gateway-provider-rewrite-func-service-registry/pom.xml
@@ -79,6 +79,19 @@
             <scope>test</scope>
         </dependency>
         <dependency>
+            <groupId>org.eclipse.jetty</groupId>
+            <artifactId>jetty-servlet</artifactId>
+            <classifier>tests</classifier>
+            <scope>test</scope>
+        </dependency>
+        <dependency>
+            <groupId>org.eclipse.jetty</groupId>
+            <artifactId>jetty-http</artifactId>
+            <classifier>tests</classifier>
+            <scope>test</scope>
+        </dependency>
+
+        <dependency>
             <groupId>org.apache.velocity</groupId>
             <artifactId>velocity</artifactId>
             <scope>test</scope>

http://git-wip-us.apache.org/repos/asf/knox/blob/d5d95e32/gateway-provider-rewrite-func-service-registry/src/test/java/org/apache/knox/gateway/svcregfunc/impl/ServiceRegistryFunctionsTest.java
----------------------------------------------------------------------
diff --git 
a/gateway-provider-rewrite-func-service-registry/src/test/java/org/apache/knox/gateway/svcregfunc/impl/ServiceRegistryFunctionsTest.java
 
b/gateway-provider-rewrite-func-service-registry/src/test/java/org/apache/knox/gateway/svcregfunc/impl/ServiceRegistryFunctionsTest.java
index f8e7560..ee1b9eb 100644
--- 
a/gateway-provider-rewrite-func-service-registry/src/test/java/org/apache/knox/gateway/svcregfunc/impl/ServiceRegistryFunctionsTest.java
+++ 
b/gateway-provider-rewrite-func-service-registry/src/test/java/org/apache/knox/gateway/svcregfunc/impl/ServiceRegistryFunctionsTest.java
@@ -33,7 +33,6 @@ import org.eclipse.jetty.http.HttpTester;
 import org.eclipse.jetty.servlet.FilterHolder;
 import org.eclipse.jetty.servlet.ServletHolder;
 import org.eclipse.jetty.servlet.ServletTester;
-import org.eclipse.jetty.util.ArrayQueue;
 import org.eclipse.jetty.util.log.Log;
 import org.hamcrest.core.Is;
 import org.junit.Test;
@@ -53,9 +52,11 @@ import java.net.URL;
 import java.nio.charset.StandardCharsets;
 import java.security.PrivilegedActionException;
 import java.security.PrivilegedExceptionAction;
+import java.util.ArrayDeque;
 import java.util.EnumSet;
 import java.util.HashMap;
 import java.util.Map;
+import java.util.Queue;
 
 import static org.hamcrest.MatcherAssert.assertThat;
 
@@ -65,7 +66,7 @@ public class ServiceRegistryFunctionsTest {
   private ServletTester server;
   private HttpTester.Request request;
   private HttpTester.Response response;
-  private ArrayQueue<MockInteraction> interactions;
+  private Queue<MockInteraction> interactions;
   private MockInteraction interaction;
 
   private static URL getTestResource( String name ) {
@@ -105,8 +106,8 @@ public class ServiceRegistryFunctionsTest {
       }
     }
     rewriteFilter.setFilter( new UrlRewriteServletFilter() );
-
-    interactions = new ArrayQueue<>();
+    
+    interactions = new ArrayDeque<>();
 
     ServletHolder servlet = server.addServlet( MockServlet.class, "/" );
     servlet.setServlet( new MockServlet( "mock-servlet", interactions ) );

http://git-wip-us.apache.org/repos/asf/knox/blob/d5d95e32/gateway-provider-rewrite/pom.xml
----------------------------------------------------------------------
diff --git a/gateway-provider-rewrite/pom.xml b/gateway-provider-rewrite/pom.xml
index 20aa382..f17a1f1 100644
--- a/gateway-provider-rewrite/pom.xml
+++ b/gateway-provider-rewrite/pom.xml
@@ -130,6 +130,18 @@
             <artifactId>gateway-test-utils</artifactId>
             <scope>test</scope>
         </dependency>
+        <dependency>
+            <groupId>org.eclipse.jetty</groupId>
+            <artifactId>jetty-servlet</artifactId>
+            <classifier>tests</classifier>
+            <scope>test</scope>
+        </dependency>
+        <dependency>
+            <groupId>org.eclipse.jetty</groupId>
+            <artifactId>jetty-http</artifactId>
+            <classifier>tests</classifier>
+            <scope>test</scope>
+        </dependency>
 
         <dependency>
             <groupId>org.eclipse.jetty</groupId>

http://git-wip-us.apache.org/repos/asf/knox/blob/d5d95e32/gateway-provider-rewrite/src/test/java/org/apache/knox/gateway/filter/rewrite/api/UrlRewriteServletContextListenerTest.java
----------------------------------------------------------------------
diff --git 
a/gateway-provider-rewrite/src/test/java/org/apache/knox/gateway/filter/rewrite/api/UrlRewriteServletContextListenerTest.java
 
b/gateway-provider-rewrite/src/test/java/org/apache/knox/gateway/filter/rewrite/api/UrlRewriteServletContextListenerTest.java
index 4223a46..0552b00 100644
--- 
a/gateway-provider-rewrite/src/test/java/org/apache/knox/gateway/filter/rewrite/api/UrlRewriteServletContextListenerTest.java
+++ 
b/gateway-provider-rewrite/src/test/java/org/apache/knox/gateway/filter/rewrite/api/UrlRewriteServletContextListenerTest.java
@@ -23,20 +23,21 @@ import org.eclipse.jetty.servlet.FilterHolder;
 import org.eclipse.jetty.servlet.ServletHolder;
 import org.eclipse.jetty.http.HttpTester;
 import org.eclipse.jetty.servlet.ServletTester;
-import org.eclipse.jetty.util.ArrayQueue;
 import org.junit.After;
 import org.junit.Before;
 
 import javax.servlet.DispatcherType;
 import java.net.URL;
+import java.util.ArrayDeque;
 import java.util.EnumSet;
+import java.util.Queue;
 
 public class UrlRewriteServletContextListenerTest {
 
   private ServletTester server;
   private HttpTester.Request request;
   private HttpTester.Response response;
-  private ArrayQueue<MockInteraction> interactions;
+  private Queue<MockInteraction> interactions;
   private MockInteraction interaction;
 
   private static URL getTestResource( String name ) {
@@ -58,7 +59,7 @@ public class UrlRewriteServletContextListenerTest {
     FilterHolder filter = server.addFilter( UrlRewriteServletFilter.class, 
"/*", EnumSet.of( DispatcherType.REQUEST ) );
     filter.setFilter( new UrlRewriteServletFilter() );
 
-    interactions = new ArrayQueue<>();
+    interactions = new ArrayDeque<>();
 
     ServletHolder servlet = server.addServlet( MockServlet.class, "/" );
     servlet.setServlet( new MockServlet( "mock-servlet", interactions ) );

http://git-wip-us.apache.org/repos/asf/knox/blob/d5d95e32/gateway-provider-rewrite/src/test/java/org/apache/knox/gateway/filter/rewrite/api/UrlRewriteServletFilterTest.java
----------------------------------------------------------------------
diff --git 
a/gateway-provider-rewrite/src/test/java/org/apache/knox/gateway/filter/rewrite/api/UrlRewriteServletFilterTest.java
 
b/gateway-provider-rewrite/src/test/java/org/apache/knox/gateway/filter/rewrite/api/UrlRewriteServletFilterTest.java
index f857dcf..ab2a9fc 100644
--- 
a/gateway-provider-rewrite/src/test/java/org/apache/knox/gateway/filter/rewrite/api/UrlRewriteServletFilterTest.java
+++ 
b/gateway-provider-rewrite/src/test/java/org/apache/knox/gateway/filter/rewrite/api/UrlRewriteServletFilterTest.java
@@ -30,7 +30,6 @@ import org.eclipse.jetty.http.HttpTester;
 import org.eclipse.jetty.servlet.FilterHolder;
 import org.eclipse.jetty.servlet.ServletHolder;
 import org.eclipse.jetty.servlet.ServletTester;
-import org.eclipse.jetty.util.ArrayQueue;
 import org.junit.After;
 import org.junit.Test;
 
@@ -46,10 +45,12 @@ import java.io.IOException;
 import java.net.URISyntaxException;
 import java.net.URL;
 import java.nio.charset.StandardCharsets;
+import java.util.ArrayDeque;
 import java.util.EnumSet;
 import java.util.Enumeration;
 import java.util.HashMap;
 import java.util.Map;
+import java.util.Queue;
 
 import static org.hamcrest.MatcherAssert.assertThat;
 import static org.hamcrest.Matchers.anyOf;
@@ -63,7 +64,7 @@ public class UrlRewriteServletFilterTest {
   private ServletTester server;
   private HttpTester.Request request;
   private HttpTester.Response response;
-  private ArrayQueue<MockInteraction> interactions;
+  private Queue<MockInteraction> interactions;
   private MockInteraction interaction;
 
   private static URL getTestResource( String name ) {
@@ -90,7 +91,7 @@ public class UrlRewriteServletFilterTest {
     }
     rewriteFilter.setFilter( new UrlRewriteServletFilter() );
 
-    interactions = new ArrayQueue<>();
+    interactions = new ArrayDeque<>();
 
     ServletHolder servlet = server.addServlet( MockServlet.class, "/" );
     servlet.setServlet( new MockServlet( "mock-servlet", interactions ) );

http://git-wip-us.apache.org/repos/asf/knox/blob/d5d95e32/gateway-provider-rewrite/src/test/java/org/apache/knox/gateway/filter/rewrite/impl/FrontendFunctionProcessorTest.java
----------------------------------------------------------------------
diff --git 
a/gateway-provider-rewrite/src/test/java/org/apache/knox/gateway/filter/rewrite/impl/FrontendFunctionProcessorTest.java
 
b/gateway-provider-rewrite/src/test/java/org/apache/knox/gateway/filter/rewrite/impl/FrontendFunctionProcessorTest.java
index de87d75..a0791fb 100644
--- 
a/gateway-provider-rewrite/src/test/java/org/apache/knox/gateway/filter/rewrite/impl/FrontendFunctionProcessorTest.java
+++ 
b/gateway-provider-rewrite/src/test/java/org/apache/knox/gateway/filter/rewrite/impl/FrontendFunctionProcessorTest.java
@@ -37,7 +37,6 @@ import org.eclipse.jetty.http.HttpTester;
 import org.eclipse.jetty.servlet.FilterHolder;
 import org.eclipse.jetty.servlet.ServletHolder;
 import org.eclipse.jetty.servlet.ServletTester;
-import org.eclipse.jetty.util.ArrayQueue;
 import org.eclipse.jetty.util.Attributes;
 import org.eclipse.jetty.util.AttributesMap;
 import org.eclipse.jetty.util.log.Log;
@@ -59,10 +58,12 @@ import java.net.URISyntaxException;
 import java.nio.charset.StandardCharsets;
 import java.security.PrivilegedActionException;
 import java.security.PrivilegedExceptionAction;
+import java.util.ArrayDeque;
 import java.util.EnumSet;
 import java.util.HashMap;
 import java.util.Iterator;
 import java.util.Map;
+import java.util.Queue;
 import java.util.ServiceLoader;
 
 import static org.hamcrest.CoreMatchers.containsString;
@@ -76,7 +77,7 @@ public class FrontendFunctionProcessorTest {
   private ServletTester server;
   private HttpTester.Request request;
   private HttpTester.Response response;
-  private ArrayQueue<MockInteraction> interactions;
+  private Queue<MockInteraction> interactions;
   private MockInteraction interaction;
 
   @SuppressWarnings("rawtypes")
@@ -157,8 +158,8 @@ public class FrontendFunctionProcessorTest {
       }
     }
     rewriteFilter.setFilter( new UrlRewriteServletFilter() );
-
-    interactions = new ArrayQueue<>();
+    
+    interactions = new ArrayDeque<>();
 
     ServletHolder servlet = server.addServlet( MockServlet.class, "/" );
     servlet.setServlet( new MockServlet( "mock-servlet", interactions ) );

http://git-wip-us.apache.org/repos/asf/knox/blob/d5d95e32/gateway-server/src/main/java/org/apache/knox/gateway/filter/RequestUpdateHandler.java
----------------------------------------------------------------------
diff --git 
a/gateway-server/src/main/java/org/apache/knox/gateway/filter/RequestUpdateHandler.java
 
b/gateway-server/src/main/java/org/apache/knox/gateway/filter/RequestUpdateHandler.java
index 61bd09d..342c988 100644
--- 
a/gateway-server/src/main/java/org/apache/knox/gateway/filter/RequestUpdateHandler.java
+++ 
b/gateway-server/src/main/java/org/apache/knox/gateway/filter/RequestUpdateHandler.java
@@ -87,8 +87,7 @@ public class RequestUpdateHandler extends ScopedHandler {
     // if the request already has the /{gatewaypath}/{topology} part then skip
     if (!StringUtils.startsWithIgnoreCase(target, redirectContext)) {
       baseRequest.setPathInfo(redirectContext + baseRequest.getPathInfo());
-      baseRequest.setUri(
-          new HttpURI(redirectContext + baseRequest.getUri().toString()));
+      baseRequest.setHttpURI(new HttpURI(redirectContext + 
baseRequest.getHttpURI()));
 
       LOG.topologyPortMappingUpdateRequest(target, newTarget);
       nextHandle(newTarget, baseRequest, newRequest, response);

http://git-wip-us.apache.org/repos/asf/knox/blob/d5d95e32/gateway-server/src/main/java/org/apache/knox/gateway/trace/AccessHandler.java
----------------------------------------------------------------------
diff --git 
a/gateway-server/src/main/java/org/apache/knox/gateway/trace/AccessHandler.java 
b/gateway-server/src/main/java/org/apache/knox/gateway/trace/AccessHandler.java
index 366e1b0..558a739 100644
--- 
a/gateway-server/src/main/java/org/apache/knox/gateway/trace/AccessHandler.java
+++ 
b/gateway-server/src/main/java/org/apache/knox/gateway/trace/AccessHandler.java
@@ -37,7 +37,7 @@ public class AccessHandler extends AbstractLifeCycle 
implements RequestLog {
       sb.append( "|" );
       sb.append( request.getMethod() );
       sb.append( "|" );
-      sb.append( request.getUri().toString() );
+      sb.append( request.getHttpURI() );
       sb.append( "|" );
       sb.append( request.getContentLength() );
       sb.append( "|" );

http://git-wip-us.apache.org/repos/asf/knox/blob/d5d95e32/gateway-test-release/pom.xml
----------------------------------------------------------------------
diff --git a/gateway-test-release/pom.xml b/gateway-test-release/pom.xml
index be49e30..605ad67 100644
--- a/gateway-test-release/pom.xml
+++ b/gateway-test-release/pom.xml
@@ -35,15 +35,47 @@
     </modules>
 
     <properties>
-        <!-- This property name must be different thant jetty.version
-             Otherwise Jetty 9.2.x and 9.3.x will not both be available.
-             Jetty 9.2.x is needed for Knox and Jetty 9.3.x for Hadoop -->
-        <hadoop-jetty.version>9.3.19.v20170502</hadoop-jetty.version>
+        <!-- This property name must be different than jetty.version
+             Otherwise Jetty 9.3.x and 9.4.x will not both be available.
+             Jetty 9.4.x is needed for Knox and Jetty 9.3.x for Hadoop -->
+        <hadoop-jetty.version>9.3.25.v20180904</hadoop-jetty.version>
     </properties>
 
     <dependencies>
         <dependency>
             <groupId>org.eclipse.jetty</groupId>
+            <artifactId>jetty-client</artifactId>
+            <version>${hadoop-jetty.version}</version>
+            <exclusions>
+                <exclusion>
+                    <groupId>org.eclipse.jetty</groupId>
+                    <artifactId>jetty-http</artifactId>
+                </exclusion>
+                <exclusion>
+                    <groupId>org.eclipse.jetty</groupId>
+                    <artifactId>jetty-io</artifactId>
+                </exclusion>
+            </exclusions>
+            <scope>test</scope>
+        </dependency>
+        <dependency>
+            <groupId>org.eclipse.jetty</groupId>
+            <artifactId>jetty-annotations</artifactId>
+            <version>${hadoop-jetty.version}</version>
+            <exclusions>
+                <exclusion>
+                    <groupId>org.eclipse.jetty</groupId>
+                    <artifactId>jetty-webapp</artifactId>
+                </exclusion>
+                <exclusion>
+                    <groupId>org.eclipse.jetty</groupId>
+                    <artifactId>jetty-util</artifactId>
+                </exclusion>
+            </exclusions>
+            <scope>test</scope>
+        </dependency>
+        <dependency>
+            <groupId>org.eclipse.jetty</groupId>
             <artifactId>jetty-server</artifactId>
             <version>${hadoop-jetty.version}</version>
             <exclusions>

http://git-wip-us.apache.org/repos/asf/knox/blob/d5d95e32/gateway-test-utils/pom.xml
----------------------------------------------------------------------
diff --git a/gateway-test-utils/pom.xml b/gateway-test-utils/pom.xml
index f90edb0..eddfb21 100644
--- a/gateway-test-utils/pom.xml
+++ b/gateway-test-utils/pom.xml
@@ -123,6 +123,16 @@
             <artifactId>javax.servlet-api</artifactId>
             <scope>provided</scope>
         </dependency>
-        
+
+        <dependency>
+            <groupId>org.eclipse.jetty</groupId>
+            <artifactId>jetty-servlet</artifactId>
+            <classifier>tests</classifier>
+        </dependency>
+        <dependency>
+            <groupId>org.eclipse.jetty</groupId>
+            <artifactId>jetty-http</artifactId>
+            <classifier>tests</classifier>
+        </dependency>
     </dependencies>
 </project>

http://git-wip-us.apache.org/repos/asf/knox/blob/d5d95e32/gateway-test/src/test/java/org/apache/knox/gateway/GatewayMultiFuncTest.java
----------------------------------------------------------------------
diff --git 
a/gateway-test/src/test/java/org/apache/knox/gateway/GatewayMultiFuncTest.java 
b/gateway-test/src/test/java/org/apache/knox/gateway/GatewayMultiFuncTest.java
index 5fb94b3..1deb738 100644
--- 
a/gateway-test/src/test/java/org/apache/knox/gateway/GatewayMultiFuncTest.java
+++ 
b/gateway-test/src/test/java/org/apache/knox/gateway/GatewayMultiFuncTest.java
@@ -185,7 +185,7 @@ public class GatewayMultiFuncTest {
         .then()
         //.log().all()
         .statusCode( HttpStatus.SC_OK )
-        .contentType( "application/json; charset=UTF-8" )
+        .contentType( "application/json;charset=UTF-8" )
         .when().get( gatewayUrl + "/knox678/repeat" ).andReturn().asString();
     assertThat( json, is("{\"msg\":\"H\u00eallo\"}") );
     assertThat( mock.isEmpty(), is(true) );
@@ -232,7 +232,7 @@ public class GatewayMultiFuncTest {
         .status( HttpStatus.SC_CREATED )
         .content( "{\"name\":\"value\"}".getBytes(StandardCharsets.UTF_8) )
         .contentLength( -1 )
-        .contentType( "application/json; charset=UTF-8" )
+        .contentType( "application/json;charset=UTF-8" )
         .header( "Location", gatewayUrl + "/knox681/repeat" );
 
     String uname = "guest";
@@ -259,7 +259,7 @@ public class GatewayMultiFuncTest {
     CloseableHttpResponse response = client.execute( request, context );
     assertThat( response.getStatusLine().getStatusCode(), is( 
HttpStatus.SC_CREATED ) );
     assertThat( response.getFirstHeader( "Location" ).getValue(), 
endsWith("/gateway/knox681/repeat" ) );
-    assertThat( response.getFirstHeader( "Content-Type" ).getValue(), 
is("application/json; charset=UTF-8") );
+    assertThat( response.getFirstHeader( "Content-Type" ).getValue(), 
is("application/json;charset=utf-8") );
     String body = new String( IOUtils.toByteArray( 
response.getEntity().getContent() ), StandardCharsets.UTF_8 );
     assertThat( body, is( "{\"name\":\"value\"}" ) );
     response.close();

http://git-wip-us.apache.org/repos/asf/knox/blob/d5d95e32/gateway-test/src/test/java/org/apache/knox/gateway/SimpleDescriptorHandlerFuncTest.java
----------------------------------------------------------------------
diff --git 
a/gateway-test/src/test/java/org/apache/knox/gateway/SimpleDescriptorHandlerFuncTest.java
 
b/gateway-test/src/test/java/org/apache/knox/gateway/SimpleDescriptorHandlerFuncTest.java
index f7c2103..e01dcbb 100644
--- 
a/gateway-test/src/test/java/org/apache/knox/gateway/SimpleDescriptorHandlerFuncTest.java
+++ 
b/gateway-test/src/test/java/org/apache/knox/gateway/SimpleDescriptorHandlerFuncTest.java
@@ -165,6 +165,7 @@ public class SimpleDescriptorHandlerFuncTest {
       
EasyMock.expect(config.getGatewayDeploymentDir()).andReturn(testDeployDir.getAbsolutePath()).anyTimes();
       
EasyMock.expect(config.getGatewayAddress()).andReturn(gatewayAddress).anyTimes();
       
EasyMock.expect(config.getGatewayPortMappings()).andReturn(Collections.emptyMap()).anyTimes();
+      EasyMock.expect(config.getThreadPoolMax()).andReturn(200).anyTimes();
       EasyMock.replay(config);
 
       // Setup the Gateway Services

http://git-wip-us.apache.org/repos/asf/knox/blob/d5d95e32/pom.xml
----------------------------------------------------------------------
diff --git a/pom.xml b/pom.xml
index b12b3b2..e6e9103 100644
--- a/pom.xml
+++ b/pom.xml
@@ -137,8 +137,10 @@
         <apacheds.version>2.0.0-M24</apacheds.version>
         <apacheds-jdbm.version>2.0.0-M5</apacheds-jdbm.version>
         <ant-nodeps.version>1.8.1</ant-nodeps.version>
+        <asm.version>7.0</asm.version>
         
<buildnumber-maven-plugin.version>1.4</buildnumber-maven-plugin.version>
         <checkstyle.version>8.14</checkstyle.version>
+        <cglib.version>3.2.9</cglib.version>
         <commons-beanutils.version>1.9.3</commons-beanutils.version>
         <commons-cli.version>1.4</commons-cli.version>
         <commons-codec.version>1.11</commons-codec.version>
@@ -174,7 +176,7 @@
         <javax.websocket-api.version>1.1</javax.websocket-api.version>
         <jericho-html.version>3.4</jericho-html.version>
         <jersey.version>2.6</jersey.version>
-        <jetty.version>9.2.26.v20180806</jetty.version>
+        <jetty.version>9.4.12.v20180830</jetty.version>
         <jline.version>2.14.6</jline.version>
         <jna.version>5.0.0</jna.version>
         <joda-time.version>2.10.1</joda-time.version>
@@ -956,6 +958,19 @@
             </dependency>
 
             <dependency>
+                <groupId>org.eclipse.jetty</groupId>
+                <artifactId>jetty-servlet</artifactId>
+                <version>${jetty.version}</version>
+                <classifier>tests</classifier>
+            </dependency>
+            <dependency>
+                <groupId>org.eclipse.jetty</groupId>
+                <artifactId>jetty-http</artifactId>
+                <version>${jetty.version}</version>
+                <classifier>tests</classifier>
+            </dependency>
+
+            <dependency>
                 <groupId>javax.annotation</groupId>
                 <artifactId>javax.annotation-api</artifactId>
                 <version>${javax.annotation-api.version}</version>
@@ -1244,6 +1259,12 @@
             </dependency>
 
             <dependency>
+                <groupId>cglib</groupId>
+                <artifactId>cglib</artifactId>
+                <version>${cglib.version}</version>
+            </dependency>
+
+            <dependency>
                 <groupId>commons-beanutils</groupId>
                 <artifactId>commons-beanutils</artifactId>
                 <version>${commons-beanutils.version}</version>
@@ -1277,6 +1298,12 @@
                 <groupId>org.apache.commons</groupId>
                 <artifactId>commons-digester3</artifactId>
                 <version>${commons-digester3.version}</version>
+                <exclusions>
+                    <exclusion>
+                        <groupId>cglib</groupId>
+                        <artifactId>cglib</artifactId>
+                    </exclusion>
+                </exclusions>
             </dependency>
             <dependency>
                 <groupId>commons-io</groupId>
@@ -1445,6 +1472,17 @@
             </dependency>
 
             <dependency>
+                <groupId>org.ow2.asm</groupId>
+                <artifactId>asm</artifactId>
+                <version>${asm.version}</version>
+            </dependency>
+            <dependency>
+                <groupId>org.ow2.asm</groupId>
+                <artifactId>asm-commons</artifactId>
+                <version>${asm.version}</version>
+            </dependency>
+
+            <dependency>
                 <groupId>org.eclipse.jetty</groupId>
                 <artifactId>jetty-io</artifactId>
                 <version>${jetty.version}</version>
@@ -1482,6 +1520,11 @@
                 <artifactId>javax.websocket-api</artifactId>
                 <version>${javax.websocket-api.version}</version>
             </dependency>
+            <dependency>
+                <groupId>javax.websocket</groupId>
+                <artifactId>javax.websocket-client-api</artifactId>
+                <version>${javax.websocket-api.version}</version>
+            </dependency>
             
             <dependency>
                 <groupId>org.eclipse.jetty.websocket</groupId>

Reply via email to