jdaugherty commented on PR #13863: URL: https://github.com/apache/grails-core/pull/13863#issuecomment-4920464588
@codeconsole I pushed the test showing the material differences. In summary: 4 failures — each one a Grails 8 behavior change: 1. DELETE form body — json.title expected null, got The Stand ([method:DELETE, title:The Stand, pages:1153]): Boot's OrderedFormContentFilter now parses the DELETE body into params. 2. Static resource — /boot-default-mapping.txt expected 404, got 200: Boot's catch-all resource handler now serves classpath:/public at the root. 3. Welcome page — / expected 404, got 200: Boot's WelcomePageHandlerMapping now serves the static index.html. 4. Locale — expected fr_FR (Accept-Language), got de: grails-i18n's SessionLocaleResolver is active, so ?lang=de works. I think `#4` is expected and a better behavior, so I'm ok with that. But 1-3 are definitely behavior differences that are material. We shouldn't be serving assets that Grails doesn't know about nor should we have other mappings. -- 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]
