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 trotelling to Recovery controller #600

Open
pral2a opened this issue Feb 14, 2022 · 0 comments
Open

Add trotelling to Recovery controller #600

pral2a opened this issue Feb 14, 2022 · 0 comments
Labels

Comments

@pral2a
Copy link
Member

pral2a commented Feb 14, 2022

Describe the bug
Looks like recovery password controller doesn't limit the number of requests, allowing someone to programatically use it to generate email spam or overload the SMTP service

To Reproduce
Steps to reproduce the behaviour:

  1. Go to https://www.fablabs.io/recoveries/new
  2. Type your email and click Reset password
  3. Repeat multiple times in a short period
  4. Check your inbox

Expected behaviour
Rate limit password resets by email to X amount per hour. Other limitation (ip) can also be used.

Additional context
The issue was reported by an unknown user at webmasters (at) fablabs.io. This is not a high priority issue.

image (4)

@pral2a pral2a added the bug label Feb 14, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

1 participant