-
Notifications
You must be signed in to change notification settings - Fork 4.9k
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-18916] camel-xslt-saxon: allow logger injection #10488
Conversation
🌟 Thank you for your contribution to the Apache Camel project! 🌟 🐫 Maintainers, please note that first-time contributors require manual approval for the GitHub Actions to run. 🤖 Use the command If necessary Apache Camel Committers may access logs and test results in the job summaries! |
a48e2df
to
9e5726d
Compare
Can we use another name than messageConsumer - that smells like EIPish (camel message and consumer). Its a logging listener or something. |
components/camel-xslt/src/main/java/org/apache/camel/component/xslt/XsltBuilder.java
Outdated
Show resolved
Hide resolved
.../camel-xslt-saxon/src/main/java/org/apache/camel/component/xslt/saxon/XsltSaxonEndpoint.java
Outdated
Show resolved
Hide resolved
...s/camel-xslt-saxon/src/main/java/org/apache/camel/component/xslt/saxon/XsltSaxonBuilder.java
Outdated
Show resolved
Hide resolved
/component-test xslt xslt-saxon Result ✅ The tests passed successfully |
🤖 The Apache Camel test robot will run the tests for you 👍 |
Description
camel-xslt-saxon: allow logger injection
Target
camel-3.x
, whereas Camel 4 uses themain
branch)Tracking
Apache Camel coding standards and style
mvn clean install -DskipTests
locally and I have committed all auto-generated changes