-
Notifications
You must be signed in to change notification settings - Fork 38.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
Add QUERY
HTTP method
#33430
base: main
Are you sure you want to change the base?
Add QUERY
HTTP method
#33430
Conversation
Thanks for the initial efforts @desiderantes - at this point the linked issue hasn't been triaged and is very unlikely to make it to the 6.2 version. We'll need thorough reviews and testing for this one. Note that since this is a new HTTP method and new feature, I don't think we will want to introduce it everywhere and for all variants of our web support. For example, I'm wondering if surfacing it as a first class citizen in |
@bclozel Yes no worries, just wanted to get this started somehow. I do think that supporting a new HTTP Method shouldn't be half-hearted, and ideally it would mean support everywhere, not just in the basic |
@desiderantes We're trying to balance maintenance and evolution, especially around parts that have been very stable for a long time. Adding this method as first class in |
Marking as blocked for now because #32975 (comment) |
Fixes #32975