Reactive REST client duplicates paths, described by subresources

[fedinskiy]

Describe the bug

When reactive REST client is used to access sub-resources of some HTTP resource, it duplicates paths which describe these sub-resources. For example, if there is such method in rest client:

@Path("/resource")
public interface Client {
 @Path("/sub")
    SubClient getSubResource()
 }
}

then call client.getSubResource().getSomething() will lead to HTTP request to resource/sub/sub/something.

Expected behavior

Sub-resources should not lead to duplicated paths.

Actual behavior

Paths are duplicated

How to Reproduce?

1. git clone -b reproducer/duplicate-paths [email protected]:fedinskiy/quarkus-test-suite.git
2. cd http/rest-client-reactive
3. mvn clean verify -Dit.test=ReactiveRestClientIT#deepLevel ( or ReactiveRestClientIT#subResource)
4. There is a log which shows, that requests are send to URLs like http://localhost:1101/books/author/profession/profession/wage/wage/currency/currency/name

Output of uname -a or ver

5.16.13-200.fc35.x86_64

Output of java -version

11.0.12, vendor: GraalVM Community

GraalVM version (if different from Java)

No response

Quarkus version or git rev

2.7.5.Final

Build tool (ie. output of mvnw --version or gradlew --version)

Apache Maven 3.8.4 (9b656c72d54e5bacbed989b64718c159fe39b537)

Additional information

No response

[quarkus-bot]

/cc @michalszynkiewicz

[geoand]

@michalszynkiewicz can you take a look at this please?
As @sberyozkin mentions, this is likely causing problems with the Keycloak admin client

[michalszynkiewicz]

I'll try to fix this today

[michalszynkiewicz]

it turned out to be slightly more complex than anticipated but I should be able to fix it tomorrow :)

[ghost]

Hi, any news? I am troubled by this problem. When can it be fixed? @michalszynkiewicz

[geoand]

Michal is working on it. It will be fixed soon (in 2.8.3 at the earliest). That's as specific as we can be.

[ghost]

Michal is working on it. It will be fixed soon (in 2.8.3 at the earliest). That's as specific as we can be.

2.8.3, how long will it take? I'm developing a project. If it takes too long, I can't wait for 2.8.3. I may need to solve the problem in another way.

[geoand]

There is no exact date.
Could be next week or the week after

[ghost]

Great! I see it has been fixed! It will be added to next version 2.8.2, right? And, when 2.8.2 will be released?@geoand

[geoand]

It's unlikely to make it into 2.8.2 (which may be in the process of being released)

[gsmet]

I will release 2.8.2 tomorrow. This patch will be in if it's merged by this evening.

[ghost]

I see 2.8.2 released, great! I love quarkus! Great team! Great quarkus!

[geoand]

🥰