Repository navigation
ForwardedParser calculates invalid absolute URI when request target is already absolute #2830
Description
Version
4.5.22, but the code is the same in the main branch
Context
Absolute request URIs are not computed correctly by HttpServerRequestWrapper class when the target is
absolute. For example, with this request:
GET http://localhost:8080/hello HTTP/1.1
Host: some.random.host.com:9876
The io.vertx.ext.web.impl.HttpServerRequestWrapper#absoluteURI() method returns http://some.random.host.com:9876http://localhost:8080/hello, which is obviously wrong.
I would expect the result to be one of
http://localhost:8080/hellohttp://some.random.host.com:9876/hello
The problem seems to be caused by [ForwardedParser#calculate()](https://github.com/vert-x3/vertx-web/blob/master/vertx-web/src/main/java/io/vertx/ext/web/impl/ForwardedParser.java#L136):
if (host != null) {
this.authority = HostAndPort.create(host, port);
host = host + (port >= 0 ? ":" + port : "");
absoluteURI = scheme + "://" + host + delegate.uri();
}Here delegate.uri() is already absolute, and is just appended as-is.
Steps to reproduce
Clone the reproducer and run mvn:test. The tests should pass, they are actually "verifying" the current faulty behavior.
Activity
tsegismont commentedon Nov 27, 2025
Thanks for the report @tmonney ! Would you like to contribute a fix?
tmonney commentedon Dec 1, 2025
I can give it a try, yes
Fix request absolute URI handling
tmonney commentedon Dec 12, 2025
@tsegismont , did you have a chance to have a look?
Fix request absolute URI handling
Fix request absolute URI handling
Fix request absolute URI handling (vert-x3#2833)
Fix request absolute URI handling (vert-x3#2833)
Fix request absolute URI handling (vert-x3#2833)
tsegismont commentedon Jan 23, 2026
Fixed by #2842
Fix request absolute URI handling (#2833) (#2841)
Fix request absolute URI handling (#2833) (#2842)