Update Jetty to most up to date version that supports Java SE 1.7.

http://www.eclipse.org/jetty/documentation/9.4.x/what-jetty-version.html Fixes googleapis#130
mattwhisenhunt · Mar 16, 2018 · a86ac68 · a86ac68
1 parent d6eb235
commit a86ac68
Show file tree

Hide file tree

Showing 3 changed files with 42 additions and 15 deletions.
diff --git a/google-oauth-client-jetty/pom.xml b/google-oauth-client-jetty/pom.xml
@@ -84,8 +84,20 @@
       <artifactId>google-oauth-client-java6</artifactId>
     </dependency>
     <dependency>
-      <groupId>org.mortbay.jetty</groupId>
-      <artifactId>jetty</artifactId>
+      <groupId>org.eclipse.jetty</groupId>
+      <artifactId>jetty-server</artifactId>
+    </dependency>
+    <dependency>
+      <groupId>org.eclipse.jetty</groupId>
+      <artifactId>jetty-util</artifactId>
+    </dependency>
+    <dependency>
+      <groupId>org.eclipse.jetty</groupId>
+      <artifactId>jetty-http</artifactId>
+    </dependency>
+    <dependency>
+      <groupId>org.eclipse.jetty</groupId>
+      <artifactId>jetty-io</artifactId>
     </dependency>
     <dependency>
       <groupId>junit</groupId>

diff --git a/...src/main/java/com/google/api/client/extensions/jetty/auth/oauth2/LocalServerReceiver.java b/...src/main/java/com/google/api/client/extensions/jetty/auth/oauth2/LocalServerReceiver.java
@@ -18,13 +18,14 @@
 import com.google.api.client.util.Throwables;
 import java.io.IOException;
 import java.io.PrintWriter;
+import java.net.InetSocketAddress;
 import java.util.concurrent.Semaphore;
 import javax.servlet.http.HttpServletRequest;
 import javax.servlet.http.HttpServletResponse;
-import org.mortbay.jetty.Connector;
-import org.mortbay.jetty.Request;
-import org.mortbay.jetty.Server;
-import org.mortbay.jetty.handler.AbstractHandler;
+import org.eclipse.jetty.server.Request;
+import org.eclipse.jetty.server.Server;
+import org.eclipse.jetty.server.NetworkConnector;
+import org.eclipse.jetty.server.handler.AbstractHandler;
 
 /**
  * OAuth 2.0 verification code receiver that runs a Jetty server on a free port, waiting for a
@@ -115,10 +116,9 @@ public LocalServerReceiver() {
 
   @Override
   public String getRedirectUri() throws IOException {
-    server = new Server(port != -1 ? port : 0);
-    Connector connector = server.getConnectors()[0];
-    connector.setHost(host);
-    server.addHandler(new CallbackHandler());
+    server = new Server(new InetSocketAddress(host, port != -1 ? port : 0));
+    NetworkConnector connector = (NetworkConnector) server.getConnectors()[0];
+    server.setHandler(new CallbackHandler());
     try {
       server.start();
       port = connector.getLocalPort();
@@ -253,8 +253,8 @@ public Builder setLandingPages(String successLandingPageUrl, String failureLandi
   class CallbackHandler extends AbstractHandler {
 
     @Override
-    public void handle(
-        String target, HttpServletRequest request, HttpServletResponse response, int dispatch)
+    public void handle(String target, Request baseRequest, HttpServletRequest request,
+        HttpServletResponse response)
         throws IOException {
       if (!CALLBACK_PATH.equals(target)) {
         return;

diff --git a/pom.xml b/pom.xml
@@ -182,8 +182,23 @@
         <version>${project.version}</version>
       </dependency>
       <dependency>
-        <groupId>org.mortbay.jetty</groupId>
-        <artifactId>jetty</artifactId>
+        <groupId>org.eclipse.jetty</groupId>
+        <artifactId>jetty-server</artifactId>
+        <version>${project.jetty.version}</version>
+      </dependency>
+      <dependency>
+        <groupId>org.eclipse.jetty</groupId>
+        <artifactId>jetty-util</artifactId>
+        <version>${project.jetty.version}</version>
+      </dependency>
+      <dependency>
+        <groupId>org.eclipse.jetty</groupId>
+        <artifactId>jetty-http</artifactId>
+        <version>${project.jetty.version}</version>
+      </dependency>
+      <dependency>
+        <groupId>org.eclipse.jetty</groupId>
+        <artifactId>jetty-io</artifactId>
         <version>${project.jetty.version}</version>
       </dependency>
       <dependency>
@@ -497,7 +512,7 @@
     <project.gson.version>2.1</project.gson.version>
     <project.jackson-core-asl.version>1.9.11</project.jackson-core-asl.version>
     <project.jackson-core2.version>2.9.2</project.jackson-core2.version>
-    <project.jetty.version>6.1.26</project.jetty.version>
+    <project.jetty.version>9.2.24.v20180105</project.jetty.version>
     <project.protobuf-java.version>2.4.1</project.protobuf-java.version>
     <project.guava.version>19.0</project.guava.version>
     <project.xpp3.version>1.1.4c</project.xpp3.version>