WebMvcAutoConfigurationAdapter needs an addInterceptors method to allow LocaleChangeInterceptor to be set

[csavory]

I would like to add LocaleChangeInterceptor to a spring boot application. LocaleResolver resolver is easy because it is a bean, but LocaleChangeInterceptor has to be added to a registry.

I see this method on WebMvcAutoConfigurationAdapter:

public void addFormatters(FormatterRegistry registry) {

Could we add this too?

@Override
      public void addInterceptors(InterceptorRegistry registry) {

            for (HandlerInterceptor<?, ?> interceptor : getBeansOfType(HandlerInterceptor.class)) {
                registry.addInterceptor(interceptor);
            }
      }

so that i don't have to write my own @EnableWebMvc class and lose the default Spring Boot stuff.

[dsyer]

I think that's just a misunderstanding of the role of the WebMvcAutoConfigurationAdapter. To add interceptors (or any other normal Spring MVC feature), normally you just add your own WebConfigurerAdapter (and don't add @EnableWebMvc to it). We can certainly call this out more clearly in the documentation.

[PedroAlvarado]

@dsyer I can't seem to find this WebConfigurerAdapter type. Are you referring to WebSecurityConfigurerAdapter?

[dsyer]

Try WebMvcConfigurerAdapter (it's a building block from Spring MVC and very well documented there).