Web Client: Query parameters missing for SSL requests

[svenjacobs]

Version

• vert.x core: 3.4.1
• vert.x web: 3.4.1

Context

I noticed that query parameters are lost when the request is SSL encrypted. For instance this is a request which works:

webClient.get("example", "/test")
        .putHeader("Accept-Version", "v1")
        .putHeader("Authorization", authorization)
        .addQueryParam("page", page.toString())
        .addQueryParam("per_page", perPage.toString())
        .send(...)

I created a test endpoint at my backend and logged the query parameters so I can confirm that they are incoming. However once I switch to SSL I can see that the query parameters are missing:

webClient.get(443, "example", "/test")
        .ssl(true)
        .putHeader("Accept-Version", "v1")
        .putHeader("Authorization", authorization)
        .addQueryParam("page", page.toString())
        .addQueryParam("per_page", perPage.toString())
        .send(...)

I'm not sure what I'm doing wrong here...

[vietj]

would you mind to create a test case ? that would help to find the issue :-)

[svenjacobs]

I found the problematic line in HttpRequestImpl. I believe it should be .setUri(requestURI) at line 279. Luckily there's already the PR #562 open addressing this.

[svenjacobs]

Closing since PR has been merged