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.

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

Support for default limit #1

Closed
laurentS opened this issue Apr 20, 2020 · 3 comments
Closed

Support for default limit #1

laurentS opened this issue Apr 20, 2020 · 3 comments

Comments

@laurentS
Copy link
Owner

There should be a way to declare a default set of limits that apply to all endpoints without explicitly marking each endpoint with the decorator.

@Rested
Copy link
Collaborator

Rested commented Jul 1, 2020

ran into this, was there a reason not to implement as a starlette BaseHTTPMiddleware? it seems like you started doing that here

# app: Starlette = None,
happy to have a crack at implementing that and the exempt decorator

@laurentS
Copy link
Owner Author

laurentS commented Jul 2, 2020 via email

@laurentS
Copy link
Owner Author

laurentS commented Oct 1, 2020

Closing this as default limit is now available via the middleware, thank @Rested !

@laurentS laurentS closed this as completed Oct 1, 2020
ME-ON1 added a commit to ME-ON1/slowapi that referenced this issue Jun 16, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants