This repository was archived by the owner on Dec 4, 2023. It is now read-only.
Repository navigation
This repository was archived by the owner on Dec 4, 2023. It is now read-only.
com.microsoft.bot.schema.Serialization ObjectMapper is unable to serialize Java8 Date/Time with newest jackson. #1454
Description
Is your feature request related to a problem? Please describe.
Problem: newer Jackson libraries (e.g. v 2.12.x) are unable to deserialize Java 8 time and date objects because a JDK8 Module is not added in ObjectMapper at com.microsoft.bot.schema.Serialization, that ObjectMapper is tightly coupled and as a client of the library we're unable to add additional modules in order to avoid this issue,
On jackson 2.12 the serialization throws a
Specific warnings thrown by jackson libs <2.12
WARNING: An illegal reflective access operation has occurred
WARNING: Illegal reflective access by com.fasterxml.jackson.databind.util.ClassUtil (jar:file:/{ommited}!/BOOT-INF/lib/jackson-databind-2.11.3.jar!/) to field java.time.OffsetDateTime.offset
WARNING: Please consider reporting this to the maintainers of com.fasterxml.jackson.databind.util.ClassUtil
WARNING: Use --illegal-access=warn to enable warnings of further illegal reflective access operations
WARNING: All illegal access operations will be denied in a future release
Describe the solution you'd like
Register the modules into the object mapper with
objectMapper.registerModule(new Jdk8Module());
objectMapper.registerModule(new JavaTimeModule());
``
by either:
* Register these modules into com.microsoft.bot.schema.Serialization by the sdk
* exposing the objectmapper and allow sdk users to add the modules by their own by .
**Describe alternatives you've considered**
See above
**Additional context**
Tested with the following versions:
JDK: adoptopenddk 13 from public docker image adoptopenjdk/openjdk13:alpine-jre
Spring Version: 2.4.0
Jackson libraries: 2.11.3, 2.12.6. 2.13.1
bot-integration-spring && bot-dialogs: 4.14.1
Related stackoverflow: https://stackoverflow.com/questions/27952472/serialize-deserialize-java-8-java-time-with-jackson-json-mapper
Activity
tracyboehrer commentedon Feb 24, 2022
Erik Giron (@kreigiron) My first thought is that the additional modules are registered by way of inclusion in the POM's. For date and time, this is specifically the jackson-datatype-jsr310 module.
Of note also is that we've only fully tested on Java8. Though I do believe I've run it on 11 without issue. That particular "illegal reflective access" is a 9+ warning.
Have you tried this with JDK 8?
kreigiron commentedon Feb 24, 2022
tracyboehrer (@tracyboehrer) actually the JDK8Module is part of jackson-datatype-jsr310 so yes, we need to include that artifact, but it seems that we may need to manually add the JDK8Module into the mapper otherwise it won't register (at least that did not work for me).
I tried with JDK8 but due to GC constraints we need to run on java 13+, my current workaround is to downgrade jackson when running on that jvm version, but if we try to use jackson provided within newest Spring boot (2.12+) it won't work.
Thanks.
tracyboehrer commentedon Feb 24, 2022
Erik Giron (@kreigiron) Let us dig into it. With 8, those modules are just available after a call to objectMapper.findAndRegisterModules();.
TheBudgetMan commentedon Jun 13, 2022
We have also run into this issue.
Steps to reproduce:
.jarand run it with Java 11 (openjdk 11.0.2):java -jar app.jarWe get the following exception
Downgrading jackson or java is not an option for us unfortunately.
Have you got any updates on this issue, or some suggested workarounds?
TheBudgetMan commentedon Jun 13, 2022
tracyboehrer (@tracyboehrer) we did a little experimentation trying to correct the issue. We made it go away by instantiating
ObjectMapperlike this:This might be a superficial solution, as
findAndAddModulesshould pick it up (it did not).We did it first in
com.microsoft.bot.schema.Serialization(to solve the error above), but it turns out there are a lot of places whereObjectMapperis not instantiated like this, so other exceptions like this start appearing.tracyboehrer commentedon Jun 13, 2022
Thanks all. The warning initially reported happens when using Java 9+.
But the root problem is the inconsistent ObjectMapper initialization. Apparently, we neglected to resolve that. Without either the call to ObjectMapper.findAndRegisterModules, or explicitly adding the Java 8 Date/Time module, there would be problems with any payload that has a date/time property.
Or are you saying that findAndRegisterModules is also no longer working?
TheBudgetMan commentedon Jun 13, 2022
It seems so. When we tried using only
findAndAddModules, it did not work. Also, if I recall correctly,Serializationhas thefindAndRegisterModulescall in it.RitaFer commentedon Aug 15, 2023
I'm with this error in my teams bot, i've no ideia about what to do for fix. Even those samples don't work in my server.