Skip to content

Add support for QUERY method #60521

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.

Sign up for GitHub

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

Jump to bottom
Open
nicolas-grekas opened this issue May 23, 2025 · 2 comments
Open

Add support for QUERY method #60521

nicolas-grekas opened this issue May 23, 2025 · 2 comments
Labels
Help wanted Issues and PRs which are looking for volunteers to complete them. HttpFoundation HttpKernel

Comments

@nicolas-grekas
Copy link
Member

Description

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

Not sure how far this should go. At least the Request class can list this method. Not sure about HttpCache.

Help wanted to dig the topic 🙏

Example

No response
@nicolas-grekas nicolas-grekas added the Help wanted Issues and PRs which are looking for volunteers to complete them. label May 23, 2025
@GromNaN
Copy link
Member

GromNaN commented May 23, 2025 •
edited
Loading

That method is ideal for read API with a request payload, like a GraphQL endpoint for query operations.

HttpCache should treat these requests as GET, but use the request body (a hash) in the cache key.

The response to a QUERY method is cacheable; a cache MAY use it to satisfy subsequent QUERY requests
The cache key for a QUERY request MUST incorporate the request content. When doing so, caches SHOULD first normalize request content to remove semantically insignificant differences

@MatTheCat
Copy link
Contributor

This would probably impact #59358

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Help wanted Issues and PRs which are looking for volunteers to complete them. HttpFoundation HttpKernel
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@nicolas-grekas @GromNaN @MatTheCat @xabbuh