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

Set OracleJdbcJsonTextObjectMapper higher priority than OracleJdbcJsonBinaryObjectMapper #412

Merged
merged 1 commit into from
Apr 6, 2023

Conversation

radovanradic
Copy link
Contributor

In situation when we have these two oracle json mappers OracleJdbcJsonTextObjectMapper and OracleJdbcJsonBinaryObjectMapper on the classpath and without micronaut-serde-jackson which defines primary JsonMapper then app is not able to resolve JsonMapper injection in some of the classes in micronaut-data so we can help this way, setting OracleJdbcJsonTextObjectMapper higher priority than OracleJdbcJsonBinaryObjectMapper and then make it default JsonMapper.
If there is some better solution for this case, please let me know and I will update this. Without this, in apps that include serde-oracle-jdbc-json we will have issues when single JsonMapper needs to be injected.

@sonarqubecloud
Copy link

sonarqubecloud bot commented Apr 6, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@dstepanov dstepanov merged commit 8032272 into master Apr 6, 2023
@dstepanov dstepanov deleted the oracle-mapper-priority branch April 6, 2023 13:48
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

Successfully merging this pull request may close these issues.

2 participants