Don't repeat query params in access log

[jaikiran]

Fixes #9295

The "%r" format specifier used by the common access log implies the request line. The implementation in RequestLineAttribute currently uses exchange.request().uri() which includes both the request path and query params (the javadoc of that io.vertx.core.http.HttpServerRequest.uri() doesn't explain much of what to expect of the return value but at least the implementation returns both the path and the query params. So there's no need to additionally append the query params explicitly. The commit in this PR fixes that and includes a test to reproduce the issue and verify the fix.

[joan-miralles]

What about if the URL contains some character like #? What's the behaviour of the io.vertx.core.http.HttpServerRequest.uri()?

Thanks!

[jaikiran]

The # is a client side construct. It's never sent to the server.

[cescoffier]

@gsmet this one could be merged before the cut. Harmless.

[gsmet]

Merged, thanks!

[joan-miralles]

Thanks a lot!