-
Notifications
You must be signed in to change notification settings - Fork 40.8k
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
Configure Couchbase to use the application's ObjectMapper #24616
Comments
On the surface, this seems like a good suggestion, however, there's a downside. If we auto-configure the There's also some interesting logic regarding @micopiira Can you explain a bit more about why you're plugging in the |
@philwebb The default CouchbaseAutoConfiguration does not support serializing Java 8/JSR310 types (Optional, Instant, etc.) out of the box, because the DefaultJsonSerializers ObjectMapper does not register Jdk8Module or JavaTimeModule. Ofcourse I can configure this myself, but I would expect a Spring Boot application to support this out of the box. And regarding your earlier comment of opting-out, I guess users could always call |
Thanks @micopiira, that's useful feedback. |
Now that we know that it's easy to restore Couchbase's defaults (thanks for the hint, @micopiira), I think it makes sense for us to configure Couchbase to use Boot's |
Could the CouchbaseAutoConfiguration use existing Jackson ObjectMapper bean instance by default?
Maybe add something like this to the CouchbaseAutoConfiguration class?
The text was updated successfully, but these errors were encountered: