Fix for: Exception occurring when encoding the path containing space

Fixes #358
quarkiverse · Mar 28, 2024 · e002719 · e002719
1 parent e8c2d4c
commit e002719
Show file tree

Hide file tree

Showing 2 changed files with 21 additions and 1 deletion.
diff --git a/integration-test/src/test/java/com/tietoevry/quarkus/resteasy/problem/JaxRsMappersIT.java b/integration-test/src/test/java/com/tietoevry/quarkus/resteasy/problem/JaxRsMappersIT.java
@@ -78,6 +78,18 @@ void jaxRsNotFoundShouldReturn404ProblemInsteadOfDefaultRestEasyDefaultResponse(
                 .body("stacktrace", nullValue());
     }
 
+    @Test
+    void jaxRsNotFoundShouldHandleSpacesInUriProperly() {
+        given()
+                .queryParam("message", SAMPLE_DETAIL)
+                .get("/throw/jax-rs/not-found exception")
+                .then()
+                .statusCode(NOT_FOUND.getStatusCode())
+                .body("title", equalTo(NOT_FOUND.getReasonPhrase()))
+                .body("status", equalTo(NOT_FOUND.getStatusCode()))
+                .body("instance", equalTo("/throw/jax-rs/not-found%20exception"));
+    }
+
     @Test
     void jaxRsForbiddenShouldReturn403ProblemInsteadOfDefaultRestEasyDefaultResponse() {
         given()

diff --git a/...n/java/com/tietoevry/quarkus/resteasy/problem/postprocessing/ProblemDefaultsProvider.java b/...n/java/com/tietoevry/quarkus/resteasy/problem/postprocessing/ProblemDefaultsProvider.java
@@ -2,6 +2,7 @@
 
 import com.tietoevry.quarkus.resteasy.problem.HttpProblem;
 import java.net.URI;
+import java.net.URISyntaxException;
 
 /**
  * Replaces <code>null</code> value of <code>instance</code> with URI of currently served endpoint, i.e
@@ -26,7 +27,14 @@ public HttpProblem apply(HttpProblem problem, ProblemContext context) {
     }
 
     private URI defaultInstance(ProblemContext context) {
-        return context.path == null ? null : URI.create(context.path);
+        if (context.path == null) {
+            return null;
+        }
+        try {
+            return new URI(context.path.replaceAll(" ", "%20"));
+        } catch (URISyntaxException e) {
+            return null;
+        }
     }
 
 }