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

Limit front-end login attempts #273

Closed
pavlenex opened this issue Sep 12, 2018 · 3 comments · Fixed by #274
Closed

Limit front-end login attempts #273

pavlenex opened this issue Sep 12, 2018 · 3 comments · Fixed by #274

Comments

@pavlenex
Copy link
Contributor

It's a good idea to have a way to limit brute-force login attempts on the front-end of BTCPay.

I'm sure many users or even admins won't set 2FA or if we implement U2F (as I suggested earlier in #53).

My suggestions :

  • Limit the number of logins and lock account after few attempts for a certain time or require email verification
  • Add google anti-spam "are you a robot" verification
@Kukks
Copy link
Member

Kukks commented Sep 12, 2018

@bitcoinshirt BTCPayServer uses Asp.Net Identity Core internally which supports lockout. Pr is attached for nicolas to decide. :)

@pavlenex
Copy link
Contributor Author

@Kukks Can you clarify how it will work?
A. Lock the account for x minutes after y attempts?
B. Lock the account entirely and require e-mail verification?

@Kukks
Copy link
Member

Kukks commented Sep 17, 2018

It's not that elaborate @bitcoinshirt
The mechanism allows us to configure how many times a user can attempt to login with credentials before a locking cooldown period is enabled.
The default settings are:

  • 5 minutes cooldown period
  • 5 attempts to trigger lockdown

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

Successfully merging a pull request may close this issue.

2 participants