Skip to content

Commit 351b754

Browse files
committed
Change default setting wrt #1341 for 2.8.x (to enabled); may change once we switch master to be 2.9.0-SNAPSHOT
1 parent 44bc080 commit 351b754

File tree

1 file changed

+9
-6
lines changed

1 file changed

+9
-6
lines changed

src/main/java/com/fasterxml/jackson/databind/DeserializationFeature.java

+9-6
Original file line numberDiff line numberDiff line change
@@ -246,14 +246,17 @@ public enum DeserializationFeature implements ConfigFeature
246246

247247
/**
248248
* Feature that determines what happens when a property annotated with
249-
* {@link com.fasterxml.jackson.annotation.JsonTypeInfo.As#EXTERNAL_PROPERTY} is missing.
250-
* This is disabled by default, so that no error is thrown when a subtype property is
251-
* missing, unless the property is explicitly marked as `required`. If it is enabled, or
252-
* the property is marked as 'required' then a {@link JsonMappingException} will be thrown.
249+
* {@link com.fasterxml.jackson.annotation.JsonTypeInfo.As#EXTERNAL_PROPERTY} is missing,
250+
* but associated type id is available. If enabled, {@link JsonMappingException} is always
251+
* thrown when property value is missing (if type id does exist);
252+
* if disabled, exception is only thrown if property is marked as `required`.
253+
*<p>
254+
* Feature is enabled by default, so that exception is thrown when a subtype property is
255+
* missing.
253256
*
254-
* @since 2.8
257+
* @since 2.9
255258
*/
256-
FAIL_ON_EXTERNAL_TYPE_ID_MISSING_PROPERTY(false),
259+
FAIL_ON_EXTERNAL_TYPE_ID_MISSING_PROPERTY(true),
257260

258261
/**
259262
* Feature that determines whether Jackson code should catch

0 commit comments

Comments
 (0)