Skip to content

KAFKA-20270#21660

Open
averemee-si wants to merge 4 commits intoapache:trunkfrom
averemee-si:KAFKA-20270
Open

KAFKA-20270#21660
averemee-si wants to merge 4 commits intoapache:trunkfrom
averemee-si:KAFKA-20270

Conversation

@averemee-si
Copy link
Contributor

Fix jetty-server vulnerability CVE-2026-1605

Updated Jetty version to 12.0.32 to fix CVE-2026-1605
Updated Jetty dependencies to version 12.0.32.
@github-actions github-actions bot added triage PRs from the community dependencies Pull requests that update a dependency file build Gradle build or GitHub Actions small Small PRs labels Mar 6, 2026
// 12.0.25 is the version that includes the CVE-2025-5115 fix while only using the
// SLF4J 2.x fluent API in test code, avoiding the runtime incompatibility.
jetty: "12.0.25",
jetty: "12.0.32",
Copy link
Member

Choose a reason for hiding this comment

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

See the comment above. We had to downgrade from 12.0.32 in #21559 because of an incompatibility with slf4j 2.X

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Thanks for the clarification.
Interestingly, my Connect Worker, in which I replaced version 12.0.25 with 12.0.33, has been running without problems (including REST API) since Friday and has already transferred two terabytes of changes.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

There are 707 source files dependent on org.slf4j (1210 entries). They depend on

org.slf4j.Logger;
org.slf4j.LoggerFactory;
org.slf4j.MDC;
org.slf4j.Marker;
org.slf4j.MarkerFactory;
org.slf4j.event.Level;
org.slf4j.helpers.FormattingTuple;
org.slf4j.helpers.MessageFormatter;
org.slf4j.spi.LocationAwareLogger;

test classes also contains

assertFalse(PluginUtils.shouldLoadInIsolation("org.slf4j."));
assertFalse(PluginUtils.shouldLoadInIsolation("org.slf4j.LoggerFactory"));

I think upgrading slf4j to version 2.0.17 for the entire Kafka bundle is possible.

Working on this.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

The main cause of unit test failures is org.apache.kafka.common.utils.LogCaptureAppender, which behaves differently when using slf4j versions 1.7.36 and 2.0.17

Looking into it.

Copy link
Member

Choose a reason for hiding this comment

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

If you want to look into the slf4j dependency, I'd suggest you open a separate issue/PR. Note that other people are looking into it already #21452 (comment), so try syncing with them.

@github-actions github-actions bot removed the triage PRs from the community label Mar 10, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

build Gradle build or GitHub Actions dependencies Pull requests that update a dependency file small Small PRs

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants