Support UUID serialization in REST Client multipart bodies

Fixes: quarkusio#31298
gsmet · Feb 20, 2023 · b185920 · b185920
1 parent 5849491
commit b185920
Show file tree

Hide file tree

Showing 4 changed files with 25 additions and 1 deletion.
diff --git a/...active/runtime/src/main/java/io/quarkus/jaxrs/client/reactive/runtime/RestClientBase.java b/...active/runtime/src/main/java/io/quarkus/jaxrs/client/reactive/runtime/RestClientBase.java
@@ -5,6 +5,7 @@
 import java.lang.reflect.Type;
 import java.util.List;
 import java.util.Map;
+import java.util.UUID;
 import java.util.concurrent.ConcurrentHashMap;
 import java.util.function.Supplier;
 
@@ -101,6 +102,18 @@ public String toString(Boolean value) {
             return value == null ? null : value.toString();
         }
     };
+
+    private static final ParamConverter<UUID> UUID_CONVERTER = new ParamConverter<>() {
+        @Override
+        public UUID fromString(String value) {
+            return value == null ? null : UUID.fromString(value);
+        }
+
+        @Override
+        public String toString(UUID value) {
+            return value == null ? null : value.toString();
+        }
+    };
     private final List<ParamConverterProvider> paramConverterProviders;
     private final Map<Class<?>, ParamConverterProvider> providerForClass = new ConcurrentHashMap<>();
 
@@ -197,6 +210,9 @@ public <T> ParamConverter<T> getConverter(Class<T> rawType, Type genericType, An
             if (rawType == boolean.class || rawType == Boolean.class) {
                 return (ParamConverter<T>) BOOLEAN_CONVERTER;
             }
+            if (rawType == UUID.class) {
+                return (ParamConverter<T>) UUID_CONVERTER;
+            }
             return null;
         }
 

diff --git a/integration-tests/rest-client/src/main/java/io/quarkus/it/rest/client/MultipartBody.java b/integration-tests/rest-client/src/main/java/io/quarkus/it/rest/client/MultipartBody.java
@@ -1,6 +1,7 @@
 package io.quarkus.it.rest.client;
 
 import java.io.InputStream;
+import java.util.UUID;
 
 import jakarta.ws.rs.FormParam;
 import jakarta.ws.rs.core.MediaType;
@@ -16,4 +17,8 @@ public class MultipartBody {
     @FormParam("fileName")
     @PartType(MediaType.TEXT_PLAIN)
     public String fileName;
+
+    @FormParam("uuid")
+    @PartType(MediaType.TEXT_PLAIN)
+    public UUID uuid;
 }
diff --git a/...on-tests/rest-client/src/main/java/io/quarkus/it/rest/client/MultipartClientResource.java b/...on-tests/rest-client/src/main/java/io/quarkus/it/rest/client/MultipartClientResource.java
@@ -2,6 +2,7 @@
 
 import java.io.ByteArrayInputStream;
 import java.nio.charset.StandardCharsets;
+import java.util.UUID;
 
 import jakarta.inject.Inject;
 import jakarta.ws.rs.Consumes;
@@ -27,6 +28,7 @@ public String sendFile() throws Exception {
         MultipartBody body = new MultipartBody();
         body.fileName = "greeting.txt";
         body.file = new ByteArrayInputStream("HELLO WORLD".getBytes(StandardCharsets.UTF_8));
+        body.uuid = UUID.randomUUID();
         return service.sendMultipartData(body);
     }
 }
diff --git a/...tion-tests/rest-client/src/test/java/io/quarkus/it/rest/client/MultipartResourceTest.java b/...tion-tests/rest-client/src/test/java/io/quarkus/it/rest/client/MultipartResourceTest.java
@@ -21,7 +21,8 @@ public void testMultipartDataIsSent() {
                 .body(containsString("Content-Disposition: form-data; name=\"file\""),
                         containsString("HELLO WORLD"),
                         containsString("Content-Disposition: form-data; name=\"fileName\""),
-                        containsString("greeting.txt"));
+                        containsString("greeting.txt"),
+                        containsString("Content-Disposition: form-data; name=\"uuid\""));
 
         given()
                 .when().get("/q/metrics")