Nettoyage et PEP8 pour chgpass

This commit is contained in:
Pierre-Elliott Bécue 2015-12-01 02:44:32 +01:00
parent 525c77c2ce
commit 8f715ff771

View file

@ -47,7 +47,7 @@ def check_password(password, no_cracklib=False, dialog=False):
# Nounou mode
if no_cracklib:
if len(password) >= config.password.root_min_len:
return True
return True, msg
else:
upp = 0
low = 0
@ -119,7 +119,7 @@ def check_password(password, no_cracklib=False, dialog=False):
return False, msg
@lc_ldap.shortcuts.with_ldap_conn(retries=2, delay=5, constructor=lc_ldap.shortcuts.lc_ldap_admin)
def change_password(ldap, login=None, verbose=False, no_cracklib=False, **args):
def change_password(ldap, login=None, verbose=False, no_cracklib=False, **kwargs):
"""
Change le mot de passe en fonction des arguments
"""
@ -167,19 +167,23 @@ Tentative de changement du mot de passe de %s par %s.
prenom = "Club"
else:
prenom = user['prenom'][0]
affich_tools.cprint("Changement du mot de passe de %s %s." %
(prenom, user['nom'][0]),
"vert")
affich_tools.cprint(
"Changement du mot de passe de %s %s." % (
prenom,
user['nom'][0]
),
"vert",
)
# Règles du jeu
# (J'ai perdu)
if verbose:
affich_tools.cprint(u"""Règles :
affich_tools.cprint(
u"""Règles :
Longueur standard : %s, root : %s,
Minimums : chiffres : %s, minuscules : %s, majuscules : %s, autres : %s,
Scores de longueur : chiffres : %s, minuscules : %s, majuscules : %s, autres : %s,
Cracklib : %s.""" % (
config.password.min_len,
Cracklib : %s.""" % (config.password.min_len,
config.password.root_min_len,
config.password.min_cif,
config.password.min_low,
@ -189,22 +193,24 @@ Cracklib : %s.""" % (
config.password.low_value,
config.password.upp_value,
config.password.oth_value,
"Oui" * (not no_cracklib) + "Non" * (no_cracklib)
"Oui" * (not no_cracklib) + "Non" * (no_cracklib),
),
'jaune')
'jaune',
)
else:
affich_tools.cprint(u"""Le nouveau mot de passe doit comporter au minimum %s caractères.
affich_tools.cprint(
u"""Le nouveau mot de passe doit comporter au minimum %s caractères.
Il ne doit pas être basé sur un mot du dictionnaire.
Il doit contenir au moins %s chiffre(s), %s minuscule(s),
%s majuscule(s) et au moins %s autre(s) caractère(s).
CTRL+D ou CTRL+C provoquent un abandon.""" %
(
config.password.min_len,
CTRL+D ou CTRL+C provoquent un abandon.""" % (config.password.min_len,
config.password.min_cif,
config.password.min_low,
config.password.min_upp,
config.password.min_oth
), 'jaune')
),
'jaune',
)
try:
while True:
@ -234,22 +240,31 @@ CTRL+D ou CTRL+C provoquent un abandon.""" %
affich_tools.cprint(u"Mot de passe de %s changé." % (user['uid'][0]), "vert")
if __name__ == "__main__":
parser = argparse.ArgumentParser(
description="Recherche dans la base des adhérents",
add_help=False)
parser.add_argument('-h', '--help',
parser = argparse.ArgumentParser(description="Recherche dans la base des adhérents",
add_help=False,
)
parser.add_argument('-h',
'--help',
help="Affiche ce message et quitte.",
action="store_true")
parser.add_argument('-n', '--no-cracklib',
action="store_true",
)
parser.add_argument('-n',
'--no-cracklib',
help="Permet de contourner les règles de choix du mot de passe" +
"(réservé aux nounous).",
action="store_true")
parser.add_argument('-v', '--verbose',
action="store_true",
)
parser.add_argument('-v',
'--verbose',
help="Permet de contourner les règles de choix du mot de passe" +
"(réservé aux nounous).",
action="store_true")
parser.add_argument('login', type=str, nargs="?",
help="L'utilisateur dont on veut changer le mot de passe.")
action="store_true",
)
parser.add_argument('login',
type=str,
nargs="?",
help="L'utilisateur dont on veut changer le mot de passe.",
)
args = parser.parse_args()