You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Unlike WebMvcAutoConfiguration.WebMvcAutoConfigurationAdapter (a WebMvcConfigurer), WebFluxAutoConfiguration.WebFluxConfig (a WebFluxConfigurer) is unordered. This means that it implicitly has lowest precedence and manes that it's impossible to guarantee that any other WebFluxConfigurer will go after it. Our WebMvcConfigurer is @Order(0). We should consider applying the same order to our WebFluxConfiguer as well.
See #12389 for details of why the ordering was configured on the MVC side. If we agree to make this change, we'll need to think about when we want to make it. Theoretically, changing the ordering will break an application that's relying on the current ordering but I'm not sure how likely that is. FWIW, we made the MVC change early in 2.0.x's life (2.0.1).
The text was updated successfully, but these errors were encountered:
wilkinsona
changed the title
The auto-configured WebFluxConfigurer is unordered so it may override customisations made in a user's WebFluxConfigurer
The auto-configured WebFluxConfigurer is unordered so it may override customizations made in a user's WebFluxConfigurer
Feb 24, 2021
Unlike
WebMvcAutoConfiguration.WebMvcAutoConfigurationAdapter
(aWebMvcConfigurer
),WebFluxAutoConfiguration.WebFluxConfig
(aWebFluxConfigurer
) is unordered. This means that it implicitly has lowest precedence and manes that it's impossible to guarantee that any otherWebFluxConfigurer
will go after it. OurWebMvcConfigurer
is@Order(0)
. We should consider applying the same order to ourWebFluxConfiguer
as well.See #12389 for details of why the ordering was configured on the MVC side. If we agree to make this change, we'll need to think about when we want to make it. Theoretically, changing the ordering will break an application that's relying on the current ordering but I'm not sure how likely that is. FWIW, we made the MVC change early in 2.0.x's life (2.0.1).
The text was updated successfully, but these errors were encountered: