Merge pull request #38995 from geoand/#38798-again

Take client methods into account in server endpoint indexer

extensions/resteasy-reactive/quarkus-resteasy-reactive/deployment/src/main/java/io/quarkus/resteasy/reactive/server/deployment/QuarkusServerEndpointIndexer.java

@@ -40,6 +40,7 @@ public class QuarkusServerEndpointIndexer
         extends ServerEndpointIndexer {
 
     private static final org.jboss.logging.Logger LOGGER = Logger.getLogger(QuarkusServerEndpointIndexer.class);
+    private static final String REST_CLIENT_NOT_BODY_ANNOTATION = "io.quarkus.rest.client.reactive.NotBody";
 
     private final Capabilities capabilities;
     private final BuildProducer<GeneratedClassBuildItem> generatedClassBuildItemBuildProducer;
@@ -263,4 +264,14 @@ protected void logMissingJsonWarning(MethodInfo info) {
                 + "' but no JSON extension has been added. Consider adding 'quarkus-resteasy-reactive-jackson' (recommended) or 'quarkus-resteasy-reactive-jsonb'.");
     }
 
+    @Override
+    protected void warnAboutMissUsedBodyParameter(DotName httpMethod, MethodInfo methodInfo) {
+        // This indexer also picks up REST client methods as well as there is no bulletproof way of distinguishing the two.
+        // That is why we check for client specific annotations here
+        if (methodInfo.hasAnnotation(REST_CLIENT_NOT_BODY_ANNOTATION)) {
+            return;
+        }
+        super.warnAboutMissUsedBodyParameter(httpMethod, methodInfo);
+    }
+
 }