Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

CAMEL-19713: Fix logging by ensuring use of SLF4J 2.0 #13916

Merged
merged 1 commit into from
Apr 24, 2024
Merged

Conversation

klease
Copy link
Contributor

@klease klease commented Apr 24, 2024

The org.wildfly.security:wildfly-elytron jar dependency of camel-elytron includes the SLF4J 1.x classes and there was a logback provider in the classpath.

Description

Target

  • I checked that the commit is targeting the correct branch (note that Camel 3 uses camel-3.x, whereas Camel 4 uses the main branch)

Tracking

  • If this is a large change, bug fix, or code improvement, I checked there is a JIRA issue filed for the change (usually before you start working on it).

Apache Camel coding standards and style

  • I checked that each commit in the pull request has a meaningful subject line and body.
  • I have run mvn clean install -DskipTests locally and I have committed all auto-generated changes

The org.wildfly.security:wildfly-elytron jar dependency of camel-elytron includes
the SLF4J 1.x classes and there was a logback provider in the classpath.
Copy link
Contributor

@orpiske orpiske left a comment

Choose a reason for hiding this comment

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

Thank you!

@klease klease merged commit 66ae838 into main Apr 24, 2024
2 of 4 checks passed
@klease klease deleted the CAMEL-19713 branch April 24, 2024 12:39
4nt01ne pushed a commit to 4nt01ne/camel that referenced this pull request May 14, 2024
The org.wildfly.security:wildfly-elytron jar dependency of camel-elytron includes
the SLF4J 1.x classes and there was a logback provider in the classpath.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants