Issue-88: fixed param serialisation in case field is a reserved name - …

…#88
hyperledger-labs · Nov 18, 2023 · 34fc6fe · 34fc6fe
1 parent ae155fa
commit 34fc6fe
Show file tree

Hide file tree

Showing 5 changed files with 26 additions and 3 deletions.
diff --git a/pom.xml b/pom.xml
@@ -61,7 +61,7 @@
         <jackson.databind.version>2.15.2</jackson.databind.version>
         <junit.jupiter.api.version>5.10.0</junit.jupiter.api.version>
         <kotlin.version>1.9.0</kotlin.version>
-        <lombok.version>1.18.28</lombok.version>
+        <lombok.version>1.18.30</lombok.version>
         <lombok.maven.plugin.version>1.18.20.0</lombok.maven.plugin.version>
         <okhttp.version>4.11.0</okhttp.version>
         <pmd.version>6.55.0</pmd.version>

diff --git a/src/main/java/org/hyperledger/aries/api/AcaPyRequestFilter.java b/src/main/java/org/hyperledger/aries/api/AcaPyRequestFilter.java
@@ -49,7 +49,7 @@ default HttpUrl.Builder buildParams(@NonNull HttpUrl.Builder b) {
                         value = ((Boolean) o).toString().toLowerCase(Locale.US);
                     }
                     if (value != null) {
-                        b.addQueryParameter(FieldNamingPolicy.LOWER_CASE_WITH_UNDERSCORES.translateName(f), value);
+                        b.addQueryParameter(translateFieldName(f), value);
                     }
                 }
             } catch (IllegalAccessException e) {
@@ -58,4 +58,12 @@ default HttpUrl.Builder buildParams(@NonNull HttpUrl.Builder b) {
         });
         return b;
     }
+
+    private static String translateFieldName(Field field) {
+        SerializedName sn = field.getAnnotation(SerializedName.class);
+        if (sn != null) {
+            return sn.value();
+        }
+        return FieldNamingPolicy.LOWER_CASE_WITH_UNDERSCORES.translateName(field);
+    }
 }
diff --git a/src/main/java/org/hyperledger/aries/api/connection/CreateInvitationParams.java b/src/main/java/org/hyperledger/aries/api/connection/CreateInvitationParams.java
@@ -7,6 +7,7 @@
  */
 package org.hyperledger.aries.api.connection;
 
+import com.google.gson.annotations.SerializedName;
 import lombok.AllArgsConstructor;
 import lombok.Builder;
 import lombok.Data;
@@ -18,5 +19,6 @@ public class CreateInvitationParams implements AcaPyRequestFilter {
     private String alias;
     private Boolean autoAccept;
     private Boolean multiUse;
+    @SerializedName("public")
     private Boolean isPublic;
 }
diff --git a/...in/java/org/hyperledger/aries/api/present_proof/PresentationRequestCredentialsFilter.java b/...in/java/org/hyperledger/aries/api/present_proof/PresentationRequestCredentialsFilter.java
@@ -40,7 +40,7 @@ public HttpUrl.Builder buildParams(@NonNull HttpUrl.Builder b) {
         if (StringUtils.isNotEmpty(extraQuery)) {
             b.addQueryParameter("extra_query", extraQuery);
         }
-        if (referent != null && referent.size() > 0) {
+        if (referent != null && !referent.isEmpty()) {
             StringBuilder sb = new StringBuilder();
             Iterator<String> it = referent.stream().iterator();
             while (it.hasNext()) {

diff --git a/src/test/java/org/hyperledger/aries/api/AcaPyRequestFilterTest.java b/src/test/java/org/hyperledger/aries/api/AcaPyRequestFilterTest.java
@@ -55,6 +55,13 @@ void testListWalletDid() {
         Assertions.assertEquals(url + "?key_type=bls12381g2", f.buildParams(b).toString());
     }
 
+    @Test
+    void testAnnotatedBoolean() {
+        HttpUrl.Builder b = Objects.requireNonNull(base).newBuilder();
+        AnnotatedBoolean ab = AnnotatedBoolean.builder().isPublic(Boolean.TRUE).build();
+        Assertions.assertEquals(url + "?public=true", ab.buildParams(b).toString());
+    }
+
     @Test
     void testEmptyString() {
         HttpUrl.Builder b = Objects.requireNonNull(base).newBuilder();
@@ -79,4 +86,10 @@ private enum AnnotatedDummyEnum {
         @SerializedName("value_2")
         VALUE2
     }
+
+    @Builder
+    private static final class AnnotatedBoolean implements AcaPyRequestFilter {
+        @SerializedName("public")
+        private Boolean isPublic;
+    }
 }