forked from ScoDoc/ScoDoc
Merge pull request 'check mail address' (#156) from jmplace/ScoDoc-Lille:email_check into master
Reviewed-on: https://scodoc.org/git/viennet/ScoDoc/pulls/156
This commit is contained in:
commit
e6e1835cca
@ -182,11 +182,13 @@ def import_users(users):
|
|||||||
line = line + 1
|
line = line + 1
|
||||||
user_ok, msg = sco_users.check_modif_user(
|
user_ok, msg = sco_users.check_modif_user(
|
||||||
0,
|
0,
|
||||||
|
ignore_optionals=False,
|
||||||
user_name=u["user_name"],
|
user_name=u["user_name"],
|
||||||
nom=u["nom"],
|
nom=u["nom"],
|
||||||
prenom=u["prenom"],
|
prenom=u["prenom"],
|
||||||
email=u["email"],
|
email=u["email"],
|
||||||
roles=u["roles"].split(","),
|
roles=u["roles"].split(","),
|
||||||
|
dept=u["dept"],
|
||||||
)
|
)
|
||||||
if not user_ok:
|
if not user_ok:
|
||||||
append_msg("identifiant '%s' %s" % (u["user_name"], msg))
|
append_msg("identifiant '%s' %s" % (u["user_name"], msg))
|
||||||
@ -196,39 +198,12 @@ def import_users(users):
|
|||||||
u["passwd"] = generate_password()
|
u["passwd"] = generate_password()
|
||||||
#
|
#
|
||||||
# check identifiant
|
# check identifiant
|
||||||
if not re.match(r"^[a-zA-Z0-9@\\\-_\\\.]*$", u["user_name"]):
|
|
||||||
user_ok = False
|
|
||||||
append_msg(
|
|
||||||
"identifiant '%s' invalide (pas d'accents ni de caractères spéciaux)"
|
|
||||||
% u["user_name"]
|
|
||||||
)
|
|
||||||
if len(u["user_name"]) > 64:
|
|
||||||
user_ok = False
|
|
||||||
append_msg(
|
|
||||||
"identifiant '%s' trop long (64 caractères)" % u["user_name"]
|
|
||||||
)
|
|
||||||
if len(u["nom"]) > 64:
|
|
||||||
user_ok = False
|
|
||||||
append_msg("nom '%s' trop long (64 caractères)" % u["nom"])
|
|
||||||
if len(u["prenom"]) > 64:
|
|
||||||
user_ok = False
|
|
||||||
append_msg("prenom '%s' trop long (64 caractères)" % u["prenom"])
|
|
||||||
if len(u["email"]) > 120:
|
|
||||||
user_ok = False
|
|
||||||
append_msg("email '%s' trop long (120 caractères)" % u["email"])
|
|
||||||
# check that tha same user_name has not already been described in this import
|
|
||||||
if u["user_name"] in created.keys():
|
if u["user_name"] in created.keys():
|
||||||
user_ok = False
|
user_ok = False
|
||||||
append_msg(
|
append_msg(
|
||||||
"l'utilisateur '%s' a déjà été décrit ligne %s"
|
"l'utilisateur '%s' a déjà été décrit ligne %s"
|
||||||
% (u["user_name"], created[u["user_name"]]["line"])
|
% (u["user_name"], created[u["user_name"]]["line"])
|
||||||
)
|
)
|
||||||
# check département
|
|
||||||
if u["dept"] != "":
|
|
||||||
dept = Departement.query.filter_by(acronym=u["dept"]).first()
|
|
||||||
if dept is None:
|
|
||||||
user_ok = False
|
|
||||||
append_msg("département '%s' inexistant" % u["dept"])
|
|
||||||
# check roles / ignore whitespaces around roles / build roles_string
|
# check roles / ignore whitespaces around roles / build roles_string
|
||||||
# roles_string (expected by User) appears as column 'roles' in excel file
|
# roles_string (expected by User) appears as column 'roles' in excel file
|
||||||
roles_list = []
|
roles_list = []
|
||||||
|
@ -29,13 +29,14 @@
|
|||||||
"""
|
"""
|
||||||
|
|
||||||
# Anciennement ZScoUsers.py, fonctions de gestion des données réécrite avec flask/SQLAlchemy
|
# Anciennement ZScoUsers.py, fonctions de gestion des données réécrite avec flask/SQLAlchemy
|
||||||
|
import re
|
||||||
|
|
||||||
from flask import url_for, g, request
|
from flask import url_for, g, request
|
||||||
from flask_login import current_user
|
from flask_login import current_user
|
||||||
|
|
||||||
import cracklib # pylint: disable=import-error
|
import cracklib # pylint: disable=import-error
|
||||||
|
|
||||||
from app import db
|
from app import db, Departement
|
||||||
|
|
||||||
from app.auth.models import Permission
|
from app.auth.models import Permission
|
||||||
from app.auth.models import User
|
from app.auth.models import User
|
||||||
@ -384,7 +385,16 @@ def user_info_page(user_name=None):
|
|||||||
return "\n".join(H) + F
|
return "\n".join(H) + F
|
||||||
|
|
||||||
|
|
||||||
def check_modif_user(edit, user_name="", nom="", prenom="", email="", roles=[]):
|
def check_modif_user(
|
||||||
|
edit,
|
||||||
|
ignore_optionals=False,
|
||||||
|
user_name="",
|
||||||
|
nom="",
|
||||||
|
prenom="",
|
||||||
|
email="",
|
||||||
|
dept="",
|
||||||
|
roles=[],
|
||||||
|
):
|
||||||
"""Vérifie que cet utilisateur peut être créé (edit=0) ou modifié (edit=1)
|
"""Vérifie que cet utilisateur peut être créé (edit=0) ou modifié (edit=1)
|
||||||
Cherche homonymes.
|
Cherche homonymes.
|
||||||
returns (ok, msg)
|
returns (ok, msg)
|
||||||
@ -392,17 +402,44 @@ def check_modif_user(edit, user_name="", nom="", prenom="", email="", roles=[]):
|
|||||||
(si ok est faux, l'utilisateur peut quand même forcer la creation)
|
(si ok est faux, l'utilisateur peut quand même forcer la creation)
|
||||||
- msg: message warning a presenter l'utilisateur
|
- msg: message warning a presenter l'utilisateur
|
||||||
"""
|
"""
|
||||||
if not user_name or not nom or not prenom:
|
MSG_OPT = """Attention: %s (vous pouvez forcer l'opération en cochant "<em>Ignorer les avertissements</em>" en bas de page)"""
|
||||||
return False, "champ requis vide"
|
|
||||||
if not email:
|
|
||||||
return False, "vous devriez indiquer le mail de l'utilisateur créé !"
|
|
||||||
# ce login existe ?
|
# ce login existe ?
|
||||||
user = _user_list(user_name)
|
user = _user_list(user_name)
|
||||||
if edit and not user: # safety net, le user_name ne devrait pas changer
|
if edit and not user: # safety net, le user_name ne devrait pas changer
|
||||||
return False, "identifiant %s inexistant" % user_name
|
return False, "identifiant %s inexistant" % user_name
|
||||||
if not edit and user:
|
if not edit and user:
|
||||||
return False, "identifiant %s déjà utilisé" % user_name
|
return False, "identifiant %s déjà utilisé" % user_name
|
||||||
|
if not user_name or not nom or not prenom:
|
||||||
|
return False, "champ requis vide"
|
||||||
|
if not re.match(r"^[a-zA-Z0-9@\\\-_\\\.]*$", user_name):
|
||||||
|
return (
|
||||||
|
False,
|
||||||
|
"identifiant '%s' invalide (pas d'accents ni de caractères spéciaux)"
|
||||||
|
% user_name,
|
||||||
|
)
|
||||||
|
if ignore_optionals and len(user_name) > 64:
|
||||||
|
return False, "identifiant '%s' trop long (64 caractères)" % user_name
|
||||||
|
if ignore_optionals and len(nom) > 64:
|
||||||
|
return False, "nom '%s' trop long (64 caractères)" % nom + MSG_OPT
|
||||||
|
if ignore_optionals and len(prenom) > 64:
|
||||||
|
return False, "prenom '%s' trop long (64 caractères)" % prenom + MSG_OPT
|
||||||
|
# check that tha same user_name has not already been described in this import
|
||||||
|
if not email:
|
||||||
|
return False, "vous devriez indiquer le mail de l'utilisateur créé !"
|
||||||
|
if len(email) > 120:
|
||||||
|
return False, "email '%s' trop long (120 caractères)" % email
|
||||||
|
if not re.fullmatch(r"\b[A-Za-z0-9._%+-]+@[A-Za-z0-9.-]+\.[A-Z|a-z]{2,}\b", email):
|
||||||
|
return False, "l'adresse mail semble incorrecte"
|
||||||
|
# check département
|
||||||
|
if (
|
||||||
|
ignore_optionals
|
||||||
|
and dept != ""
|
||||||
|
and Departement.query.filter_by(acronym=dept).first() is None
|
||||||
|
):
|
||||||
|
return False, "département '%s' inexistant" % u["dept"] + MSG_OPT
|
||||||
|
if ignore_optionals and not roles:
|
||||||
|
return False, "aucun rôle sélectionné, êtes vous sûr ?" + MSG_OPT
|
||||||
|
# ok
|
||||||
# Des noms/prénoms semblables existent ?
|
# Des noms/prénoms semblables existent ?
|
||||||
nom = nom.lower().strip()
|
nom = nom.lower().strip()
|
||||||
prenom = prenom.lower().strip()
|
prenom = prenom.lower().strip()
|
||||||
@ -422,12 +459,9 @@ def check_modif_user(edit, user_name="", nom="", prenom="", email="", roles=[]):
|
|||||||
"%s %s (pseudo=%s)" % (x.prenom, x.nom, x.user_name)
|
"%s %s (pseudo=%s)" % (x.prenom, x.nom, x.user_name)
|
||||||
for x in similar_users
|
for x in similar_users
|
||||||
]
|
]
|
||||||
),
|
) + MSG_OPT,
|
||||||
)
|
)
|
||||||
# Roles ?
|
# Roles ?
|
||||||
if not roles:
|
|
||||||
return False, "aucun rôle sélectionné, êtes vous sûr ?"
|
|
||||||
# ok
|
|
||||||
return True, ""
|
return True, ""
|
||||||
|
|
||||||
|
|
||||||
|
@ -284,7 +284,7 @@ def create_user_form(user_name=None, edit=0, all_roles=1):
|
|||||||
{
|
{
|
||||||
"input_type": "separator",
|
"input_type": "separator",
|
||||||
"title": "L'utilisateur appartient au département %s"
|
"title": "L'utilisateur appartient au département %s"
|
||||||
% auth_dept,
|
% auth_dept,
|
||||||
},
|
},
|
||||||
)
|
)
|
||||||
)
|
)
|
||||||
@ -295,7 +295,7 @@ def create_user_form(user_name=None, edit=0, all_roles=1):
|
|||||||
{
|
{
|
||||||
"input_type": "separator",
|
"input_type": "separator",
|
||||||
"title": "L'utilisateur sera crée dans le département %s"
|
"title": "L'utilisateur sera crée dans le département %s"
|
||||||
% auth_dept,
|
% auth_dept,
|
||||||
},
|
},
|
||||||
)
|
)
|
||||||
)
|
)
|
||||||
@ -381,24 +381,20 @@ def create_user_form(user_name=None, edit=0, all_roles=1):
|
|||||||
H.append(tf_error_message("""Erreur: %s""" % err))
|
H.append(tf_error_message("""Erreur: %s""" % err))
|
||||||
return "\n".join(H) + "\n" + tf[1] + F
|
return "\n".join(H) + "\n" + tf[1] + F
|
||||||
|
|
||||||
if not force:
|
ok, msg = sco_users.check_modif_user(
|
||||||
ok, msg = sco_users.check_modif_user(
|
edit,
|
||||||
edit,
|
ignore_optionals= force,
|
||||||
user_name=user_name,
|
user_name=user_name,
|
||||||
nom=vals["nom"],
|
nom=vals["nom"],
|
||||||
prenom=vals["prenom"],
|
prenom=vals["prenom"],
|
||||||
email=vals["email"],
|
email=vals["email"],
|
||||||
roles=vals["roles"],
|
roles=vals["roles"],
|
||||||
|
)
|
||||||
|
if not ok:
|
||||||
|
H.append(
|
||||||
|
tf_error_message(msg)
|
||||||
)
|
)
|
||||||
if not ok:
|
return "\n".join(H) + "\n" + tf[1] + F
|
||||||
H.append(
|
|
||||||
tf_error_message(
|
|
||||||
"""Attention: %s (vous pouvez forcer l'opération en cochant "<em>Ignorer les avertissements</em>" en bas de page)"""
|
|
||||||
% msg
|
|
||||||
)
|
|
||||||
)
|
|
||||||
|
|
||||||
return "\n".join(H) + "\n" + tf[1] + F
|
|
||||||
|
|
||||||
if "date_expiration" in vals:
|
if "date_expiration" in vals:
|
||||||
try:
|
try:
|
||||||
@ -617,9 +613,9 @@ def form_change_password(user_name=None):
|
|||||||
# check access
|
# check access
|
||||||
if not can_handle_passwd(u):
|
if not can_handle_passwd(u):
|
||||||
return (
|
return (
|
||||||
"\n".join(H)
|
"\n".join(H)
|
||||||
+ "<p>Vous n'avez pas la permission de changer ce mot de passe</p>"
|
+ "<p>Vous n'avez pas la permission de changer ce mot de passe</p>"
|
||||||
+ F
|
+ F
|
||||||
)
|
)
|
||||||
#
|
#
|
||||||
H.append(
|
H.append(
|
||||||
@ -685,7 +681,7 @@ def change_password(user_name, password, password2):
|
|||||||
"<h2>Changement effectué !</h2><p>Ne notez pas ce mot de passe, mais mémorisez le !</p><p>Rappel: il est <b>interdit</b> de communiquer son mot de passe à un tiers, même si c'est un collègue de confiance !</p><p><b>Si vous n'êtes pas administrateur, le système va vous redemander votre login et nouveau mot de passe au prochain accès.</b></p>"
|
"<h2>Changement effectué !</h2><p>Ne notez pas ce mot de passe, mais mémorisez le !</p><p>Rappel: il est <b>interdit</b> de communiquer son mot de passe à un tiers, même si c'est un collègue de confiance !</p><p><b>Si vous n'êtes pas administrateur, le système va vous redemander votre login et nouveau mot de passe au prochain accès.</b></p>"
|
||||||
)
|
)
|
||||||
return (
|
return (
|
||||||
"""<?xml version="1.0" encoding="%s"?>
|
"""<?xml version="1.0" encoding="%s"?>
|
||||||
<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd">
|
<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd">
|
||||||
<html>
|
<html>
|
||||||
<head>
|
<head>
|
||||||
@ -693,10 +689,10 @@ def change_password(user_name, password, password2):
|
|||||||
<meta http-equiv="Content-Type" content="text/html; charset=%s" />
|
<meta http-equiv="Content-Type" content="text/html; charset=%s" />
|
||||||
<body><h1>Mot de passe changé !</h1>
|
<body><h1>Mot de passe changé !</h1>
|
||||||
"""
|
"""
|
||||||
% (scu.SCO_ENCODING, scu.SCO_ENCODING)
|
% (scu.SCO_ENCODING, scu.SCO_ENCODING)
|
||||||
+ "\n".join(H)
|
+ "\n".join(H)
|
||||||
+ '<a href="%s" class="stdlink">Continuer</a></body></html>'
|
+ '<a href="%s" class="stdlink">Continuer</a></body></html>'
|
||||||
% scu.ScoURL()
|
% scu.ScoURL()
|
||||||
)
|
)
|
||||||
return html_sco_header.sco_header() + "\n".join(H) + F
|
return html_sco_header.sco_header() + "\n".join(H) + F
|
||||||
|
|
||||||
@ -711,7 +707,7 @@ def toggle_active_user(user_name: str = None):
|
|||||||
raise ScoValueError("invalid user_name")
|
raise ScoValueError("invalid user_name")
|
||||||
form = DeactivateUserForm()
|
form = DeactivateUserForm()
|
||||||
if (
|
if (
|
||||||
request.method == "POST" and form.cancel.data
|
request.method == "POST" and form.cancel.data
|
||||||
): # if cancel button is clicked, the form.cancel.data will be True
|
): # if cancel button is clicked, the form.cancel.data will be True
|
||||||
# flash
|
# flash
|
||||||
return redirect(url_for("users.index_html", scodoc_dept=g.scodoc_dept))
|
return redirect(url_for("users.index_html", scodoc_dept=g.scodoc_dept))
|
||||||
|
Loading…
Reference in New Issue
Block a user