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

[DE-636] Non-interfering Jackson annotations #513

Merged
merged 2 commits into from
Oct 4, 2023

Conversation

rashtao
Copy link
Collaborator

@rashtao rashtao commented Jul 26, 2023

This PR aims to remove all usages of @JacksonAnnotationsInside annotations from serde annotation. The equivalent behavior should be implemented by extending JacksonAnnotationIntrospector in both Jackson Serde and Internal Serde.
In this way ArangoDB serde annotations will not be processed by other Jackson instances used in the application.


fixes #512

@cla-bot cla-bot bot added the cla-signed label Jul 26, 2023
@rashtao rashtao self-assigned this Jul 26, 2023
@sonarqubecloud
Copy link

SonarCloud Quality Gate failed.    Quality Gate failed

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

0.0% 0.0% Coverage
0.0% 0.0% Duplication

idea Catch issues before they fail your Quality Gate with our IDE extension sonarlint SonarLint

@rashtao rashtao changed the title Non-interfering Jackson annotations [DE-636] Non-interfering Jackson annotations Jul 26, 2023
@rashtao rashtao force-pushed the bugfix/non_interfering_jackson_annotations branch from 964af50 to 59b73ea Compare October 4, 2023 21:02
@rashtao rashtao marked this pull request as ready for review October 4, 2023 21:04
@sonarqubecloud
Copy link

sonarqubecloud bot commented Oct 4, 2023

SonarCloud Quality Gate failed.    Quality Gate failed

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

0.0% 0.0% Coverage
0.0% 0.0% Duplication

warning The version of Java (11.0.20.1) you have used to run this analysis is deprecated and we will stop accepting it soon. Please update to at least Java 17.
Read more here

idea Catch issues before they fail your Quality Gate with our IDE extension sonarlint SonarLint

@rashtao rashtao merged commit 05a639e into main Oct 4, 2023
@rashtao rashtao deleted the bugfix/non_interfering_jackson_annotations branch March 4, 2024 08:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Interfering Jackson annotations
1 participant