Skip to content

Upgrade jackson-databind to 2.12.7#14770

Merged
abhishekagarwal87 merged 5 commits intoapache:masterfrom
tejaswini-imply:upgrade-jackson-databind
Aug 9, 2023
Merged

Upgrade jackson-databind to 2.12.7#14770
abhishekagarwal87 merged 5 commits intoapache:masterfrom
tejaswini-imply:upgrade-jackson-databind

Conversation

@tejaswini-imply
Copy link
Copy Markdown
Member

@tejaswini-imply tejaswini-imply commented Aug 7, 2023

The current version of jackson-databind is flagged for vulnerabilities CVE-2020-28491 (Although cbor format is not used in druid), CVE-2020-36518 (Seems genuine as deeply nested json in can cause resource exhaustion). Updating the dependency to the latest version 2.12.7 to fix these vulnerabilities.

Now that Hadoop2 is being removed, this upgrade is deemed to be safely made.

} else if (m instanceof AnnotatedMethod) {
genericType = ((AnnotatedMethod) m).getAnnotated().getGenericReturnType();
} else if (m instanceof AnnotatedParameter) {
genericType = ((AnnotatedParameter) m).getOwner().getGenericParameterType(((AnnotatedParameter) m).getIndex());

Check notice

Code scanning / CodeQL

Deprecated method or constructor invocation

Invoking [AnnotatedWithParams.getGenericParameterType](1) should be avoided because it has been deprecated.
@abhishekagarwal87
Copy link
Copy Markdown
Contributor

FYI anyone reading this PR, we cannot upgrade to a version beyond 2.12 because of this change FasterXML/jackson-jaxrs-providers#134
If we want to upgrade to 2.15, one option is to copy the source code of javax.ws.rs.core.NoContentException

@abhishekagarwal87 abhishekagarwal87 merged commit 550a66d into apache:master Aug 9, 2023
@LakshSingla LakshSingla added this to the 28.0 milestone Oct 12, 2023
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.

4 participants