[Rejected] Rewrite of the login rewrite
#6
Login methods and limited PIN guesses were implemented, the rest probably not (I'll try to check it later today):
(2014-09-04, 11:38 AM)Nik101010 Wrote:
  • there are up to 4 database queries depending on the case fetching nearly the same data (failed login count, username, password, updating failed login count, fetching failed login count)
  • Display number of failed login settings does not work (setting to yes does not show the remaining login attempts)
  • The failed login count in the database is only used for the captcha setting, not for blocking the entire login
  • When a captcha is required the login page does not show the captcha the first time it is required

At least the 2nd and 4th bullet should definitely be fixed IMO. 1st - optimization - wouldn't be bad either. 3rd one - not sure what was meant - the user isn't blocked from logging in after X attempts for a specific period of time or?
Reply


Messages In This Thread
Rewrite of the login rewrite - by Nik101010 - 2014-09-04, 11:38 AM
RE: Rewrite of the login rewrite - by Eldenroot - 2014-09-05, 11:06 AM
RE: Rewrite of the login rewrite - by Destroy666 - 2014-11-17, 10:03 PM
RE: Rewrite of the login rewrite - by Nik101010 - 2014-11-17, 10:11 PM
RE: Rewrite of the login rewrite - by Jones H - 2015-06-25, 09:36 AM
RE: Rewrite of the login rewrite - by Destroy666 - 2015-06-25, 09:47 AM
RE: Rewrite of the login rewrite - by Nik101010 - 2015-06-25, 10:08 AM
RE: Rewrite of the login rewrite - by Jones H - 2015-07-02, 08:39 AM
RE: Rewrite of the login rewrite - by Destroy666 - 2015-07-02, 03:02 PM

Forum Jump:


Users browsing this thread: 1 Guest(s)