mirrors/django
1
0
Fork 0
mirror of https://github.com/django/django.git synced 2025-03-11 09:52:38 +00:00
Code Issues 32 Releases Wiki Activity

Improved performance of loading common passwords in CommonPasswordValidator.

Browse Source 
CommonPasswordValidator.__init__ previously called either splitlines or
readlines, creating an unneeded intermediate list in memory. For large
custom password files, this could be burdensome.
This commit is contained in:
Brad Solomon 2019-05-21 21:36:56 -04:00 committed by Mariusz Felisiak
parent f011d9ea56
commit 28eac41510
1 changed files with 3 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
django/contrib/auth/password_validation.py
View File

@ -171,13 +171,11 @@ class CommonPasswordValidator:
def __init__(self, password_list_path=DEFAULT_PASSWORD_LIST_PATH): def __init__(self, password_list_path=DEFAULT_PASSWORD_LIST_PATH):
try: try:
with gzip.open(str(password_list_path)) as f: with gzip.open(str(password_list_path), 'rt') as f:
common_passwords_lines = f.read().decode().splitlines() self.passwords = {x.strip() for x in f}
except OSError: except OSError:
with open(str(password_list_path)) as f: with open(str(password_list_path)) as f:
common_passwords_lines = f.readlines() self.passwords = {x.strip() for x in f}
self.passwords = {p.strip() for p in common_passwords_lines}
def validate(self, password, user=None): def validate(self, password, user=None):
if password.lower().strip() in self.passwords: if password.lower().strip() in self.passwords: