codeconsole commented on PR #13863:
URL: https://github.com/apache/grails-core/pull/13863#issuecomment-4792387714

   @sbglasius thanks for the review — I've documented this so users can see why 
an app might be affected and how to control it:
   
   - **Upgrade notes**: new section *"27. @EnableWebMvc No Longer Added to 
Grails Applications"* in `grails-doc` (`upgrading80x.adoc`). It covers the two 
user-visible behaviors and their opt-outs — the `Circular view path` symptom 
from Boot's `defaultViewResolver` (`grails.web.removeDefaultViewResolverBean`, 
default `true`) and the `GrailsWebRequest` binding.
   - **PR description**: updated to spell out the full scope (the 
`defaultViewResolver` consolidation and the `GrailsWebRequestFilter` → 
`RequestContextFilter` change), since the original only mentioned the 
`@EnableWebMvc` removal — which I think is what made the impact hard to gauge.
   - **Tests**: added `ControllersAutoConfigurationSpec`, which proves Boot's 
`requestContextFilter` actually backs off (with a contrast case showing it's 
present without the Grails auto-config), so the binding behavior is 
regression-guarded.
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to