Enhancement: Support the QUERY HTTP method

[marcuslimdw]

Summary

Add QUERY to HttpMethod and create a dedicated query route handler decorator that, like the others, delegates to HTTPRouteHandler. In CSRFConfig and other places (not 100% sure where), also add QUERY as a safe method.

Basic Example

@HTTPRouteHandler("/", http_method="QUERY")
def query() -> None:
    pass

becomes:

@query("/")  # type: ignore
def query() -> None:
    pass

Drawbacks and Impact

I'm not familiar enough with Litestar's routing to be able to say what the performance impact would be.

Unresolved questions

No response

[euri10]

sounds weird, no query http exists (https://developer.mozilla.org/en-US/docs/Web/HTTP/Reference/Methods), would you explain what you have in mind ?

[marcuslimdw]

https://httpwg.org/http-extensions/draft-ietf-httpbis-safe-method-w-body.html

long story short it's GET (safe) but it allows a body

Node, in particular, supports it

currently I think the main thing for me is that Litestar doesn't support arbitrary HTTP methods without circumventing type checkers, though there isn't actually any technical reason to do so (see e.g. Go's stdlib, which just accepts a string for the HTTP method)

[Kumzy]

AFAIK, it's still a draft and not officially published.
https://datatracker.ietf.org/doc/draft-ietf-httpbis-safe-method-w-body/

There is no plan to support it in the near future or change on how to handle HTTP methods.
It can be reviewed later when the draft will be in a more advanced state.