ProjectLighthouse icon indicating copy to clipboard operation
ProjectLighthouse copied to clipboard

Enhanced Login Security

Open m88youngling opened this issue 3 years ago • 4 comments

The Lighthouse website is currently vulnerable to brute force password attacks. This is an especially critical issue for admin and moderator (once this role is implemented) users who have control over the platform.

Before a larger beta can be launched, the following or equivalents of the following may need to be implemented,

  • [ ] Password attempt diminishing returns. Every failed sign in attempt makes the user wait longer before they can try to enter their password again. If certain number of attempts fail, require user to password reset via email.
  • [x] Verify account / password reset via email.
  • [ ] option to enable 2FA.
  • [x] Some kind of CAPTCHA challenge to weed out bots. I think Shadow suggested ReCAPTCHA a while back (#143)

m88youngling avatar Feb 04 '22 15:02 m88youngling

Oh, apparently linking a pull request will make it close instead of marking the task as complete. Fun.

jvyden avatar Feb 05 '22 23:02 jvyden

Do any 2FA services have API available to make this possible fairly easily? I imagine services like Google or Microsoft Authenticator make this possible with 3rd party integrations.

m88youngling avatar Sep 19 '22 15:09 m88youngling

require user to password reset via email

I have a feeling that this can be abused to do a pseudo-DoS attack on a user

TorutheRedFox avatar Sep 19 '22 15:09 TorutheRedFox

require user to password reset via email

I have a feeling that this can be abused to do a pseudo-DoS attack on a user

Good thinking. Beacon's rate limit should stop that from happening, but native rate limit in the code may help. I don't know which is better, or if both can be used.

m88youngling avatar Sep 19 '22 16:09 m88youngling

If we already have server rate limiting on Beacon, are password diminishing returns necessary? Brute force attempts would theoretically already be filtered.

m88youngling avatar Dec 14 '22 03:12 m88youngling

It's probably alright to not have that last feature, so I will go ahead and close this

m88youngling avatar Jan 17 '23 02:01 m88youngling