Allow WebFlux ApiVersionResolver to return a Mono by JKaplanEmpty-Nes · Pull Request #36084 · spring-projects/spring-framework

Allow WebFlux ApiVersionResolver to return a Mono by JKaplanEmpty-Nes · Pull Request #36084 · spring-projects/spring-framework

JKaplanEmpty-Nes changed the title ~~Change the API version resolution from being a nullable String to a Mono String~~ Change the WebFlux API version resolution from being a nullable String to a Mono String

bclozel added the in: web

Issues in web modules (web, webmvc, webflux, websocket)

label

rstoyanchev changed the title ~~Change the WebFlux API version resolution from being a nullable String to a Mono String~~ Allow WebFlux ApiVersionResolver to return a Mono

Signed-off-by: Jonathan Kaplan <jkaplan@empty-nes.com>

rstoyanchev pushed a commit that referenced this pull request

See gh-36084

Signed-off-by: Jonathan Kaplan <jkaplan@empty-nes.com>

rstoyanchev added a commit that referenced this pull request

rstoyanchev added a commit that referenced this pull request

rstoyanchev added a commit that referenced this pull request

- deprecate sync method on ApiVersionResolver
- add SyncApiVersionResolver
- refactor resolverParseAndValidateApiVersion method

See gh-36084

rstoyanchev added a commit that referenced this pull request

Resolve conflicts and refactor merged code in AbstractHandlerMapping

See gh-36084

rstoyanchev added a commit that referenced this pull request

Instead of making it async and having a sync subinterface variant,
this restores ApiVersionResolver to be as it was with an async
subinterface variant.

ApiVersionStrategy, and the infrastructure invoking it, remains
async first, but also accommodates sync resolvers.

This should provide a better balance with backwards compatibility
while also accommodating async version resolution as the less
common scenario.

See gh-36084