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-main] Wiremocked tests do not work after upgrade of jetty #5592

Closed
JiriOndrusek opened this issue Dec 6, 2023 · 2 comments
Closed
Assignees

Comments

@JiriOndrusek
Copy link
Contributor

Bug description

Camel brought an update of jetty which caiuses following error (for each integration test using wiremock)

2023-12-06T08:58:43.7966773Z Caused by: java.lang.IncompatibleClassChangeError: class org.eclipse.jetty.http2.server.HttpChannelOverHTTP2 has interface org.eclipse.jetty.server.HttpChannel as super class
2023-12-06T08:58:43.7968686Z 	at java.base/java.lang.ClassLoader.defineClass1(Native Method)
2023-12-06T08:58:43.7969824Z 	at java.base/java.lang.ClassLoader.defineClass(ClassLoader.java:1017)
2023-12-06T08:58:43.7971156Z 	at io.quarkus.bootstrap.classloading.QuarkusClassLoader.loadClass(QuarkusClassLoader.java:508)
2023-12-06T08:58:43.7972719Z 	at io.quarkus.bootstrap.classloading.QuarkusClassLoader.loadClass(QuarkusClassLoader.java:468)
2023-12-06T08:58:43.7974570Z 	at org.eclipse.jetty.http2.server.AbstractHTTP2ServerConnectionFactory.<init>(AbstractHTTP2ServerConnectionFactory.java:80)
2023-12-06T08:58:43.7976507Z 	at org.eclipse.jetty.http2.server.HTTP2ServerConnectionFactory.<init>(HTTP2ServerConnectionFactory.java:53)
2023-12-06T08:58:43.7978293Z 	at org.eclipse.jetty.http2.server.HTTP2CServerConnectionFactory.<init>(HTTP2CServerConnectionFactory.java:53)
2023-12-06T08:58:43.7980096Z 	at org.eclipse.jetty.http2.server.HTTP2CServerConnectionFactory.<init>(HTTP2CServerConnectionFactory.java:48)
2023-12-06T08:58:43.7981934Z 	at com.github.tomakehurst.wiremock.jetty11.Jetty11HttpServer.createHttpConnector(Jetty11HttpServer.java:53)
2023-12-06T08:58:43.7983866Z 	at com.github.tomakehurst.wiremock.jetty.JettyHttpServer.<init>(JettyHttpServer.java:89)
2023-12-06T08:58:43.7985366Z 	at com.github.tomakehurst.wiremock.jetty11.Jetty11HttpServer.<init>(Jetty11HttpServer.java:44)
2023-12-06T08:58:43.7987100Z 	at com.github.tomakehurst.wiremock.jetty.JettyHttpServerFactory.buildHttpServer(JettyHttpServerFactory.java:31)
2023-12-06T08:58:43.7988903Z 	at com.github.tomakehurst.wiremock.WireMockServer.<init>(WireMockServer.java:75)
@JiriOndrusek JiriOndrusek added the bug Something isn't working label Dec 6, 2023
@JiriOndrusek JiriOndrusek changed the title [camel-main] Wiremocked tests does not work after upgrade of jetty [camel-main] Wiremocked tests do not work after upgrade of jetty Dec 6, 2023
@jamesnetherton
Copy link
Contributor

jamesnetherton commented Dec 6, 2023

Not sure how best to handle this. WireMock is tied to Jetty 11.x. There's a couple of open issues for upgrading Jetty:

wiremock/wiremock#2513
wiremock/wiremock#2395

I also noticed this thing has been added to the BOM, which is probably wrong and needs investigating:

https://github.com/apache/camel-quarkus/blob/camel-main/poms/bom/src/main/generated/flattened-reduced-pom.xml#L7730-L7734

@jamesnetherton
Copy link
Contributor

This is fixed by #5593 & 0a7c928.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants