-
Notifications
You must be signed in to change notification settings - Fork 9.3k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
REST service with 'SearchCriteria' is of type GET #1085
Comments
I'm reading the related bugs, and maybe I'm not seeing the possibility on how to send the searchcriteria with GET.. |
@rklerck We are still working on the documentation for our REST apis but a search sample would be something like: |
GET vs PUT vs POST is one of those semi religious topics. This one was quite a good read http://stackoverflow.com/questions/978061/http-get-with-request-body. As you work your way down you start with doing a POST to and end point, with a slightly sheepish "oh well". Then later it says things like some proxies may not work. Then "elastic search encourages it". GET is semantically more correct as there are no side effects. |
Thanks for the explanation. I will close this issue than. |
[EngCom] Public Pull Requests - MAGETWO-68979: Fix breadcrumbs extra space #9514 - MAGETWO-68977: fix for Class Magento\Framework\Console\CLI not found in case sensitive scenarios #9539 - MAGETWO-68861: Fix data deletion using the multiple delete command #9446 - MAGETWO-67722: Add ability to inject exception code in LocalizedException #9363
MSI-858: [Configuration-Stocks] Admin user applied Mass action: Delet…
I'm experimenting with the available REST services (for example 'rest/V1/products'), but in 'webapi.xml' it is routed as a GET service. But since a json request needs to be send to the service with the searchcriteria, it should by of type 'PUT', right?
It's for every service as far as I looked. Changed it to PUT and it works..
The text was updated successfully, but these errors were encountered: