@SlyngDK please review.

@SlyngDK can we release this since it is an important fix?

@IvanPuntev
I have looked into this and thinks, the default should be to use the default provided ObjectMapper.
And maybe create support for provide a custom ObjectMapper for logging.

@SlyngDK How do you propose to add the support for a custom object mapper? Currently the extension hardcodes the jackson factory in its processor at build time - https://github.com/quarkiverse/quarkus-logging-json/blob/main/deployment/src/main/java/io/quarkiverse/loggingjson/deployment/LoggingJsonProcessor.java#L53
There are no CDI injection, config or runtime hook.

Hi @IvanPuntev do you know when the above changes will be merged?

Hi @IvanPuntev do you know when the above changes will be merged?

I'm preparing a better solution and will commit it soon. After review it will be merged.

The new changes are here - #395

Closing in favor of #395