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

Please add support for the QUERY HTTP method #743

Open
gdubicki opened this issue Jan 11, 2025 · 1 comment
Open

Please add support for the QUERY HTTP method #743

gdubicki opened this issue Jan 11, 2025 · 1 comment

Comments

@gdubicki
Copy link

gdubicki commented Jan 11, 2025

On 2025-01-07 the QUERY HTTP method, "a safe, idempotent request method that can carry request content." has reached the level of a Proposed Standard consensus and is on track to become a Proposed Standard.

It solves the problem of making complex queries where the GET method is sometimes sufficient as it (according to the standard) lacks support for sending the query in the body, leaving only the query part of the URL which is limited. An often used workaround of using the POST method has a downside in that it's just the wrong method for this purpose as queries do not cause side-effects and this limits the retryabilty and cacheability of such queries.

Therefore It would be great to implement support for QUERY in this library.

For example node.js has already implemented it and released in June 2024: nodejs/node#51562

If you are open to it, I am interested in providing a PR.

(Related: tokio-rs/axum#1691)

sigmavirus24 added a commit to sigmavirus24/hyperium.http that referenced this issue Jan 12, 2025
[draft-ietf-httpbis-safe-method-w-body-07][] was recently accepted by
the HTTP WG (although so recently that it does not have an official RFC
number).

The method and requests using it are meant to be idempotent and
retryable.

Closes hyperium#743

[draft-ietf-httpbis-safe-method-w-body-07]: https://datatracker.ietf.org/doc/draft-ietf-httpbis-safe-method-w-body/
@gdubicki
Copy link
Author

gdubicki commented Jan 12, 2025

There already was a PR for it here too: #717

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