-
-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
body for HTTP GET request #141
Labels
Comments
few more interesting discussions: TL;DR technically, it's ok to send body with GET/DELETE requests. Semantically - it's not. In real world only few uses such "feature". |
+1 let's unify 'data' processing for all type of requests |
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
When aiohttp.request gets
GET
method with specifieddata
that merges data to url along withparams
.I think that's wrong. HTTP protocol standard (http://www.w3.org/Protocols/rfc2616/rfc2616-sec9.html) says nothing about presence/absence body for GET methods.
Moreover elasticsearch uses GETs with body for some REST API methods: see http://www.elasticsearch.org/guide/en/elasticsearch/reference/current/docs-multi-get.html and http://www.elasticsearch.org/guide/en/elasticsearch/reference/current/search-multi-search.html#search-multi-search
I guess to always process
data
as forPOST
request if specified.The text was updated successfully, but these errors were encountered: