Skip to content
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

Allow wildcard in path for path_limit #119

Closed
abarkine opened this issue Mar 29, 2021 · 2 comments · Fixed by #121
Closed

Allow wildcard in path for path_limit #119

abarkine opened this issue Mar 29, 2021 · 2 comments · Fixed by #121

Comments

@abarkine
Copy link
Contributor

Currently, only method wildcard works [here]. As a feature request, we can add the functionality for allowing wildcard in path matches.

Example use-case: Rate limit all POST requests

If my request is valid, I can create the PR for it.
@jaytaph
Copy link
Owner

jaytaph commented Apr 6, 2021

Sounds like a good idea. If you have a PR, we're happy to look into it and merge it

@abarkine abarkine mentioned this issue Apr 6, 2021
Merged
@abarkine
Copy link
Contributor Author

abarkine commented Apr 6, 2021

Thank you for your reply. I've created the PR.

@abarkine abarkine mentioned this issue Apr 7, 2021
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Allow wildcard in path
2 participants
@jaytaph @abarkine