Collection deserializer should create empty collection instead of null (when input is null)

[pjfanning]

• fix added using 31efc10
• this is a temp workaround - Scala 3 has the same fix but implemented another way - Collection deserializer should create empty collection instead of null (when input is null)
• Jackson 2 workaround needed due to JsonDeserializer getNullValue does not match NullValueProvider jackson-databind#5056

[psliwa]

In my opinion this change is unsafe. With this change when DeserializationFeature.FAIL_ON_NULL_CREATOR_PROPERTIES is used and collection value is null then exception is not thrown (https://github.com/FasterXML/jackson-module-scala?tab=readme-ov-file#deserializationfeaturefail_on_null_creator_properties - this documentation is outdated). I would expect exception in such case would be thrown. It is impossible right now to configure jackson to old behaviour - to throw exception when collection value in json is missing or is null. It can be useful in few use cases - eg in my use case I would like to detect null values for collection for unexpected schema changes - it is better to detect such schema changes during deserialization (in api or database) and throw exception rather that use empty collection (which may cause invalid data passed to application logic).

Could this PR be re-considered and above concerns could be taken into account? In my opinion without any feature flag this change is a bug. For java collection behaviour is assigning null values for null json properties and FAIL_ON_NULL_CREATOR_PROPERTIES feature flag works.

[pjfanning]

I have no current plans to revert this. The change is more consistent with the Java collection deserialization.
Jackson is very configurable. It is very pluggable. It is one option to subclass the deserializer in this module and to modify the behaviour in the subclass.

[pjfanning]

@psliwa if you are interested in submitting a PR, I will review it - but the PR needs proof that the behaviour in this module for a Scala List differs from how jackson-databind deals with a Java List. And also, Scala developers tend to hate nulls.

Have you considered having an Option[List[T]] in your class so that you can check for null input (which should deserialize as None) and empty input (which should deserialize as Some(List.empty))

[pjfanning]

I've update the README.

[psliwa]

Thanks for your comments. It is easy to check how Java collections handing differs from Scala collections after this merge, I can provide later some snippet to show difference but preparing PR knowing it will be rejected doesn't make sense to me. Java collections handling honor FAIL_ON_NULL_CREATOR_PROPERTIES property, null is deserialized to null. Deserializing null to empty collection by default and without any configuration possibilities in my opinion is strange behaviour even in scala ecosystem. Adding Option[List[???]] also doesn't make sense in such cases because such field is not optional from application level - the best way would be to use FAIL_ON_NULL_CREATOR_PROPERTIES in order to force null check in convenient way. I am not sure how it works in Scala 3 (you mentioned this in PR description) so maybe I don't have full context and this is why for me this behaviour is strange and unsafe.

[pjfanning]

@psliwa I merged #733 and there is a 2.19.1-SNAPSHOT with this change

[psliwa]

Many thanks 🙏 Solution in this new PR looks like good compromise, thanks