Skip to content

Conversation

@waa-ben
Copy link

@waa-ben waa-ben commented Jan 15, 2026

It'd be exceptionally unlikely to be hit, but this will clear the CVE from this project. See CVE-2025-68161 for details.

<dependency prefix="spotbugs.log4j-slf4j18-impl" artifact="org.apache.logging.log4j:log4j-slf4j18-impl:2.24.3" usage="${spotbugs.lib}"/>
<dependency prefix="spotbugs.log4j-api" artifact="org.apache.logging.log4j:log4j-api:2.25.3" usage="${spotbugs.lib}"/>
<dependency prefix="spotbugs.log4j-core" artifact="org.apache.logging.log4j:log4j-core:2.25.3" usage="${spotbugs.lib}"/>
<dependency prefix="spotbugs.log4j-slf4j18-impl" artifact="org.apache.logging.log4j:log4j-slf4j18-impl:2.25.3" usage="${spotbugs.lib}"/>
Copy link
Author

@waa-ben waa-ben Jan 15, 2026

Choose a reason for hiding this comment

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

I'm not familiar with spotbugs, but this line appears to be be incorrect as:

  1. Those Maven coordinates aren't correct (before or after this change), the log4j slf4j 1.8 binding library caps out at version 2.18.0 as far as I can see
  2. I can't see this in the dependency tree at all, at least in the gradle build
    I can see in the past this version seems to be in lock-step with the log4j version though, so I've included it here.

Let me know if you'd rather references to this binding be removed, or if there's something I've missed with it's inclusion

@pjfanning
Copy link
Member

xmlbeans only has a dependency on log4j-api. No CVEs in log4j-api. There are some in log4j-core but we don't force you to add that jar and you can control the dependency version in your build files.

I'll probably merge this or something similar but we have no current plans for any xmlbeans releases.

@waa-ben
Copy link
Author

waa-ben commented Jan 16, 2026

I agree that it shouldn't apply, but they've unfortunately logged it under the CPE cpe:2.3:a:apache:log4j:*:*:*:*:*:*:*:* which my vulnerability scanner interprets as applying to all log4j packages. Not a huge deal as we can override it, of course.

We're currently blocked from upgrading because of XML-660 (thanks for merging that by the way), is there any chance getting a release on to the roadmap? I'm happy to donate some time to the process if there's some work that needs to be done, our customers' auditors are much happier with official releases than self-published forks, even if we're not making code changes outside of what's been accepted here.

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