Skip to content

Suppress CVE-2025-49128 for jackson-core shaded in hadoop-client-runtime#19105

Open
ashwintumma23 wants to merge 1 commit intoapache:masterfrom
ashwintumma23:jacksonSupp
Open

Suppress CVE-2025-49128 for jackson-core shaded in hadoop-client-runtime#19105
ashwintumma23 wants to merge 1 commit intoapache:masterfrom
ashwintumma23:jacksonSupp

Conversation

@ashwintumma23
Copy link
Contributor

Description

Adds an OWASP dependency-check suppression for CVE-2025-49128, which is flagged against the
jackson-core copy shaded inside hadoop-client-runtime-3.3.6.jar.

Why this is suppressed

  • Druid's standalone jackson-core is already at 2.19.2 (managed via jackson-bom), which is
    not affected by this CVE. The flag originates from an older jackson-core version shaded internally
    inside hadoop-client-runtime-3.3.6.jar. There is no fix available in the Hadoop 3.3.x line yet,
    making this unavoidable without a major Hadoop upgrade.

  • CVE-2025-52999 (also flagged against the same shaded copy) was already suppressed in the same
    block — this change brings CVE-2025-49128 in line with that existing suppression. It also fixes a typo in that entry.

Release note

  • Suppress CVE-2025-49128 for jackson-core shaded inside hadoop-client-runtime. Druid's own jackson-core is unaffected at 2.19.2.

Key changed/added classes in this PR
  • owasp-dependency-check-suppressions.xml: Added CVE-2025-49128 to the hadoop-client-runtime-3.3.6.jar
    suppression block, alongside fixing a typo the existing CVE-2025-52999 entry.

This PR has:

  • been self-reviewed.
  • added documentation for new or modified features or behaviors.
  • a release note entry in the PR description.
  • added Javadocs for most classes and all non-trivial methods. Linked related entities via Javadoc links.
  • added or updated version, license, or notice information in licenses.yaml
  • [] added comments explaining the "why" and the intent of the code wherever would not be obvious for an unfamiliar reader.
  • added unit tests or modified existing tests to cover new code paths, ensuring the threshold for code coverage is met.
  • added integration tests.
  • been tested in a test Druid cluster.

@abhishekrb19
Copy link
Contributor

Since hadoop ingestion is being removed in #19109, would this suppression be relevant?

@clintropolis
Copy link
Member

Since hadoop ingestion is being removed in #19109, would this suppression be relevant?

not sure, maybe since like the hdfs and other hadoopy extensions also probably contain this? I can always check by running the thing next time I have that branch checked out.

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.

3 participants