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

feat: add support for HTTP QUERY method #745

Closed
wants to merge 1 commit into from

Conversation

gdubicki
Copy link

@gdubicki gdubicki commented Jan 12, 2025

The HTTP QUERY method is a means of making a safe, idempotent request that contains content. The primary motivator for this case is graphql, replacing HTTP POST for read only requests, however it is application agnostic.

This method is still in draft but since 2025-01-07 it has reached consensus so it should be stable enough to make it unlikely to change this implementation.

https://datatracker.ietf.org/doc/draft-ietf-httpbis-safe-method-w-body/

Closes #743

@gdubicki gdubicki marked this pull request as ready for review January 12, 2025 14:19
@gdubicki
Copy link
Author

This is an alternative, (arguably?) a bit more complete version of #717. IMHO either that PR should be expanded and this one closed or this should take over.

@gdubicki gdubicki force-pushed the 743-feature branch 3 times, most recently from 860b01a to a4b0d0d Compare January 12, 2025 14:41
The HTTP QUERY method is a means of making a safe, idempotent request
that contains content. The primary motivator for this case is graphql,
replacing HTTP POST for read only requests, however it is application
agnostic.

This method is still in draft but since 2025-01-07 it has reached consensus
so it should be stable enough to make it unlikely to change this implementation.

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

Closing in favor of #717, see this for rationale.

@gdubicki gdubicki closed this Jan 12, 2025
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.

Please add support for the QUERY HTTP method
1 participant