Register resource classes for reflection when ContainerResponseFilter…

… exists This is needed because those filters can call setEntityStream which then forces the use of the slow path for calling writers Closes: #42537
quarkusio · Aug 26, 2024 · 550b84b · 550b84b
1 parent 2105d46
commit 550b84b
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 0 deletions.
diff --git a/...c/main/java/io/quarkus/resteasy/reactive/server/deployment/ResteasyReactiveProcessor.java b/...c/main/java/io/quarkus/resteasy/reactive/server/deployment/ResteasyReactiveProcessor.java
@@ -1078,6 +1078,7 @@ public void additionalReflection(BeanArchiveIndexBuildItem beanArchiveIndexBuild
             SetupEndpointsResultBuildItem setupEndpointsResult,
             List<MessageBodyReaderBuildItem> messageBodyReaderBuildItems,
             List<MessageBodyWriterBuildItem> messageBodyWriterBuildItems,
+            ResourceInterceptorsBuildItem resourceInterceptorsBuildItem,
             BuildProducer<ReflectiveClassBuildItem> producer) {
         List<ResourceClass> resourceClasses = setupEndpointsResult.getResourceClasses();
         IndexView index = beanArchiveIndexBuildItem.getIndex();
@@ -1122,6 +1123,13 @@ public void additionalReflection(BeanArchiveIndexBuildItem beanArchiveIndexBuild
                 }
             }
         }
+
+        // when a ContainerResponseFilter exists, it can potentially do responseContext.setEntityStream()
+        // which then forces the use of the slow path for calling writers
+        if (!resourceInterceptorsBuildItem.getResourceInterceptors().getContainerResponseFilters().isEmpty()) {
+            serializersRequireResourceReflection = true;
+        }
+
         if (serializersRequireResourceReflection) {
             producer.produce(ReflectiveClassBuildItem
                     .builder(resourceClasses.stream().map(ResourceClass::getClassName).toArray(String[]::new))

diff --git a/.../runtime/src/main/java/org/jboss/resteasy/reactive/common/model/InterceptorContainer.java b/.../runtime/src/main/java/org/jboss/resteasy/reactive/common/model/InterceptorContainer.java
@@ -33,6 +33,10 @@ public void sort() {
         Collections.sort(nameResourceInterceptors);
     }
 
+    public boolean isEmpty() {
+        return globalResourceInterceptors.isEmpty() && nameResourceInterceptors.isEmpty();
+    }
+
     public ResourceInterceptor<T> create() {
         return new ResourceInterceptor<>();
     }