-
Notifications
You must be signed in to change notification settings - Fork 25
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
[#155244960] add rate limiter #585
base: master
Are you sure you want to change the base?
Conversation
Affected stories
New dependencies added: rate-limiter-flexibleAuthor: animir Description: Node.js rate limiter by key and protection from DDoS and Brute-Force attacks in process Memory, Redis, MongoDb, Memcached, MySQL, PostgreSQL, Cluster or PM Homepage: https://github.com/animir/node-rate-limiter-flexible#readme
Generated by 🚫 dangerJS |
Codecov Report
@@ Coverage Diff @@
## master #585 +/- ##
==========================================
+ Coverage 82.92% 83.20% +0.27%
==========================================
Files 48 49 +1
Lines 1394 1417 +23
Branches 245 246 +1
==========================================
+ Hits 1156 1179 +23
Misses 226 226
Partials 12 12
Continue to review full report at Codecov.
|
Co-Authored-By: Daniele Manni <BurnedMarshal@users.noreply.github.com>
@cloudify can you check this PR and default vaules pls ? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm
Co-Authored-By: Daniele Manni <BurnedMarshal@users.noreply.github.com>
Add a rate limit to post / put endpoints. Defaults to 10000 calls per 1 hour for IP address.
Note that it does not rate limit for GET endpoints.