Merge pull request #2079 from jbsky/master
fix(auth): prevent login variable overwrite during failed cache cleanup
This commit is contained in:
@@ -275,8 +275,8 @@ class BaseAuth:
|
|||||||
logger.debug("Login failed cache cleanup start (entries: %d)", cache_failed_cleanup_entries)
|
logger.debug("Login failed cache cleanup start (entries: %d)", cache_failed_cleanup_entries)
|
||||||
if cache_failed_cleanup_entries > 0:
|
if cache_failed_cleanup_entries > 0:
|
||||||
for digest in cache_failed_cleanup:
|
for digest in cache_failed_cleanup:
|
||||||
(login, age_failed) = cache_failed_cleanup[digest]
|
(login_expired, age_failed) = cache_failed_cleanup[digest]
|
||||||
logger.debug("Login failed cache entry for user+password expired: '%s' (age: %d > %d sec)", login_cache, age_failed, self._cache_failed_logins_expiry)
|
logger.debug("Login failed cache entry for user+password expired: '%s' (age: %d > %d sec)", login_expired, age_failed, self._cache_failed_logins_expiry)
|
||||||
del self._cache_failed[digest]
|
del self._cache_failed[digest]
|
||||||
self._lock.release()
|
self._lock.release()
|
||||||
logger.debug("Login failed cache investigation finished")
|
logger.debug("Login failed cache investigation finished")
|
||||||
|
|||||||
Reference in New Issue
Block a user