Do not strip trailing slashes when using Reactive REST Client

Fixes: #27050
quarkusio · Aug 1, 2022 · 94ee05e · 94ee05e
1 parent 18dcadb
commit 94ee05e
Show file tree

Hide file tree

Showing 3 changed files with 9 additions and 4 deletions.
diff --git a/...rc/test/java/io/quarkus/rest/client/reactive/stork/StorkResponseTimeLoadBalancerTest.java b/...rc/test/java/io/quarkus/rest/client/reactive/stork/StorkResponseTimeLoadBalancerTest.java
@@ -34,7 +34,7 @@ public class StorkResponseTimeLoadBalancerTest {
     @BeforeAll
     public static void setUp() {
         server = new WireMockServer(options().port(8766));
-        server.stubFor(WireMock.post("/hello")
+        server.stubFor(WireMock.post("/hello/")
                 .willReturn(aResponse().withFixedDelay(1000)
                         .withBody(SLOW_RESPONSE).withStatus(200)));
         server.start();

diff --git a/...processor/src/main/java/org/jboss/resteasy/reactive/common/processor/EndpointIndexer.java b/...processor/src/main/java/org/jboss/resteasy/reactive/common/processor/EndpointIndexer.java
@@ -392,7 +392,7 @@ protected List<ResourceMethod> createEndpoints(ClassInfo currentClassInfo,
                             methodPath = "/" + methodPath;
                         }
                         if (methodPath.endsWith("/")) {
-                            methodPath = methodPath.substring(0, methodPath.length() - 1);
+                            methodPath = handleTrailingSlash(methodPath);
                         }
                     } else {
                         methodPath = "";
@@ -451,6 +451,10 @@ protected List<ResourceMethod> createEndpoints(ClassInfo currentClassInfo,
         return ret;
     }
 
+    protected String handleTrailingSlash(String path) {
+        return path;
+    }
+
     private void validateHttpAnnotations(MethodInfo info) {
         List<AnnotationInstance> annotationInstances = info.annotations();
         Set<DotName> allMethodAnnotations = new HashSet<>(annotationInstances.size());

diff --git a/...sor/src/main/java/org/jboss/resteasy/reactive/server/processor/ServerEndpointIndexer.java b/...sor/src/main/java/org/jboss/resteasy/reactive/server/processor/ServerEndpointIndexer.java
@@ -381,8 +381,9 @@ protected void handleArrayParam(Map<String, String> existingConverters, String e
         builder.setConverter(new ArrayConverter.ArraySupplier(converter, elementType));
     }
 
-    protected void handlePathSegmentParam(ServerIndexedParameter builder) {
-        builder.setConverter(new PathSegmentParamConverter.Supplier());
+    @Override
+    protected String handleTrailingSlash(String path) {
+        return path.substring(0, path.length() - 1);
     }
 
     protected void handleTemporalParam(ServerIndexedParameter builder, DotName paramType,