-
-
Notifications
You must be signed in to change notification settings - Fork 136
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
Ion Polymorphic deserialization in 2.12 breaks wrt use of Native Type Ids when upgrading from 2.8 #270
Comments
manaigrn-amzn
added a commit
to manaigrn-amzn/jackson-dataformats-binary
that referenced
this issue
Apr 1, 2021
manaigrn-amzn
added a commit
to manaigrn-amzn/jackson-dataformats-binary
that referenced
this issue
Apr 1, 2021
manaigrn-amzn
added a commit
to manaigrn-amzn/jackson-dataformats-binary
that referenced
this issue
Apr 1, 2021
manaigrn-amzn
added a commit
to manaigrn-amzn/jackson-dataformats-binary
that referenced
this issue
Apr 1, 2021
manaigrn-amzn
added a commit
to manaigrn-amzn/jackson-dataformats-binary
that referenced
this issue
Apr 1, 2021
cowtowncoder
pushed a commit
that referenced
this issue
Apr 6, 2021
Solved by addition of:
|
cowtowncoder
changed the title
[Jackson-dataformat-ion - 2.12x] Polymorphic deserialization breaks when upgrading from 2.8
Ion Polymorphic deserialization in 2.12 breaks wrt use of Native Type Ids when upgrading from 2.8
Apr 6, 2021
cowtowncoder
added a commit
that referenced
this issue
Apr 6, 2021
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Given the following classes:
The following ion can be deserialized using Jackson-dataformat-ion 2.8:
Current Behavior
When using Jackson-dataformat-ion 2.12.x, the following exception is thrown:
This happens because IonParser.canReadTypeId() always returns true, it makes Jackson-databind incorrectly read the field name token
type::
as a native type id.Expected Behavior
There should not be an exception. This could be accomplished by allowing the check for native type ids to be disabled in the
IonParser
while deserializing, similar to what was added in theIonGenerator
in #232 during serialization.Then, the IonMapper could be configured with the following:
Note: disabling
IonGenerator
.Feature.USE_NATIVE_TYPE_ID
(which is already available) doesn't do the trick here.The text was updated successfully, but these errors were encountered: