Support all the registered HTTP methods in the Method enum#332
Support all the registered HTTP methods in the Method enum#332yoshuawuyts merged 1 commit intohttp-rs:mainfrom zenekron:all-registered-http-methods
Method enum#332Conversation
|
Out of curiosity, what is the use case for having WebDAV extensions in http-types? |
Besides being able to speak WebDAV itself, there are other tools that reuse subsets of these methods for other purposes. In my case specifically: I'm porting a small Terraform HTTP backend I wrote from warp to tide, but in order to implement state locking I need to be able to handle |
|
Does this mean you'd also want affordances in tide's router for these methods? |
I'm not too sure what you mean by that (not a native English speaker here), but if you're talking about creating methods like That being said if you think it's appropriate I can open a PR against tide as well, just let me know. |
That is indeed what @jbr meant. |
I agree with this rationale; I don't think we need to expose shorthands for them, but having them as part of the enum seems reasonable enough. |
This PR extends the
Methodenum to add all the missing methods from IANA's HTTP Method Registry.See the issue #331 .
Changes:
AsRef<str>,FromStrandMethod::is_safeaccordinglyDisplayto leverageAsRef<str>