Fix spring-ws test failures by adding log4j-jcl bridge#22335
Merged
Croway merged 2 commits intocamel-4.14.xfrom Mar 30, 2026
Merged
Fix spring-ws test failures by adding log4j-jcl bridge#22335Croway merged 2 commits intocamel-4.14.xfrom
Croway merged 2 commits intocamel-4.14.xfrom
Conversation
The spring-ws component tests were failing with ~296 failures due to: Could not initialize class org.apache.commons.logging.LogAdapter$Log4jLog Root cause: The httpclient dependency (used in tests) brings commons-logging transitively, but the JCL-to-Log4j2 bridge (log4j-jcl) was missing from the test classpath. Without this bridge, commons-logging cannot initialize when Log4j2 is present. This fix adds log4j-jcl as a test dependency. The version is managed in the parent pom.xml, so no explicit version is needed.
oscerd
approved these changes
Mar 30, 2026
…idge The logger entry for MessageTracing used Log4j 1.x syntax which caused ConfigurationException when log4j-jcl routes JCL through Log4j2 core. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Contributor
Author
|
failure is unrelated |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
This PR fixes ~296 test failures in the
camel-spring-wscomponent caused by a missing JCL-to-Log4j2 bridge dependency.Problem
Tests were failing with:
Root Cause
httpclientdependency (used in tests) bringscommons-loggingtransitivelylog4j-jcl) was missing from the test classpathSolution
Added
log4j-jclas a test-scoped dependency tocamel-spring-ws/pom.xml. The dependency version is already managed in the parentpom.xml, so no explicit version declaration is needed.Testing
This fix should resolve all ~296 test failures in the spring-ws component that were occurring due to the LogAdapter initialization error.