Croway commented on code in PR #12208:
URL: https://github.com/apache/camel/pull/12208#discussion_r1410472437
##########
components/camel-cxf/camel-cxf-rest/pom.xml:
##########
@@ -126,43 +126,30 @@
<dependency>
<groupId>org.apache.cxf</groupId>
- <artifactId>cxf-rt-transports-http-jetty</artifactId>
+ <artifactId>cxf-rt-transports-http-undertow</artifactId>
Review Comment:
in CXF, `cxf-rt-transports-http-jetty` is used for testing, and
`cxf-rt-transports-http-jetty` is still using Jetty 11, therefore, during
runtime, due to the exclusion and the usage of camel-jetty (that is using jetty
12) I had some ClassNotFound/MethodNotFound.
This is why I replaced (for testing purposes) jetty with undertow, I totally
understand the risk, once merged, I'll execute some camel spring boot and camel
quarkus test and examples to be sure that it is working as expected, as you can
see by the camel test report, there are no test failures in cxf.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
