Don't use objectmapper for strings in deserialize (#307)

Co-authored-by: nsimonides <[email protected]>
flock-community · Dec 9, 2024 · 1ef2d54 · 1ef2d54
1 parent 04eb553
commit 1ef2d54
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 4 deletions.
diff --git a/...lock/wirespec/integration/spring/java/configuration/WirespecSerializationConfiguration.kt b/...lock/wirespec/integration/spring/java/configuration/WirespecSerializationConfiguration.kt
@@ -28,6 +28,10 @@ open class WirespecSerializationConfiguration {
 
         override fun <T : Any> deserialize(raw: String?, valueType: Type?): T? = raw?.let {
             when {
+                valueType == String::class.java -> {
+                    @Suppress("UNCHECKED_CAST")
+                    raw as T
+                }
                 isStringIterable(valueType) -> {
                     @Suppress("UNCHECKED_CAST")
                     raw.split(stringListDelimiter) as T

diff --git a/...ck/wirespec/integration/spring/kotlin/configuration/WirespecSerializationConfiguration.kt b/...ck/wirespec/integration/spring/kotlin/configuration/WirespecSerializationConfiguration.kt
@@ -31,10 +31,10 @@ open class WirespecSerializationConfiguration {
             }
 
         override fun <T> deserialize(raw: String, kType: KType): T =
-            if (isStringIterable(kType)) {
-                raw.split(stringListDelimiter) as T
-            } else {
-                wirespecObjectMapper
+            when {
+                kType.classifier == String::class -> raw as T
+                isStringIterable(kType) -> raw.split(stringListDelimiter) as T
+                else -> wirespecObjectMapper
                     .constructType(kType.javaType)
                     .let { wirespecObjectMapper.readValue(raw, it) }
             }