Fix JAX-RS Application class hierarchy handling in RESTEasy Classic

Fixes: #16228
quarkusio · Apr 5, 2021 · d65d3b5 · d65d3b5
1 parent c7b5494
commit d65d3b5
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 2 deletions.
diff --git a/...main/java/io/quarkus/resteasy/server/common/deployment/ResteasyServerCommonProcessor.java b/...main/java/io/quarkus/resteasy/server/common/deployment/ResteasyServerCommonProcessor.java
@@ -953,6 +953,9 @@ private static Set<String> getAllowedClasses(IndexView index) {
         Application application;
         ClassInfo selectedAppClass = null;
         for (ClassInfo applicationClassInfo : applications) {
+            if (Modifier.isAbstract(applicationClassInfo.flags())) {
+                continue;
+            }
             if (selectedAppClass != null) {
                 throw new RuntimeException("More than one Application class: " + applications);
             }

diff --git a/...y/deployment/src/test/java/io/quarkus/resteasy/test/root/ApplicationPathHttpRootTest.java b/...y/deployment/src/test/java/io/quarkus/resteasy/test/root/ApplicationPathHttpRootTest.java
@@ -23,7 +23,7 @@ public class ApplicationPathHttpRootTest {
     @RegisterExtension
     static QuarkusUnitTest runner = new QuarkusUnitTest()
             .setArchiveProducer(() -> ShrinkWrap.create(JavaArchive.class)
-                    .addClasses(HelloResource.class, HelloApp.class)
+                    .addClasses(HelloResource.class, HelloApp.class, BaseApplication.class)
                     .addAsResource(new StringAsset("quarkus.http.root-path=/foo"), "application.properties"));
 
     @Test
@@ -44,7 +44,11 @@ public String hello() {
     }
 
     @ApplicationPath("hello")
-    public static class HelloApp extends Application {
+    public static class HelloApp extends BaseApplication {
+
+    }
+
+    public static abstract class BaseApplication extends Application {
 
     }