Skip to content
This repository has been archived by the owner on Nov 14, 2023. It is now read-only.

[FEATURE] Introduce Rate limiting on API #339

Open
kshitijrajsharma opened this issue Sep 23, 2022 · 0 comments · May be fixed by #335
Open

[FEATURE] Introduce Rate limiting on API #339

kshitijrajsharma opened this issue Sep 23, 2022 · 0 comments · May be fixed by #335

Comments

@kshitijrajsharma
Copy link
Member

kshitijrajsharma commented Sep 23, 2022

API already uses redis for the task queue , we can use the same for rate limiting as well , Currently focus will be to stop those continuous request that may can come through some scripts or a program ! So basically idea will be to stop the same ip if making n no of calls with n no of seconds ! But while doing that this feature request also cares about the extension of the feature implemented ,it should be easily extendable in terms of rate limiting on other API endpoint and flexible may be using a single decorator or so on . I have seen few libraries on fastapi such as slowapi , fastapi-limiter , We can evaluate other open source tools and implement this

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant