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

Add Rate Limiting to API #38

Open
uzaxirr opened this issue Nov 5, 2021 · 4 comments
Open

Add Rate Limiting to API #38

uzaxirr opened this issue Nov 5, 2021 · 4 comments
Assignees

Comments

@uzaxirr
Copy link
Member

uzaxirr commented Nov 5, 2021

2 or 1 Request per second

@brijeshshah13
Copy link

@uzair-ali10 what are you using nginx on production to route requests between dns and server?

@uzaxirr
Copy link
Member Author

uzaxirr commented Nov 28, 2021

@uzair-ali10 what are you using nginx on production to route requests between dns and server?

Nope, we don't use Nginx as of now, rate-limiting should be implemented using Express Rate Limit package

https://www.npmjs.com/package/express-rate-limit
https://www.geeksforgeeks.org/how-to-integrate-express-rate-limit-in-node-js/

@brijeshshah13
Copy link

cool. please assign this to me. also, if not nginx, how are the requests being routed?

@uzaxirr
Copy link
Member Author

uzaxirr commented Nov 28, 2021

cool. please assign this to me. also, if not nginx, how are the requests being routed?

Cloudflare

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