chore: test on Jackson 2.14.0 to avoid encountering FasterXML/jackson-databind#3240 in tests

stainless-app[bot] · stainless-app[bot] · commit fcc607ba9e5c · 2026-01-16T06:11:59.000Z
fix: date time deserialization leniency
diff --git a/README.md b/README.md
@@ -393,6 +393,8 @@ If the SDK threw an exception, but you're _certain_ the version is compatible, t
 > [!CAUTION]
 > We make no guarantee that the SDK works correctly when the Jackson version check is disabled.
 
+Also note that there are bugs in older Jackson versions that can affect the SDK. We don't work around all Jackson bugs ([example](https://github.com/FasterXML/jackson-databind/issues/3240)) and expect users to upgrade Jackson for those instead.
+
 ## Network options
 
 ### Retries
diff --git a/finch-java-core/build.gradle.kts b/finch-java-core/build.gradle.kts
@@ -5,14 +5,16 @@ plugins {
 
 configurations.all {
     resolutionStrategy {
-        // Compile and test against a lower Jackson version to ensure we're compatible with it.
-        // We publish with a higher version (see below) to ensure users depend on a secure version by default.
-        force("com.fasterxml.jackson.core:jackson-core:2.13.4")
-        force("com.fasterxml.jackson.core:jackson-databind:2.13.4")
-        force("com.fasterxml.jackson.core:jackson-annotations:2.13.4")
-        force("com.fasterxml.jackson.datatype:jackson-datatype-jdk8:2.13.4")
-        force("com.fasterxml.jackson.datatype:jackson-datatype-jsr310:2.13.4")
-        force("com.fasterxml.jackson.module:jackson-module-kotlin:2.13.4")
+        // Compile and test against a lower Jackson version to ensure we're compatible with it. Note that
+        // we generally support 2.13.4, but test against 2.14.0 because 2.13.4 has some annoying (but
+        // niche) bugs (users should upgrade if they encounter them). We publish with a higher version
+        // (see below) to ensure users depend on a secure version by default.
+        force("com.fasterxml.jackson.core:jackson-core:2.14.0")
+        force("com.fasterxml.jackson.core:jackson-databind:2.14.0")
+        force("com.fasterxml.jackson.core:jackson-annotations:2.14.0")
+        force("com.fasterxml.jackson.datatype:jackson-datatype-jdk8:2.14.0")
+        force("com.fasterxml.jackson.datatype:jackson-datatype-jsr310:2.14.0")
+        force("com.fasterxml.jackson.module:jackson-module-kotlin:2.14.0")
     }
 }
 
diff --git a/finch-java-core/src/main/kotlin/com/tryfinch/api/core/ObjectMappers.kt b/finch-java-core/src/main/kotlin/com/tryfinch/api/core/ObjectMappers.kt
@@ -24,6 +24,7 @@ import java.io.InputStream
 import java.time.DateTimeException
 import java.time.LocalDate
 import java.time.LocalDateTime
+import java.time.OffsetDateTime
 import java.time.ZonedDateTime
 import java.time.format.DateTimeFormatter
 import java.time.temporal.ChronoField
@@ -36,7 +37,7 @@ fun jsonMapper(): JsonMapper =
         .addModule(
             SimpleModule()
                 .addSerializer(InputStreamSerializer)
-                .addDeserializer(LocalDateTime::class.java, LenientLocalDateTimeDeserializer())
+                .addDeserializer(OffsetDateTime::class.java, LenientOffsetDateTimeDeserializer())
         )
         .withCoercionConfig(LogicalType.Boolean) {
             it.setCoercion(CoercionInputShape.Integer, CoercionAction.Fail)
@@ -64,6 +65,12 @@ fun jsonMapper(): JsonMapper =
                 .setCoercion(CoercionInputShape.Array, CoercionAction.Fail)
                 .setCoercion(CoercionInputShape.Object, CoercionAction.Fail)
         }
+        .withCoercionConfig(LogicalType.DateTime) {
+            it.setCoercion(CoercionInputShape.Integer, CoercionAction.Fail)
+                .setCoercion(CoercionInputShape.Float, CoercionAction.Fail)
+                .setCoercion(CoercionInputShape.Array, CoercionAction.Fail)
+                .setCoercion(CoercionInputShape.Object, CoercionAction.Fail)
+        }
         .withCoercionConfig(LogicalType.Array) {
             it.setCoercion(CoercionInputShape.Boolean, CoercionAction.Fail)
                 .setCoercion(CoercionInputShape.Integer, CoercionAction.Fail)
@@ -124,10 +131,10 @@ private object InputStreamSerializer : BaseSerializer<InputStream>(InputStream::
 }
 
 /**
- * A deserializer that can deserialize [LocalDateTime] from datetimes, dates, and zoned datetimes.
+ * A deserializer that can deserialize [OffsetDateTime] from datetimes, dates, and zoned datetimes.
  */
-private class LenientLocalDateTimeDeserializer :
-    StdDeserializer<LocalDateTime>(LocalDateTime::class.java) {
+private class LenientOffsetDateTimeDeserializer :
+    StdDeserializer<OffsetDateTime>(OffsetDateTime::class.java) {
 
     companion object {
 
@@ -141,26 +148,28 @@ private class LenientLocalDateTimeDeserializer :
 
     override fun logicalType(): LogicalType = LogicalType.DateTime
 
-    override fun deserialize(p: JsonParser, context: DeserializationContext?): LocalDateTime {
+    override fun deserialize(p: JsonParser, context: DeserializationContext): OffsetDateTime {
         val exceptions = mutableListOf<Exception>()
 
         for (formatter in DATE_TIME_FORMATTERS) {
             try {
                 val temporal = formatter.parse(p.text)
 
                 return when {
-                    !temporal.isSupported(ChronoField.HOUR_OF_DAY) ->
-                        LocalDate.from(temporal).atStartOfDay()
-                    !temporal.isSupported(ChronoField.OFFSET_SECONDS) ->
-                        LocalDateTime.from(temporal)
-                    else -> ZonedDateTime.from(temporal).toLocalDateTime()
-                }
+                        !temporal.isSupported(ChronoField.HOUR_OF_DAY) ->
+                            LocalDate.from(temporal).atStartOfDay()
+                        !temporal.isSupported(ChronoField.OFFSET_SECONDS) ->
+                            LocalDateTime.from(temporal)
+                        else -> ZonedDateTime.from(temporal).toLocalDateTime()
+                    }
+                    .atZone(context.timeZone.toZoneId())
+                    .toOffsetDateTime()
             } catch (e: DateTimeException) {
                 exceptions.add(e)
             }
         }
 
-        throw JsonParseException(p, "Cannot parse `LocalDateTime` from value: ${p.text}").apply {
+        throw JsonParseException(p, "Cannot parse `OffsetDateTime` from value: ${p.text}").apply {
             exceptions.forEach { addSuppressed(it) }
         }
     }
diff --git a/finch-java-core/src/main/kotlin/com/tryfinch/api/models/Introspection.kt b/finch-java-core/src/main/kotlin/com/tryfinch/api/models/Introspection.kt
@@ -1465,7 +1465,7 @@ private constructor(
                             .toList()
                     return when (bestMatches.size) {
                         // This can happen if what we're deserializing is completely incompatible
-                        // with all the possible variants (e.g. deserializing from object).
+                        // with all the possible variants (e.g. deserializing from boolean).
                         0 -> LastSuccessfulSync(_json = json)
                         1 -> bestMatches.single()
                         // If there's more than one match with the highest validity, then use the
@@ -2429,7 +2429,7 @@ private constructor(
                         return when (bestMatches.size) {
                             // This can happen if what we're deserializing is completely
                             // incompatible with all the possible variants (e.g. deserializing from
-                            // object).
+                            // boolean).
                             0 -> LastSuccessfulSync(_json = json)
                             1 -> bestMatches.single()
                             // If there's more than one match with the highest validity, then use
diff --git a/finch-java-core/src/main/kotlin/com/tryfinch/api/models/RequestForwardingForwardResponse.kt b/finch-java-core/src/main/kotlin/com/tryfinch/api/models/RequestForwardingForwardResponse.kt
@@ -746,7 +746,7 @@ private constructor(
                             .toList()
                     return when (bestMatches.size) {
                         // This can happen if what we're deserializing is completely incompatible
-                        // with all the possible variants (e.g. deserializing from array).
+                        // with all the possible variants (e.g. deserializing from boolean).
                         0 -> Data(_json = json)
                         1 -> bestMatches.single()
                         // If there's more than one match with the highest validity, then use the
diff --git a/finch-java-core/src/test/kotlin/com/tryfinch/api/core/ObjectMappersTest.kt b/finch-java-core/src/test/kotlin/com/tryfinch/api/core/ObjectMappersTest.kt
@@ -3,7 +3,7 @@ package com.tryfinch.api.core
 import com.fasterxml.jackson.annotation.JsonProperty
 import com.fasterxml.jackson.databind.exc.MismatchedInputException
 import com.fasterxml.jackson.module.kotlin.readValue
-import java.time.LocalDateTime
+import java.time.OffsetDateTime
 import kotlin.reflect.KClass
 import org.assertj.core.api.Assertions.assertThat
 import org.assertj.core.api.Assertions.catchThrowable
@@ -58,14 +58,6 @@ internal class ObjectMappersTest {
                     LONG to DOUBLE,
                     LONG to INTEGER,
                     CLASS to MAP,
-                    // These aren't actually valid, but coercion configs don't work for String until
-                    // v2.14.0: https://github.com/FasterXML/jackson-databind/issues/3240
-                    // We currently test on v2.13.4.
-                    BOOLEAN to STRING,
-                    FLOAT to STRING,
-                    DOUBLE to STRING,
-                    INTEGER to STRING,
-                    LONG to STRING,
                 )
         }
     }
@@ -84,7 +76,7 @@ internal class ObjectMappersTest {
         }
     }
 
-    enum class LenientLocalDateTimeTestCase(val string: String) {
+    enum class LenientOffsetDateTimeTestCase(val string: String) {
         DATE("1998-04-21"),
         DATE_TIME("1998-04-21T04:00:00"),
         ZONED_DATE_TIME_1("1998-04-21T04:00:00+03:00"),
@@ -93,10 +85,10 @@ internal class ObjectMappersTest {
 
     @ParameterizedTest
     @EnumSource
-    fun readLocalDateTime_lenient(testCase: LenientLocalDateTimeTestCase) {
+    fun readOffsetDateTime_lenient(testCase: LenientOffsetDateTimeTestCase) {
         val jsonMapper = jsonMapper()
         val json = jsonMapper.writeValueAsString(testCase.string)
 
-        assertDoesNotThrow { jsonMapper().readValue<LocalDateTime>(json) }
+        assertDoesNotThrow { jsonMapper().readValue<OffsetDateTime>(json) }
     }
 }
diff --git a/finch-java-proguard-test/build.gradle.kts b/finch-java-proguard-test/build.gradle.kts
@@ -19,7 +19,7 @@ dependencies {
     testImplementation(kotlin("test"))
     testImplementation("org.junit.jupiter:junit-jupiter-api:5.9.3")
     testImplementation("org.assertj:assertj-core:3.25.3")
-    testImplementation("com.fasterxml.jackson.module:jackson-module-kotlin:2.13.4")
+    testImplementation("com.fasterxml.jackson.module:jackson-module-kotlin:2.14.0")
 }
 
 tasks.shadowJar {

Original file line number	Diff line number	Diff line change
`@@ -19,7 +19,7 @@ dependencies {`
`19`	`19`	`testImplementation(kotlin("test"))`
`20`	`20`	`testImplementation("org.junit.jupiter:junit-jupiter-api:5.9.3")`
`21`	`21`	`testImplementation("org.assertj:assertj-core:3.25.3")`
`22`		`- testImplementation("com.fasterxml.jackson.module:jackson-module-kotlin:2.13.4")`
	`22`	`+ testImplementation("com.fasterxml.jackson.module:jackson-module-kotlin:2.14.0")`
`23`	`23`	`}`
`24`	`24`
`25`	`25`	`tasks.shadowJar {`