vert-x3 / vertx-web

HTTP web applications for Vert.x
Apache License 2.0
1.11k stars 535 forks source link

Always set the routing context in HttpServerRequestWrapper #2651

Closed tsegismont closed 1 month ago

tsegismont commented 1 month ago

Before this change, it was expected that a Route handler implementation wraps the HttpServerRequest (again, since it was already wrapped by the routing context).

This was confusing, because in Vert.x Web apps, the HttpServerRequest implemented WebServerRequest but invocations of its routingContext() method always returned null.