Looks like missing Object id was acceptable already, as per tests.

In the ObjectIdValueProperty class, the check you added to the deserializeSetAndReturn function:

if (id == null) {
    return null;
}

Is positioned after the Object id = _valueDeserializer.deserialize(p, ctxt);.

Could it be that it should be positioned before the deserialization? If we don't change it, we have exceptions like this one:
JsonMappingException: Can not deserialize instance of java.lang.String out of VALUE_NULL
when we send a JSON with a null id, with this annotation:
JsonIdentityInfo(generator = ObjectIdGenerators.PropertyGenerator.class, property = "id")

Or are we simply doing something wrong here?

Thanks