-
-
Notifications
You must be signed in to change notification settings - Fork 1.4k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Make BeanDeserializerModifier
/BeanSerializerModifier
implement java.io.Serializable
#4209
Comments
BeanDeserializerModifier
implement java.io.Serializable
Hmmh. Interesting. Yes, at least for Jackson 2.x this would make sense. Same wrt |
To ensure clarity, is the solution as straightforward as incorporating |
@mukham12 I haven't had a look but I think that'd be it. So both |
BeanDeserializerModifier
implement java.io.Serializable
BeanDeserializerModifier
/BeanSerializerModifier
implement java.io.Serializable
Is your feature request related to a problem? Please describe.
The
BeanDeserializerModifier
does not currently implementSerializable
.jackson-databind/src/main/java/com/fasterxml/jackson/databind/deser/BeanDeserializerModifier.java
Line 48 in 5be890c
On the other hand, this class is used as a field in
SimpleModule
which implementsSerializable
.This means that if a simple implementation of
BeanDeserializerModifier
is used in aSimpleModule
, theJDK
serialization may cause unintended errors.jackson-databind/src/main/java/com/fasterxml/jackson/databind/module/SimpleModule.java
Line 94 in 5be890c
Describe the solution you'd like
Either have
BeanDeserializerModifier
implementSerializable
or provide a base class likeSimpleBeanDeserializerModifier
that implementsSerializable
as well as other supporting classes.Usage example
No response
Additional context
No response
The text was updated successfully, but these errors were encountered: