JaxRS sample application code without web.xml and for EE8.

Author: ludoch

applications/pom.xml

@@ -38,5 +38,6 @@
     <module>guestbook_jakarta</module>
     <module>servletasyncapp</module>
     <module>servletasyncappjakarta</module>
+    <module>jaxrs</module>
   </modules>
 </project>

e2etests/devappservertests/src/test/java/com/google/appengine/tools/development/JaxRsTest.java

@@ -0,0 +1,70 @@
+/*
+ * Copyright 2021 Google LLC
+ *
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at
+ *
+ *     https://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+package com.google.appengine.tools.development;
+
+
+import java.io.File;
+import java.io.IOException;
+import java.util.Arrays;
+import java.util.List;
+import org.junit.Before;
+import org.junit.Test;
+import org.junit.runner.RunWith;
+import org.junit.runners.Parameterized;
+
+@RunWith(Parameterized.class)
+public class JaxRsTest extends DevAppServerTestBase {
+
+  @Parameterized.Parameters
+  public static List<Object[]> version() {
+      // Only EE8 app.
+return Arrays.asList(
+        new Object[][] {
+      //       {"java17", "9.4", "EE6"},
+              {"java17", "12.0", "EE8"},
+      //        {"java21", "12.0", "EE8"},
+     //         {"java25", "12.1", "EE8"},
+
+        });
+  
+  }
+  public JaxRsTest(String runtimeVersion, String jettyVersion, String jakartaVersion) {
+    super(runtimeVersion, jettyVersion, jakartaVersion);
+  }
+
+  @Before
+  public void setUpClass() throws IOException, InterruptedException {
+    File currentDirectory = new File("").getAbsoluteFile();
+    File appRoot =
+        new File(
+            currentDirectory,
+            "../../applications/jaxrs/target/jaxrs"
+                + "-"
+                + System.getProperty("appengine.projectversion"));
+    setUpClass(appRoot);
+  }
+
+  @Test
+  public void testJaxRs() throws Exception {
+    // App Engine Memcache access.
+    executeHttpGet(
+        "/hello",
+        "hello\n",
+        RESPONSE_200);
+
+  }
+
+}

runtime/local_jetty121/src/main/java/com/google/appengine/tools/development/jetty/JettyContainerService.java

@@ -186,7 +186,11 @@ public void exitScope(ContextHandler.APIContext context, Request request) {
     // Set the location of deployment descriptor.  This value might be null,
     // which is fine, it just means Jetty will look for it in the default
     // location (WEB-INF/web.xml).
-    context.setDescriptor(webXmlLocation == null ? null : webXmlLocation.getAbsolutePath());
+    // Only set the descriptor if the web.xml file actually exists.
+    // Jetty 12 throws an IllegalArgumentException if the descriptor path is invalid.
+      if (webXmlLocation != null && webXmlLocation.exists()) {
+          context.setDescriptor(webXmlLocation.getAbsolutePath());
+      }
 
     // Override the web.xml that Jetty automatically prepends to other
     // web.xml files.  This is where the DefaultServlet is registered,
@@ -353,6 +357,8 @@ protected void connectContainer() throws Exception {
     } else {
       server = new Server();
     }
+server.setDumpAfterStart(true);
+
     try {
       NetworkTrafficServerConnector connector =
           new NetworkTrafficServerConnector(

runtime/local_jetty121_ee11/src/main/java/com/google/appengine/tools/development/jetty/ee11/JettyContainerService.java

@@ -192,8 +192,11 @@ public void exitScope(
     // Set the location of deployment descriptor.  This value might be null,
     // which is fine, it just means Jetty will look for it in the default
     // location (WEB-INF/web.xml).
-    context.setDescriptor(webXmlLocation == null ? null : webXmlLocation.getAbsolutePath());
-
+    // Jetty 12 throws an IllegalArgumentException if the descriptor path is invalid.
+      if (webXmlLocation != null && webXmlLocation.exists()) {
+          context.setDescriptor(webXmlLocation.getAbsolutePath());
+      }
+      
     // Override the web.xml that Jetty automatically prepends to other
     // web.xml files.  This is where the DefaultServlet is registered,
     // which serves static files.  We override it to disable some