Add release notes for #3103, minor tweaks over #3608

Author: cowtowncoder

release-notes/CREDITS-2.x

@@ -1485,6 +1485,8 @@ Igor Shymko (ancane@github)
   (2.14.0)
 
 Jordi Ortolá Ankum (Tomasito665@github)
+  * Contributed #3013: Allow disabling Integer to String coercion via `CoercionConfig`
+  (2.14.0)
   * Contributed #3503: `StdDeserializer` coerces ints to floats even if configured to fail
   (2.14.0)
 

release-notes/VERSION-2.x

@@ -11,6 +11,9 @@ Project: jackson-databind
 #2541: Cannot merge polymorphic objects
  (reported by Matthew A)
  (fix contributed by James W)
+#3013: Allow disabling Integer to String coercion via `CoercionConfig`
+ (reported by @emilkostadinov)
+ (fix contributed by Jordi O-A)
 #3311: Add serializer-cache size limit to avoid Metaspace issues from
   caching Serializers
  (requested by mcolemanNOW@github)

src/main/java/com/fasterxml/jackson/databind/deser/std/StdDeserializer.java

@@ -1366,10 +1366,35 @@ protected java.util.Date _parseDate(String value, DeserializationContext ctxt)
     }
 
     /**
-     * Helper method used for accessing String value, if possible, doing
+     * @since 2.1
+     * @deprecated Since 2.14 (use the non-deprecated overload)
+     */
+    @Deprecated
+    protected final String _parseString(JsonParser p, DeserializationContext ctxt)
+        throws IOException
+    {
+        return _parseString(p, ctxt,
+                // Note: could consider passing `this`, but since this methods is
+                // often called for values "inside" type we are handling (like
+                // String array or Collection of Strings) that would probably
+                // not match. So default to "nulls as nulls" handler.
+                NullsConstantProvider.nuller());
+    }
+
+    /**
+     * Helper method used for deserializing String value, if possible, doing
      * necessary conversion or throwing exception as necessary.
+     *<p>
+     * Starting with 2.14 will also start checking {@code CoercionConfig}
+     * configuration to check whether coercion needed is allowed.
      *
-     * @since 2.1
+     * @param p Currently active parser being iterated over
+     * @param ctxt Deserialization context
+     * @param nullProvider Entity we (only) need for case of secondary value
+     *    type being coerced into {@code null}: if so, provider is asked for
+     *    possible "null replacement" value.
+     *
+     * @since 2.14 (with this signature)
      */
     protected final String _parseString(JsonParser p, DeserializationContext ctxt,
             NullValueProvider nullProvider)
@@ -1413,7 +1438,7 @@ protected final String _parseString(JsonParser p, DeserializationContext ctxt,
                 return text;
             }
         }
-        return (String) ctxt.handleUnexpectedToken(String.class, p);
+        return (String) ctxt.handleUnexpectedToken(getValueType(ctxt), p);
     }
 
     /**

src/main/java/com/fasterxml/jackson/databind/deser/std/StringArrayDeserializer.java

@@ -44,7 +44,7 @@ public final class StringArrayDeserializer
     protected JsonDeserializer<String> _elementDeserializer;
 
     /**
-     * Handler we need for dealing with nulls.
+     * Handler we need for dealing with null values as elements
      *
      * @since 2.9
      */

src/main/java/com/fasterxml/jackson/databind/deser/std/StringDeserializer.java

@@ -5,7 +5,6 @@
 import com.fasterxml.jackson.core.*;
 import com.fasterxml.jackson.databind.*;
 import com.fasterxml.jackson.databind.annotation.JacksonStdImpl;
-import com.fasterxml.jackson.databind.cfg.CoercionAction;
 import com.fasterxml.jackson.databind.jsontype.TypeDeserializer;
 import com.fasterxml.jackson.databind.type.LogicalType;
 
@@ -38,6 +37,7 @@ public Object getEmptyValue(DeserializationContext ctxt) throws JsonMappingExcep
     @Override
     public String deserialize(JsonParser p, DeserializationContext ctxt) throws IOException
     {
+        // The critical path: ensure we handle the common case first.
         if (p.hasToken(JsonToken.VALUE_STRING)) {
             return p.getText();
         }

src/test/java/com/fasterxml/jackson/databind/convert/CoerceIntToStringTest.java

@@ -9,6 +9,7 @@
 import com.fasterxml.jackson.databind.exc.MismatchedInputException;
 import com.fasterxml.jackson.databind.type.LogicalType;
 
+// [databind#3013] / PR #3608
 public class CoerceIntToStringTest extends BaseMapTest
 {
     private final ObjectMapper DEFAULT_MAPPER = newJsonMapper();