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

implement rate limiting #104

Open

f10l opened this issue Apr 26, 2020 · 0 comments · May be fixed by #113

Assignees

Labels

Member

f10l commented Apr 26, 2020

There are two issues:

General rate limiting would improve the stability: https://docs.nestjs.com/techniques/security#rate-limiting
The JWT/bcrypt creation can also slow down the server drastically, if overused.
Check if there is an easy solution to rate-limit this endpoint.

f10l added the needs assignee label

JackVarney self-assigned this

JackVarney linked a pull request

that will close this issue

Created rate limit middleware #113

Open

f10l linked a pull request

that will close this issue

Created rate limit middleware #113

Open

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment