codeconsole commented on PR #16149:
URL: https://github.com/apache/grails-core/pull/16149#issuecomment-5572564914
Thanks. Inline comments answered on their threads; the three review
observations are in c8bbcb8.
**Unexercised forward actions** — kept and put to work rather than dropped,
since they cover the P1b decision in a running container. The forward now
targets the member URL instead of naming an action, so the mappings route it:
`delete` for a POST naming DELETE, `update` for the same POST without it.
**Cross-controller `ALLOWED_METHODS_HANDLED`** — added. A new
`RequestPathForwarderController` restricts nothing and forwards into the
DELETE-only `delete`; it is admitted because the first action recorded that it
began handling.
**`isAjax` multipart test** — written, then removed. It passed against the
direct `multipart?.getParameter('ajax')` read it was meant to catch, so the
branch is not reached under that harness and it asserted nothing.
`WebUtilsSpec` keeps the coverage.
`RequestPathSpec` is nine cases now.
--
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]
