You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
IMHO the implementation is wrong, DELETEcan have a body according to RFC 7231, so DELETE should be removed from ContentValidationListener::methodsWithoutBodies.
@BreiteSeite 👍🏻 for the above. DELETE usually only requires a request body when removing many entries at once from a list (for a collection). So, based on that, something similar to the following could be added to the content validation listener (ContentValidationListener) in the function onRoute(MvcEvent $e):
Now that we have support for GET requests to provide an input filter for collections, we can likely do this for DELETE. I'm marking this as an enhancement, and soliciting help for resolving it; it should be relatively easy to accomplish.
@weierophinney I think you could close this issue as in version 1.6.0 DELETE request already supported body (it is removed from ContentValidationListener::methodsWithoutBodies) and implemented that functionality
Because of ContentValidationListener.php:63 it is not possible to validate the body of a
DELETE
request (contrary to the README.md).IMHO the implementation is wrong,
DELETE
can have a body according to RFC 7231, soDELETE
should be removed fromContentValidationListener::methodsWithoutBodies
.Originally posted by @BreiteSeite at zfcampus/zf-content-validation#60
The text was updated successfully, but these errors were encountered: