Regression in grails-views 7.0.0-M3 when forwarding from controller

[davidkron]

I am currently experimenting with migrating an existing grails application to version 7.0.0-M3. There are some problems related to forwarding requests to other actions/controllers, which result in an error like this:

jakarta.servlet.ServletException: Could not resolve view with name 'route' in servlet with name 'dispatcherServletRegistration'
	at org.springframework.web.servlet.DispatcherServlet.render(DispatcherServlet.java:1410)
	at org.springframework.web.servlet.DispatcherServlet.processDispatchResult(DispatcherServlet.java:1167)
	at org.springframework.web.servlet.DispatcherServlet.doDispatch(DispatcherServlet.java:1105)
	at org.springframework.web.servlet.DispatcherServlet.doService(DispatcherServlet.java:978)
	at org.springframework.web.servlet.FrameworkServlet.processRequest(FrameworkServlet.java:1014)
	at org.springframework.web.servlet.FrameworkServlet.doGet(FrameworkServlet.java:903)
	at jakarta.servlet.http.HttpServlet.service(HttpServlet.java:564)

In this case the action route is called and it is correct that there doesn't exist a route.gsp for this action, because it is simply a dispatcher, that does forward to a different action.

Unfortunately I can't reproduce the error using a simple grails application, so the problem seems to be some special constellation in our application, which is pretty complex.

After some debugging, the problem seems to be that after the forward dispatch (handling of the forwarded request inclusive rendering seems to work as expected), the result handling of the original action will still try to render the view, even though a rendering was already done by the forward. At that point in time, the response is still in a non-committed state, which seems to be the criteria to trigger a render:

grails-core/grails-web-common/src/main/groovy/org/grails/web/servlet/mvc/GrailsWebRequest.java

Line 371 in c401211

!currentResponse.isCommitted() &&

I managed to track down the settings of the commited state of the response to catalina's ApplicationDispatcher, where the closing of the response's PrintWriter will commit the response.

PrintWriter writer = response.getWriter();
writer.close();

In version 7.0.0-M3 of grails, the response's print writer seems to be a RoutablePrintWriter from Sitemesh, which doesn't commit anything and seems to be the source of my error. In the previous Grails version, the print writer is a CoyoteWriter, which seems to work as expected.

[jdaugherty]

Is this issue new as of the M3 of 7.x or new for you as you tried the M3 build of 7.x. ie did this work on M2 originally or 6.x?

I assume this is an issue with the upgrade from 6.x to 7.x?

[davidkron]

The issue appeared from a 6.x upgrade to 7.x. I did not use the previous milestones releases yet and went straight to M3.

[jdaugherty]

The issue appeared from a 6.x upgrade to 7.x. I did not use the previous milestones releases yet and went straight to M3.

Thank you for confirming. We know that there are several issues related to the sitemesh3 upgrade and I suspect this is another example.

[jdaugherty]

As part of the upgrade to Spring 3.5, we found several forward / error handling related issues. These issues are now fixed in 7.0.0-M4. The vote is currently underway, can you please take the time to retest this once all of 7.0.0-M4 is released? I suspect this may be fixed.

[davidkron]

I did test this today with the current 7.0.0-M4 version, but it still results in the same error.
In the meantime, I managed to work around the issue with the following:

Controller code snippet before Grails 7:

forward action: 'enhancedRouting'
return

Workaround without forwarding:

def model = enhancedRouting(cmd)
render view: 'enhancedRouting', model: model
return

[davidkron]

I managed to create a reproducer: https://github.com/davidkron/grails-issue-14193

Opening http://localhost:8080/greeting/hello will produce the error.