Skip to content
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

4.1.3 - customized objectMapper #1165

Closed
AlonPorter opened this issue Jul 18, 2024 · 2 comments
Closed

4.1.3 - customized objectMapper #1165

AlonPorter opened this issue Jul 18, 2024 · 2 comments

Comments

@AlonPorter
Copy link

Describe the bug
After upgrading to 4.1.3 we got errors in our app regarding failing to parse some json.
after invetigation we understad that we want to customize the object mapper to have FAIL_ON_TRAILING_TOKEN false, but can't override this value.
Hope I explain good.
thanks in advance.

@GreenRover
Copy link
Contributor

GreenRover commented Aug 6, 2024

I confirm since the lates release it is not linger possible to to modify the ObjectMapper.

I would like to propose to use the object mapper created/used by spring boot instead of creating an dedicated one that can not be modified.

My actual business problem ist:

  • Dates (Instant) have to be serialized as ISO string but are currently as float Unix timestamp.
  • The same objects are differently serialized when transmitted via messaging compared to an http transferred.

As ugly workaround i now de/serialize everything outside of spring cloud stream, what means dont be able to use a key feature of spring cloud function.

In my opinion such changes are a no go for a minion release.
@olegz Was an intended change or can we consider this as a bug?

Or did some one find a way to overwrite:
ContextFunctionCatalogInitializer.ContextFunctionCatalogBeanRegistrar::postProcessBeanDefinitionRegistry

@olegz
Copy link
Contributor

olegz commented Aug 16, 2024

I believe this has been addressed - #1148
You can go back to configuring ObjectMapper the standard way.
Please file a new issue if the problem persists

@olegz olegz closed this as completed Aug 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants