-
-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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 macro for route handler accepting all HTTP methods #3263
Comments
I will like to take this issue, can you help me figure out the initials and the codebase? |
Or provide more information |
You can look inside the |
When I tried to run trybuild tests using cargo test --test trybuild or even just cargo test, it shows that 0 tests were run. Am i making a mistake? @robjtede |
Good point. The trybuild tests only run on our MSRV. So Rust 1.72. |
I don't know if it's a dumb question buy may I ask whether the HTTP methods are imported from inbuilt libraries or defined within the source code itself. If they are defined please could you specify where? @robjtede |
I figured out that it is exported. Now, I will work on defining ALL method kind of thing because an error related to naming conflict was generated |
Can you guide me on that a little @robjtede ? It's been quite a task |
@sabinonweb Do you want to open a draft PR so I can see where you've gotten to? |
@robjtede please have a look |
This comment has been minimized.
This comment has been minimized.
I left some comments earlier. See the PR. |
Would adding methods in with_any_methods, similar to with_methods work @robjtede |
Hello, this issue and the pull request attached to it has been stale for a while, would you mind if i give if a try? @sabinonweb @robjtede |
As discussed in #3259, implement macros like
#[any("...")]
and#[route("...", method = any)]
to create route handlers accepting all HTTP methods (which is currently only possible with something likeapp.route("...", web::route().to(handler))
)The text was updated successfully, but these errors were encountered: