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

Using quarkus-rest-client-reactive-jackson on rest-client-quickstart fails #16143

Closed
geoand opened this issue Mar 31, 2021 · 1 comment · Fixed by #16146
Closed

Using quarkus-rest-client-reactive-jackson on rest-client-quickstart fails #16143

geoand opened this issue Mar 31, 2021 · 1 comment · Fixed by #16146
Assignees
Labels
area/rest-client kind/bug Something isn't working
Milestone

Comments

@geoand
Copy link
Contributor

geoand commented Mar 31, 2021

Describe the bug

When taking https://github.com/quarkusio/quarkus-quickstarts/tree/main/rest-client-quickstart and using quarkus-resteasy-reactive-jackson and quarkus-rest-client-reactive-jackson instead of the original RESTEasy and rest-client dependencies (and also substituting @PathParam for @RestPath), then running the tests fails with:

2021-03-31 13:47:33,076 ERROR [org.jbo.res.rea.com.cor.AbstractResteasyReactiveContext] (vert.x-eventloop-thread-8) Request failed: javax.ws.rs.ProcessingException: port p must be in range 0 <= p <= 65535
        at org.jboss.resteasy.reactive.client.impl.InvocationBuilderImpl.unwrap(InvocationBuilderImpl.java:207)
        at org.jboss.resteasy.reactive.client.impl.InvocationBuilderImpl.method(InvocationBuilderImpl.java:313)
        at org.acme.rest.client.CountriesService$$QuarkusRestClientInterface.getByName(CountriesService$$QuarkusRestClientInterface.zig:283)
        at org.acme.rest.client.CountriesService$$CDIWrapper.getByName(CountriesService$$CDIWrapper.zig:49)
        at org.acme.rest.client.CountriesService$$CDIWrapper_ClientProxy.getByName(CountriesService$$CDIWrapper_ClientProxy.zig:267)
        at org.acme.rest.client.CountriesResource.name(CountriesResource.java:26)
        at org.acme.rest.client.CountriesResource$quarkusrestinvoker$name_90180a8b8f951533128a0248c427c97ef233be88.invoke(CountriesResource$quarkusrestinvoker$name_90180a8b8f951533128a0248c427c97ef233be88.zig:39)
        at org.jboss.resteasy.reactive.server.handlers.InvocationHandler.handle(InvocationHandler.java:29)
        at org.jboss.resteasy.reactive.server.handlers.InvocationHandler.handle(InvocationHandler.java:7)
        at org.jboss.resteasy.reactive.common.core.AbstractResteasyReactiveContext.run(AbstractResteasyReactiveContext.java:122)
        at org.jboss.resteasy.reactive.server.handlers.RestInitialHandler.beginProcessing(RestInitialHandler.java:47)
        at org.jboss.resteasy.reactive.server.vertx.ResteasyReactiveVertxHandler.handle(ResteasyReactiveVertxHandler.java:17)
        at org.jboss.resteasy.reactive.server.vertx.ResteasyReactiveVertxHandler.handle(ResteasyReactiveVertxHandler.java:7)

Expected behavior

The tests should pass

@geoand geoand added the kind/bug Something isn't working label Mar 31, 2021
@quarkus-bot
Copy link

quarkus-bot bot commented Mar 31, 2021

/cc @michalszynkiewicz

geoand added a commit that referenced this issue Mar 31, 2021
Fix case where rest client doesn't recognize the default port or https
@geoand geoand assigned geoand and unassigned michalszynkiewicz Mar 31, 2021
@quarkus-bot quarkus-bot bot added this to the 1.14 - main milestone Mar 31, 2021
@gsmet gsmet modified the milestones: 1.14 - main, 1.13.1.Final Apr 3, 2021
gsmet pushed a commit to gsmet/quarkus that referenced this issue Apr 3, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/rest-client kind/bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants