diff --git a/auth/__init__.py b/auth/__init__.py index abd5589b..c869ba36 100644 --- a/auth/__init__.py +++ b/auth/__init__.py @@ -29,6 +29,7 @@ from .send_email import send_password_reset_email, EmailNotConfiguredError from .update_profile import update_profile from .count_admins import count_admins from .delete_user import delete_user +from .update_email import update_email, EmailUpdateError __all__ = [ "users_db_path", "ensure_users_schema", "is_first_user", @@ -44,4 +45,5 @@ __all__ = [ "consume_password_reset_token", "TOKEN_TTL_MINUTES", "send_password_reset_email", "EmailNotConfiguredError", "update_profile", "count_admins", "delete_user", + "update_email", "EmailUpdateError", ] diff --git a/auth/create_user.py b/auth/create_user.py index ade5d229..3016e39b 100644 --- a/auth/create_user.py +++ b/auth/create_user.py @@ -1,16 +1,13 @@ -import re - import pyotp from werkzeug.security import generate_password_hash from .ensure_schema import ensure_users_schema from .connection import connect +from .email_validation import is_valid_email from .get_user_by_email import get_user_by_email from .is_first_user import is_first_user from .password_strength import password_strength -_EMAIL_RE = re.compile(r"^[^@\s]+@[^@\s]+\.[^@\s]+$") - class UserCreationError(Exception): """Erreur de validation lisible par un humain (affichée telle quelle @@ -29,7 +26,7 @@ def create_user(email, password, nom, prenom): email = (email or "").strip().lower() nom = (nom or "").strip() prenom = (prenom or "").strip() - if not _EMAIL_RE.match(email): + if not is_valid_email(email): raise UserCreationError("Adresse email invalide.") if not nom or not prenom: raise UserCreationError("Nom et prénom sont requis.") diff --git a/auth/email_validation.py b/auth/email_validation.py new file mode 100644 index 00000000..48afcbfd --- /dev/null +++ b/auth/email_validation.py @@ -0,0 +1,7 @@ +import re + +EMAIL_RE = re.compile(r"^[^@\s]+@[^@\s]+\.[^@\s]+$") + + +def is_valid_email(email): + return bool(EMAIL_RE.match((email or "").strip())) diff --git a/auth/update_email.py b/auth/update_email.py new file mode 100644 index 00000000..1ecef335 --- /dev/null +++ b/auth/update_email.py @@ -0,0 +1,23 @@ +from .connection import connect +from .email_validation import is_valid_email +from .get_user_by_email import get_user_by_email + + +class EmailUpdateError(Exception): + """Erreur de validation lisible par un humain (affichée telle quelle + sur la page de profil) — mêmes règles qu'à l'inscription (format + valide, unicité), voir create_user.py.""" + + +def update_email(user_id, new_email): + new_email = (new_email or "").strip().lower() + if not is_valid_email(new_email): + raise EmailUpdateError("Adresse email invalide.") + existing = get_user_by_email(new_email) + if existing and existing["id"] != user_id: + raise EmailUpdateError("Un compte existe déjà avec cette adresse email.") + conn = connect() + conn.execute("UPDATE _users SET email = ? WHERE id = ?", (new_email, user_id)) + conn.commit() + conn.close() + return new_email diff --git a/db/__init__.py b/db/__init__.py index 02ce34f0..0232189c 100644 --- a/db/__init__.py +++ b/db/__init__.py @@ -32,6 +32,7 @@ from .games.game_meta import game_meta from .games.create_game import create_game from .games.update_game_name import update_game_name from .games.delete_game import delete_game +from .games.move_game import move_game from .definitions.list_definitions import list_definitions from .definitions.get_definition import get_definition @@ -63,7 +64,7 @@ from .global_vars.delete_global_variable_by_id import delete_global_variable_by_ __all__ = [ "PROJECTS_DIR", "FIELD_TYPES", "GLOBAL_VARIABLE_TYPES", "slugify", "table_name_for", "game_dir", "db_path", "connect", - "list_games", "game_meta", "create_game", "update_game_name", "delete_game", + "list_games", "game_meta", "create_game", "update_game_name", "delete_game", "move_game", "list_definitions", "get_definition", "create_definition", "rename_definition", "definitions_referencing", "add_field_to_definition", "update_field", "delete_field", "delete_definition", diff --git a/db/games/move_game.py b/db/games/move_game.py new file mode 100644 index 00000000..387ba126 --- /dev/null +++ b/db/games/move_game.py @@ -0,0 +1,23 @@ +import os + +from ..game_dir import game_dir + + +def move_game(old_slug, new_slug): + """Renomme le dossier d'un jeu vers un nouveau slug — utilisé quand un + utilisateur change son adresse email (voir routes/auth/profile.py, + project_slug = slugify(email) pour un compte "user") : le dossier + physique du jeu doit suivre. Rend le nouveau slug unique de la même + façon que create_game() si, par un hasard extrême, il correspond déjà + à un dossier existant. Renvoie le slug FINAL (potentiellement suffixé), + à enregistrer comme nouveau project_slug.""" + if old_slug == new_slug: + return old_slug + base = new_slug + n = 2 + while os.path.isdir(game_dir(base)): + base = f"{new_slug}-{n}" + n += 1 + new_slug = base + os.rename(game_dir(old_slug), game_dir(new_slug)) + return new_slug diff --git a/routes/auth/profile.py b/routes/auth/profile.py index 223bef36..73726aa2 100644 --- a/routes/auth/profile.py +++ b/routes/auth/profile.py @@ -32,6 +32,44 @@ def profile_update_name(): return _render(success="Informations mises à jour.") +@app.route("/profile/update-email", methods=["POST"]) +def profile_update_email(): + user = g.current_user + current_password = request.form.get("current_password", "") + new_email = request.form.get("email", "").strip().lower() + if not auth.verify_password(user, current_password): + return _render(error="Mot de passe incorrect.") + try: + auth.update_email(user["id"], new_email) + except auth.EmailUpdateError as exc: + return _render(error=str(exc)) + + # Le dossier d'un compte "user" porte le nom de son adresse email (voir + # create_user.py/register_2fa.py, project_slug = slugify(email)) — il + # doit suivre le changement, sans quoi project_slug ne correspondrait + # plus à aucun dossier réel. Un "admin" n'a pas de project_slug dédié : + # rien à renommer pour ce rôle. + if user["role"] != "admin" and user.get("project_slug"): + new_slug = db.move_game(user["project_slug"], db.slugify(new_email)) + auth.set_project_slug(user["id"], new_slug) + + return _render(success="Adresse email mise à jour.") + + +@app.route("/profile/regenerate-recovery-codes", methods=["POST"]) +def profile_regenerate_recovery_codes(): + user = g.current_user + current_password = request.form.get("current_password", "") + if not auth.verify_password(user, current_password): + return _render(error="Mot de passe incorrect.") + # Posés en session pour être affichés une seule fois par le modal déjà + # utilisé à l'inscription (voir core/recovery_codes_flash.py et + # templates/base.html) — les 10 anciens codes sont déjà invalidés à ce + # stade (auth.generate_recovery_codes supprime tout jeu précédent). + session["recovery_codes_to_show"] = auth.generate_recovery_codes(user["id"]) + return _render() + + @app.route("/profile/change-password", methods=["POST"]) def profile_change_password(): user = g.current_user diff --git a/templates/auth/profile.html b/templates/auth/profile.html index 0869d285..c773e0e6 100644 --- a/templates/auth/profile.html +++ b/templates/auth/profile.html @@ -12,9 +12,29 @@
+ {{ 'Compte administrateur' if user.role == 'admin' else 'Compte utilisateur' }}. + {% if user.role != 'admin' %}Le dossier de ton jeu ({{ user.project_slug }}) est renommé automatiquement si tu changes d'adresse.{% endif %} +
+ +Adresse email : {{ user.email }} ({{ 'compte administrateur' if user.role == 'admin' else 'compte utilisateur' }})