WebFlux equivalent of MvcUriComponentsBuilder [SPR-15953]

[spring-projects-issues]

Arjen Poutsma opened SPR-15953 and commented

We could use a WebFluxUriComponentsBuilder that has similar functionality as the ServletUriComponentsBuilder,

An additional method that WebFluxUriComponentsBuilder should have is fromServerRequest, taking a ServerRequest from WebFlux.fn, as that type does not implement HttpRequest, and therefore cannot be used with UriComponentsBuilder.fromHttpRequest.

---

Issue Links:

• ReactorServerHttpRequest.getUri() does not include scheme [SPR-15931] #20485 ReactorServerHttpRequest.getUri() does not include scheme
• WebFlux support for UriBuilder, UriComponentsBuilder controller method arguments [SPR-15997] #20546 WebFlux support for UriBuilder, UriComponentsBuilder controller method arguments

Referenced from: commits 1a3cc3d, a7617a2

[spring-projects-issues]

Rossen Stoyanchev commented

ServerRequestUriComponentsBuilder has methods to build from a request instance. Considering that UriComponentsBuilder.fromHttpRequest already exists what else would WebFluxUriComponentsBuilder have besides a fromServerRequest(ServerRequest) method?

Note that we will also need an equivalent of MvcUriComponentsBuilder where the name WebFluxUriComponentsBuilder would be a good fit.

[spring-projects-issues]

Rossen Stoyanchev commented

An alternative idea is something like ServerRequest.relativeToRequestUri returning UriComponentsBuilder.

[spring-projects-issues]

Arjen Poutsma commented

Initial version committed at a7617a2.

As discussed, Rossen Stoyanchev will add subsequent @Controller-related methods.

[spring-projects-issues]

Rossen Stoyanchev commented

Arjen Poutsma taking a quick look, for WebFluxUriComponentsBuilder to have @Controller methods its current location could be an issue. MvcUriComponentsBuilder by comparison in is in the same package as all the other annotation related classes. Even if we don't run into circular dependencies, arguably such a builder should be in the annotations package. Also I can't imagine that using such a WebFluxUriComponentsBuilder from the functional framework -- with a ton of @Controller methods and only 1 for ServerRequest, is ideal.

Since what is needed for ServerRequest is a trivial one-liner, could we find another solution perhaps? ServerRequest has a uri() method. What about a uriComponentsBuilder next to it?

[spring-projects-issues]

Arjen Poutsma commented

??Since what is needed for ServerRequest is a trivial one-liner, could we find another solution perhaps? ServerRequest has a uri() method. What about a uriComponentsBuilder next to it???

That sounds like a good idea. I will move the method from WebFluxUriComponentsBuilder to ServerRequest, and remove WebFluxUriComponentsBuilder altogether.

[spring-projects-issues]

Arjen Poutsma commented

Done, see 1a3cc3d

[spring-projects-issues]

Rossen Stoyanchev commented

Now that ServerRequest has an option and #20546 adds the same controllers methods, this is less critical for 5.0 GA. I'm postponing for 5.1.

[poutsma]

Closing, as I don't think we need this anymore.