Skip to content

fix(fp): suppress FPs for log4j-core and log4j-layout-template-json#8459

Merged
jeremylong merged 1 commit intodependency-check:generatedSuppressionsfrom
chadlwilson:improve-log4j-suppressions
Apr 29, 2026
Merged

fix(fp): suppress FPs for log4j-core and log4j-layout-template-json#8459
jeremylong merged 1 commit intodependency-check:generatedSuppressionsfrom
chadlwilson:improve-log4j-suppressions

Conversation

@chadlwilson
Copy link
Copy Markdown
Collaborator

@chadlwilson chadlwilson commented Apr 27, 2026

Description of Change

Migrates and consolidates the CVE-by-CVE suppressions from below into generatedSuppressions for ease of maintenance.

I guess due to the prevalence of log4j we do maintain some CVE-by-CVE suppressions here, but probably not going to be scalable longer term I suppose.

<suppress base="true">
<notes><![CDATA[
only log4j-core is vulnerable
]]></notes>
<packageUrl regex="true">^pkg:maven/org\.apache\.logging\.log4j/log4j\-(api|web)@.*$</packageUrl>
<cve>CVE-2021-44228</cve>
<cve>CVE-2021-44832</cve>
<cve>CVE-2021-45046</cve>
<cve>CVE-2021-45105</cve>
</suppress>

Suppress some marked CVEs against all log4j pkgs other than log4j-core (and log4j-layout-template-json for one of them)

NVD search

I'll remove from base suppressions after this, since it will become duplicated.

Related issues

Have test cases been added to cover the new functionality?

N/A

@chadlwilson chadlwilson force-pushed the improve-log4j-suppressions branch from 777aa53 to 376f1cd Compare April 27, 2026 06:34
Copy link
Copy Markdown
Collaborator

@marcelstoer marcelstoer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@jeremylong jeremylong added this to the 12.2.2 milestone Apr 29, 2026
@jeremylong jeremylong merged commit f3dfec3 into dependency-check:generatedSuppressions Apr 29, 2026
2 checks passed
@chadlwilson chadlwilson deleted the improve-log4j-suppressions branch April 29, 2026 12:49
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