Ajoute la régénération des codes de récupération et le changement d'email
Depuis /profile : - "Codes de récupération 2FA" : régénère les 10 codes (invalide les 10 précédents d'un coup, voir auth.generate_recovery_codes) et les affiche une seule fois via le même modal qu'à l'inscription (session flash, core/recovery_codes_flash.py) — mot de passe actuel requis. - "Adresse email" : change l'adresse (mêmes règles qu'à l'inscription — format valide, unicité — voir auth/email_validation.py, désormais partagé avec create_user.py au lieu d'être dupliqué). Pour un compte "user", le dossier de son unique projet porte le nom de son adresse (project_slug = slugify(email)) : il est renommé pour suivre le changement (db/games/move_game.py, même logique d'unicité par suffixe que create_game()), sans quoi project_slug ne correspondrait plus à aucun dossier réel. Un "admin" n'a pas de project_slug dédié : rien n'est renommé pour ce rôle. 18 nouveaux tests (tests/test_profile.py) : mauvais mot de passe refusé pour les deux actions, mauvais format/email déjà pris refusés, dossier bien renommé et project_slug mis à jour, connexion possible avec la nouvelle adresse, anciens codes de récupération bien invalidés après régénération, modal jamais réaffiché deux fois. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
This commit is contained in:
william
co-authored by
Claude Sonnet 5
Claude Sonnet 5
parent
5c2d7c49cd
commit
c8c949c430
@@ -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",
|
||||
]
|
||||
|
||||
+2
-5
@@ -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.")
|
||||
|
||||
@@ -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()))
|
||||
@@ -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
|
||||
+2
-1
@@ -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",
|
||||
|
||||
@@ -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
|
||||
@@ -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
|
||||
|
||||
@@ -12,9 +12,29 @@
|
||||
<div class="notification is-success">{{ success }}</div>
|
||||
{% endif %}
|
||||
|
||||
<div class="box">
|
||||
<h2 class="title is-5">Adresse email</h2>
|
||||
<p class="hint mb-4">
|
||||
{{ '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 %}
|
||||
</p>
|
||||
<form method="post" action="{{ url_for('profile_update_email') }}" autocomplete="off">
|
||||
<div class="field">
|
||||
<label class="label">Nouvelle adresse email</label>
|
||||
<div class="control"><input class="input" type="email" name="email" value="{{ user.email }}" required></div>
|
||||
</div>
|
||||
<div class="field">
|
||||
<label class="label">Mot de passe actuel</label>
|
||||
<div class="control"><input class="input" type="password" name="current_password" required autocomplete="current-password"></div>
|
||||
</div>
|
||||
<div class="field mt-4">
|
||||
<button type="submit" class="button primary">Changer d'adresse email</button>
|
||||
</div>
|
||||
</form>
|
||||
</div>
|
||||
|
||||
<div class="box">
|
||||
<h2 class="title is-5">Informations</h2>
|
||||
<p class="hint mb-4">Adresse email : <strong>{{ user.email }}</strong> ({{ 'compte administrateur' if user.role == 'admin' else 'compte utilisateur' }})</p>
|
||||
<form method="post" action="{{ url_for('profile_update_name') }}">
|
||||
<div class="field">
|
||||
<label class="label">Prénom</label>
|
||||
@@ -61,6 +81,20 @@
|
||||
</form>
|
||||
</div>
|
||||
|
||||
<div class="box">
|
||||
<h2 class="title is-5">Codes de récupération 2FA</h2>
|
||||
<p class="hint mb-4">Régénérer tes codes remplace immédiatement les 10 précédents par 10 nouveaux — les anciens ne fonctionneront plus. Les nouveaux codes ne sont affichés qu'une seule fois, juste après.</p>
|
||||
<form method="post" action="{{ url_for('profile_regenerate_recovery_codes') }}" autocomplete="off">
|
||||
<div class="field">
|
||||
<label class="label">Mot de passe actuel</label>
|
||||
<div class="control"><input class="input" type="password" name="current_password" required autocomplete="current-password"></div>
|
||||
</div>
|
||||
<div class="field mt-4">
|
||||
<button type="submit" class="button primary">Régénérer mes 10 codes de récupération</button>
|
||||
</div>
|
||||
</form>
|
||||
</div>
|
||||
|
||||
<div class="box" style="border:1px solid #b33;">
|
||||
<h2 class="title is-5">Supprimer mon compte</h2>
|
||||
<p class="hint mb-4">
|
||||
|
||||
@@ -141,3 +141,123 @@ def test_the_only_admin_account_cannot_be_deleted(client):
|
||||
assert "administrateur" in resp.get_data(as_text=True)
|
||||
import tests.conftest as conftest_module
|
||||
assert auth.get_user_by_id(conftest_module._TEST_ADMIN_ID) is not None
|
||||
|
||||
|
||||
def test_regenerate_recovery_codes_requires_correct_password(anon_client):
|
||||
_register(anon_client, "regencodeswrong@example.com")
|
||||
_confirm_2fa(anon_client)
|
||||
try:
|
||||
resp = anon_client.post("/profile/regenerate-recovery-codes", data={"current_password": "wrong"})
|
||||
assert "incorrect" in resp.get_data(as_text=True)
|
||||
finally:
|
||||
_cleanup_project("regencodeswrong@example.com")
|
||||
|
||||
|
||||
def test_regenerate_recovery_codes_replaces_the_old_ones_and_shows_them_once(anon_client):
|
||||
_register(anon_client, "regencodesok@example.com")
|
||||
_confirm_2fa(anon_client)
|
||||
user = auth.get_user_by_email("regencodesok@example.com")
|
||||
try:
|
||||
old_codes = auth.generate_recovery_codes(user["id"])
|
||||
|
||||
resp = anon_client.post("/profile/regenerate-recovery-codes", data={"current_password": "Sup3r$ecret!"})
|
||||
html = resp.get_data(as_text=True)
|
||||
assert "recoveryCodesModal" in html
|
||||
|
||||
# Jamais réaffichés sur un chargement suivant.
|
||||
html_again = anon_client.get("/profile").get_data(as_text=True)
|
||||
assert "recoveryCodesModal" not in html_again
|
||||
|
||||
# Les anciens codes ne fonctionnent plus, mais le nouveau lot,
|
||||
# visible dans le modal ci-dessus, doit être réellement stocké en
|
||||
# base — vérifié indirectement via verify_recovery_code : aucun des
|
||||
# anciens codes ne doit plus matcher.
|
||||
for old_code in old_codes:
|
||||
assert not auth.verify_recovery_code(user["id"], old_code)
|
||||
finally:
|
||||
_cleanup_project("regencodesok@example.com")
|
||||
|
||||
|
||||
def test_update_email_requires_correct_password(anon_client):
|
||||
_register(anon_client, "emailwrongpw@example.com")
|
||||
_confirm_2fa(anon_client)
|
||||
try:
|
||||
resp = anon_client.post("/profile/update-email", data={"email": "new@example.com", "current_password": "wrong"})
|
||||
assert "incorrect" in resp.get_data(as_text=True)
|
||||
assert auth.get_user_by_email("emailwrongpw@example.com") is not None
|
||||
finally:
|
||||
_cleanup_project("emailwrongpw@example.com")
|
||||
|
||||
|
||||
def test_update_email_rejects_invalid_format(anon_client):
|
||||
_register(anon_client, "emailinvalid@example.com")
|
||||
_confirm_2fa(anon_client)
|
||||
try:
|
||||
resp = anon_client.post("/profile/update-email", data={"email": "not-an-email", "current_password": "Sup3r$ecret!"})
|
||||
assert "invalide" in resp.get_data(as_text=True)
|
||||
finally:
|
||||
_cleanup_project("emailinvalid@example.com")
|
||||
|
||||
|
||||
def test_update_email_rejects_an_email_already_used(anon_client):
|
||||
_register(anon_client, "emailtaken1@example.com")
|
||||
_confirm_2fa(anon_client)
|
||||
anon_client.post("/logout")
|
||||
_register(anon_client, "emailtaken2@example.com")
|
||||
_confirm_2fa(anon_client)
|
||||
try:
|
||||
resp = anon_client.post("/profile/update-email", data={"email": "emailtaken1@example.com", "current_password": "Sup3r$ecret!"})
|
||||
assert "existe déjà" in resp.get_data(as_text=True)
|
||||
finally:
|
||||
_cleanup_project("emailtaken1@example.com")
|
||||
_cleanup_project("emailtaken2@example.com")
|
||||
|
||||
|
||||
def test_update_email_renames_the_user_project_folder(anon_client):
|
||||
import os
|
||||
|
||||
_register(anon_client, "oldmail@example.com")
|
||||
_confirm_2fa(anon_client)
|
||||
old_user = auth.get_user_by_email("oldmail@example.com")
|
||||
old_slug = old_user["project_slug"]
|
||||
assert os.path.isdir(db.game_dir(old_slug))
|
||||
try:
|
||||
resp = anon_client.post("/profile/update-email", data={"email": "newmail@example.com", "current_password": "Sup3r$ecret!"})
|
||||
assert "mise à jour" in resp.get_data(as_text=True)
|
||||
|
||||
assert auth.get_user_by_email("oldmail@example.com") is None
|
||||
new_user = auth.get_user_by_email("newmail@example.com")
|
||||
assert new_user is not None
|
||||
new_slug = new_user["project_slug"]
|
||||
assert new_slug == db.slugify("newmail@example.com")
|
||||
assert not os.path.isdir(db.game_dir(old_slug))
|
||||
assert os.path.isdir(db.game_dir(new_slug))
|
||||
|
||||
# La connexion se fait désormais avec la nouvelle adresse.
|
||||
anon_client.post("/logout")
|
||||
resp = anon_client.post("/login", data={"email": "newmail@example.com", "password": "Sup3r$ecret!"}, follow_redirects=False)
|
||||
assert resp.status_code == 302
|
||||
assert resp.headers["Location"] == "/login/2fa"
|
||||
finally:
|
||||
_cleanup_project("oldmail@example.com")
|
||||
_cleanup_project("newmail@example.com")
|
||||
|
||||
|
||||
def test_update_email_for_an_admin_does_not_touch_any_project_folder(client, game):
|
||||
"""`client`/`game` (conftest.py) : compte admin, pas de project_slug
|
||||
dédié — changer son adresse ne doit renommer aucun dossier. L'admin de
|
||||
test est PARTAGÉ par toute la session de tests (voir conftest.py) :
|
||||
son adresse est restaurée après coup pour ne pas perturber les autres
|
||||
tests qui s'appuient dessus."""
|
||||
import tests.conftest as conftest_module
|
||||
|
||||
try:
|
||||
resp = client.post("/profile/update-email", data={"email": "newadminmail@example.com", "current_password": "Test1234!"})
|
||||
assert "mise à jour" in resp.get_data(as_text=True)
|
||||
|
||||
user = auth.get_user_by_id(conftest_module._TEST_ADMIN_ID)
|
||||
assert user["email"] == "newadminmail@example.com"
|
||||
assert user["project_slug"] is None
|
||||
assert db.game_meta(game) is not None
|
||||
finally:
|
||||
auth.update_email(conftest_module._TEST_ADMIN_ID, "admin@pytest.local")
|
||||
|
||||
Reference in New Issue
Block a user