Skip to content
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

Allow GET requests to have a request body #8306

Closed
wants to merge 1 commit into from
Closed

Conversation

mo7ty
Copy link

@mo7ty mo7ty commented Mar 25, 2024

Although it is not recommended, if agreed between client and server, GET requests may have content.

* As per [RFC 9110 - HTTP Semantics - section 9.3.1. GET](https://datatracker.ietf.org/doc/html/rfc9110#section-9.3.1):
```
(...) A client SHOULD NOT generate content in a GET request unless it
is made directly to an origin server that has previously indicated,
in or out of band, that such a request has a purpose and will be
adequately supported.
```
@yschimke
Copy link
Collaborator

Prior discussion in #3154

@yschimke yschimke closed this Mar 25, 2024
@mo7ty
Copy link
Author

mo7ty commented Mar 25, 2024

Isn't desirable for OkHttp to be compliant with RFC 9110 - HTTP Semantics?

Or is it just technically not possible?

Is there any workaround besides using a non-compliant method?
(ref method token is case-sensitive)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants