From dd4f9c3c55fbf6d39e795598723c4f598550035e Mon Sep 17 00:00:00 2001 From: Adam Belkacemi Date: Fri, 9 Oct 2026 22:18:53 +0200 Subject: [PATCH] =?UTF-8?q?permet=20aux=20=C3=A9l=C3=A8ves=20de=20rejoindr?= =?UTF-8?q?e=20leur=20classe=20avec=20son=20code,=20et=20les=20fiches=20en?= =?UTF-8?q?=20lot?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit L'élève saisit le code de classe et son prénom, le prof accepte, l'élève voit une seule fois son identifiant et son code. Aucun e-mail. Le prof peut aussi régénérer les fiches de plusieurs élèves d'un coup. Procédure pour prouver la copie de la clé des sauvegardes. Co-Authored-By: Claude Opus 5.5 --- docs/dette.md | 2 + docs/journal.md | 13 ++ docs/runbooks/sauvegardes.md | 22 +++- .../app/Http/Controllers/ClasseController.php | 76 +++++++++--- .../Controllers/RejoindreClasseController.php | 115 ++++++++++++++++++ .../Middleware/PolitiqueSecuriteContenu.php | 2 +- portail/app/Models/Classe.php | 5 + portail/app/Models/DemandeInscription.php | 52 ++++++++ .../app/Services/GenerateurCompteEleve.php | 25 ++++ ...0000_create_demandes_inscription_table.php | 32 +++++ .../resources/views/auth/connexion.blade.php | 1 + .../resources/views/confidentialite.blade.php | 2 + .../views/mes-classes/show.blade.php | 57 ++++++++- .../views/rejoindre/attente.blade.php | 21 ++++ .../views/rejoindre/bienvenue.blade.php | 16 +++ .../views/rejoindre/formulaire.blade.php | 28 +++++ .../views/rejoindre/refusee.blade.php | 10 ++ portail/routes/console.php | 4 + portail/routes/web.php | 12 ++ portail/tests/Feature/PagesPubliquesTest.php | 6 +- portail/tests/Feature/RejoindreClasseTest.php | 98 +++++++++++++++ 21 files changed, 573 insertions(+), 26 deletions(-) create mode 100644 portail/app/Http/Controllers/RejoindreClasseController.php create mode 100644 portail/app/Models/DemandeInscription.php create mode 100644 portail/database/migrations/2026_10_09_200000_create_demandes_inscription_table.php create mode 100644 portail/resources/views/rejoindre/attente.blade.php create mode 100644 portail/resources/views/rejoindre/bienvenue.blade.php create mode 100644 portail/resources/views/rejoindre/formulaire.blade.php create mode 100644 portail/resources/views/rejoindre/refusee.blade.php create mode 100644 portail/tests/Feature/RejoindreClasseTest.php diff --git a/docs/dette.md b/docs/dette.md index c479fdc..284b697 100644 --- a/docs/dette.md +++ b/docs/dette.md @@ -25,3 +25,5 @@ retirer la ligne (et le noter dans le journal) quand c'est fait. | Slogans « du lycée au master » | Le catalogue va aujourd'hui du lycée au BUT. Positionnement laissé tel quel : décision éditoriale d'Adam. | À décider | Étape 14 | | Ancien `storage/logs/laravel.log` | Fichier unique d'avant la rotation quotidienne (étape 14) : jamais purgé, à supprimer ou archiver (accord d'Adam). | Dès que possible | Étape 14 | | Pages d'aperçu publiques (32) | Signalées comme aperçus avec données fictives ; à retirer ou remplacer au fur et à mesure que les fonctions réelles arrivent. | Au fil du bloc B | Étape 14 | +| Invitation des élèves majeurs par e-mail (V2 du CLAUDE.md) | Demande un service d'envoi d'e-mails (compte ouvert par Adam, DNS SPF/DKIM). Jamais pour les mineurs. | V2 | 2026-10-09 | +| Code de classe : caractères ambigus | Généré avec `Str::random` (0/O, 1/I/l possibles), contrairement aux codes élèves. Utiliser le même alphabet sans ambiguïté. | Bloc B | 2026-10-09 | diff --git a/docs/journal.md b/docs/journal.md index d8f0f15..09c104d 100644 --- a/docs/journal.md +++ b/docs/journal.md @@ -612,3 +612,16 @@ Adam a ouvert `/ide` et transmis la console du navigateur, plus sa fiche INPI. - **Mentions légales complétées** d'après la fiche INPI : Adam Belkacemi, entrepreneur individuel (EI), nom commercial Esquisse Formation, SIREN 838 093 268, SIRET 838 093 268 00087, 30 rue Lorenzaccio, 38100 Grenoble. La date de naissance de la fiche n'est pas reprise. Restent à compléter : l'article « Disponibilité et responsabilité » des CGU, et un éventuel délégué à la protection des données. Vérifié en production : `/ide` 1 script en ligne, 0 erreur de syntaxe, `sandbox="allow-scripts"` ; Mulish déclarée par le portail ; aucun `@import` vers Google ; `/vendor/xterm/xterm.js` et `/vendor/codemirror/codemirror.js` en 200. 35 tests Pest. + +## 2026-10-09 — Accès des élèves : rejoindre la classe avec son code, fiches en lot + +**Question d'Adam** : comment l'élève obtient-il un accès après sa création ? Faut-il un e-mail automatique pour créer un mot de passe ? + +**Réponse et décision** : pas d'e-mail. Le CLAUDE.md l'interdit pour les mineurs (minimisation) et range l'invitation par e-mail en V2 (pour les majeurs) ; le serveur n'a pas non plus de service d'envoi (il faudrait un compte chez un prestataire, ouvert par Adam). Le parcours existant (identifiant + code secret + QR à usage unique, sur une fiche imprimable) restait valable, avec deux défauts : la fiche ne s'affichait qu'une fois, et le code de classe affiché ne servait à rien. Adam n'ayant pas répondu au choix proposé, j'ai construit les deux options sans dépendance extérieure : + +- **Rejoindre avec le code de classe** (façon n°1 du CLAUDE.md) : `/rejoindre` (code + prénom + nom facultatif, limité à 10 envois par minute) → demande visible sur la page de la classe (rafraîchie toute seule quand une demande arrive) → le prof accepte → le navigateur de l'élève affiche **une seule fois** son identifiant et son code, et le connecte. Le code en clair n'existe que dans la demande (chiffré par `APP_KEY`), supprimée à l'affichage ; une demande non traitée est supprimée au bout d'une heure (`model:prune`, tous les quarts d'heure). Seul le navigateur qui a envoyé la demande peut la suivre (jeton en session, empreinte SHA-256 en base). Classe archivée ou code faux : refus. Le prof projette le code ou un QR qui pré-remplit le code. +- **Fiches en lot** : cases à cocher dans la liste des élèves, « Nouvelles fiches pour la sélection » (nouveaux codes et QR, anciens codes invalidés). +- Lien « Rejoins ta classe avec son code » sur la page de connexion ; politique de confidentialité mise à jour (nouvelle donnée : la demande, au plus une heure). +- 6 tests Pest (41 au total) ; vérifié en production : `/rejoindre` 200 avec CSP stricte, code faux refusé avec message, table créée, nettoyage planifié. + +**Clé des sauvegardes** : je ne peux ni ne dois voir la copie d'Adam. Procédure de preuve ajoutée au runbook des sauvegardes : un message chiffré sur le serveur avec la clé publique, que la copie doit déchiffrer sur le Mac. Fichier de test déposé sur le Bureau d'Adam. Corrigé aussi la commande de copie du runbook (un `scp` d'un fichier de root échouait). diff --git a/docs/runbooks/sauvegardes.md b/docs/runbooks/sauvegardes.md index 165342b..726a334 100644 --- a/docs/runbooks/sauvegardes.md +++ b/docs/runbooks/sauvegardes.md @@ -21,7 +21,7 @@ révèle rien. ## Où est la clé de déchiffrement -- Clé privée age : `/root/.age/cle-sauvegardes.txt`, permissions `600`, **ne quitte jamais le serveur**. +- Clé privée age : `/root/.age/cle-sauvegardes.txt`, permissions `600`. Elle ne quitte le serveur que pour la copie de secours d'Adam (ci-dessous), jamais vers Git ni ailleurs. - Clé publique (sans risque à diffuser) : voir la sortie de `age-keygen` lors du déploiement, ou : ``` sudo grep 'public key' /root/.age/cle-sauvegardes.txt @@ -34,9 +34,23 @@ façon de déchiffrer les sauvegardes existantes. Exemple pour la récupérer ( lancer depuis le Mac, le fichier arrive en clair sur le Mac donc à stocker immédiatement dans un endroit sûr puis à effacer du téléchargement) : ``` -scp paillasse:/root/.age/cle-sauvegardes.txt ~/Desktop/ -# puis la mettre immédiatement dans un gestionnaire de mots de passe, -# et supprimer la copie en clair sur le Desktop. +# La clé appartient à root : un simple scp échouerait. La redirection écrit +# le fichier sur le Mac sans jamais l'afficher. +ssh paillasse 'sudo cat /root/.age/cle-sauvegardes.txt' > ~/Desktop/cle-sauvegardes-syllabus.txt +``` + +Puis la ranger dans le gestionnaire de mots de passe (pièce jointe d'une +fiche « Syllabus — clé des sauvegardes ») ou sur une clé USB chiffrée, et +**supprimer la copie du Bureau**. + +### Prouver que la copie fonctionne (sans montrer la clé) + +Sur le serveur, chiffrer un message de test avec la clé **publique** ; sur le +Mac, le déchiffrer avec la **copie** (installer `age` : `brew install age`) : + +``` +ssh paillasse 'sudo bash -c "PUB=\$(grep -oP \"(?<=public key: )age1\\S+\" /root/.age/cle-sauvegardes.txt); echo \"copie OK \$(date +%F)\" | age -r \$PUB"' > ~/Desktop/test-cle.age +age -d -i ~/Desktop/test-cle.age # doit afficher « copie OK … » ``` ## Identifiants Dedibackup diff --git a/portail/app/Http/Controllers/ClasseController.php b/portail/app/Http/Controllers/ClasseController.php index 58d0a71..0e3b956 100644 --- a/portail/app/Http/Controllers/ClasseController.php +++ b/portail/app/Http/Controllers/ClasseController.php @@ -3,9 +3,11 @@ namespace App\Http\Controllers; use App\Models\Classe; +use App\Models\DemandeInscription; use App\Models\Eleve; use App\Models\Seance; use App\Services\GenerateurCompteEleve; +use Illuminate\Http\JsonResponse; use Illuminate\Http\RedirectResponse; use Illuminate\Http\Request; use Illuminate\Support\Facades\Auth; @@ -30,6 +32,7 @@ class ClasseController extends Controller return view('mes-classes.show', [ 'classe' => $classe->load('eleves'), + 'demandes' => $classe->demandesInscription()->where('statut', DemandeInscription::EN_ATTENTE)->oldest()->get(), ]); } @@ -104,25 +107,64 @@ class ClasseController extends Controller */ private function creerEleve(Classe $classe, string $prenom, ?string $nom): array { - $identifiant = $this->generateur->genererIdentifiant($prenom, $nom); - $codeSecret = $this->generateur->genererCodeSecret(); - $jetonQr = Str::random(32); + $fiche = $this->generateur->creerDans($classe, $prenom, $nom); + unset($fiche['eleve']); - $classe->eleves()->create([ - 'prenom' => $prenom, - 'nom' => $nom, - 'identifiant' => $identifiant, - 'code_secret' => Hash::make($codeSecret), - 'jeton_qr' => $jetonQr, - ]); + return $fiche; + } - return [ - 'identifiant' => $identifiant, - 'prenom' => $prenom, - 'nom' => $nom, - 'code_secret' => $codeSecret, - 'jeton_qr' => $jetonQr, - ]; + public function accepterDemande(Classe $classe, DemandeInscription $demande): RedirectResponse + { + $this->verifierProprietaire($classe); + abort_if($demande->classe_id !== $classe->id, 404); + + if ($demande->statut === DemandeInscription::EN_ATTENTE && ! $classe->estArchivee()) { + $fiche = $this->generateur->creerDans($classe, $demande->prenom, $demande->nom); + $demande->forceFill([ + 'statut' => DemandeInscription::ACCEPTEE, + 'eleve_id' => $fiche['eleve']->id, + 'code_secret_chiffre' => $fiche['code_secret'], + ])->save(); + } + + return redirect()->route('mes-classes.show', $classe); + } + + public function refuserDemande(Classe $classe, DemandeInscription $demande): RedirectResponse + { + $this->verifierProprietaire($classe); + abort_if($demande->classe_id !== $classe->id, 404); + + $demande->forceFill(['statut' => DemandeInscription::REFUSEE])->save(); + + return redirect()->route('mes-classes.show', $classe); + } + + public function nombreDemandes(Classe $classe): JsonResponse + { + $this->verifierProprietaire($classe); + + return response()->json(['en_attente' => $classe->demandesInscription()->where('statut', DemandeInscription::EN_ATTENTE)->count()]); + } + + /** + * Nouvelles fiches (nouveau code, nouveau QR) pour les élèves cochés : + * la fiche n'est visible qu'une fois, ceci permet de la refaire en lot. + */ + public function nouvellesFiches(Request $request, Classe $classe): RedirectResponse + { + $this->verifierProprietaire($classe); + $ids = $request->validate(['eleves' => ['required', 'array'], 'eleves.*' => ['integer']])['eleves']; + + $fiches = []; + foreach ($classe->eleves()->whereIn('id', $ids)->orderBy('prenom')->get() as $eleve) { + $codeSecret = $this->generateur->genererCodeSecret(); + $jetonQr = Str::random(32); + $eleve->forceFill(['code_secret' => Hash::make($codeSecret), 'jeton_qr' => $jetonQr])->save(); + $fiches[] = ['identifiant' => $eleve->identifiant, 'prenom' => $eleve->prenom, 'nom' => $eleve->nom, 'code_secret' => $codeSecret, 'jeton_qr' => $jetonQr]; + } + + return redirect()->route('mes-classes.show', $classe)->with('fiches', $fiches); } /** diff --git a/portail/app/Http/Controllers/RejoindreClasseController.php b/portail/app/Http/Controllers/RejoindreClasseController.php new file mode 100644 index 0000000..a20ebf2 --- /dev/null +++ b/portail/app/Http/Controllers/RejoindreClasseController.php @@ -0,0 +1,115 @@ + Str::upper((string) $request->query('code', ''))]); + } + + public function demander(Request $request): RedirectResponse + { + $donnees = $request->validate([ + 'code' => ['required', 'string', 'size:6'], + 'prenom' => ['required', 'string', 'max:100'], + 'nom' => ['nullable', 'string', 'max:100'], + ]); + + $classe = Classe::where('code_classe', Str::upper($donnees['code']))->first(); + + if ($classe === null || $classe->estArchivee()) { + return back()->withInput()->withErrors(['code' => 'Ce code ne correspond à aucune classe ouverte. Vérifie-le auprès de ton prof.']); + } + + $jeton = Str::random(40); + DemandeInscription::create([ + 'classe_id' => $classe->id, + 'prenom' => trim($donnees['prenom']), + 'nom' => isset($donnees['nom']) ? trim($donnees['nom']) : null, + 'jeton_empreinte' => DemandeInscription::empreinte($jeton), + 'statut' => DemandeInscription::EN_ATTENTE, + ]); + $request->session()->put(self::CLE_SESSION, $jeton); + + return redirect()->route('rejoindre.attente'); + } + + public function attente(Request $request): View|RedirectResponse + { + $demande = $this->demandeDeCeNavigateur($request); + + if ($demande === null) { + return redirect()->route('rejoindre'); + } + + if ($demande->statut === DemandeInscription::ACCEPTEE) { + return redirect()->route('rejoindre.bienvenue'); + } + + if ($demande->statut === DemandeInscription::REFUSEE) { + $demande->delete(); + $request->session()->forget(self::CLE_SESSION); + + return view('rejoindre.refusee'); + } + + return view('rejoindre.attente', ['demande' => $demande->load('classe')]); + } + + public function etat(Request $request): JsonResponse + { + return response()->json(['statut' => $this->demandeDeCeNavigateur($request)?->statut]); + } + + /** + * Le code secret n'est montré qu'une fois : la demande (et le code en + * clair qu'elle porte) est supprimée dès l'affichage, l'élève est connecté. + */ + public function bienvenue(Request $request): View|RedirectResponse + { + $demande = $this->demandeDeCeNavigateur($request); + + if ($demande === null || $demande->statut !== DemandeInscription::ACCEPTEE || $demande->eleve === null) { + return redirect()->route(Auth::guard('eleve')->check() ? 'eleve' : 'rejoindre'); + } + + $eleve = $demande->eleve; + $codeSecret = $demande->code_secret_chiffre; + $demande->delete(); + $request->session()->forget(self::CLE_SESSION); + + Auth::guard('eleve')->login($eleve); + $request->session()->regenerate(); + $eleve->forceFill(['jeton_qr' => null, 'derniere_connexion_a' => now()])->save(); + $request->session()->put('jeton_session_eleve', $eleve->demarreNouvelleSession()); + + return view('rejoindre.bienvenue', ['eleve' => $eleve, 'codeSecret' => $codeSecret]); + } + + private function demandeDeCeNavigateur(Request $request): ?DemandeInscription + { + $jeton = $request->session()->get(self::CLE_SESSION); + + return is_string($jeton) + ? DemandeInscription::where('jeton_empreinte', DemandeInscription::empreinte($jeton))->first() + : null; + } +} diff --git a/portail/app/Http/Middleware/PolitiqueSecuriteContenu.php b/portail/app/Http/Middleware/PolitiqueSecuriteContenu.php index 62072af..61178d4 100644 --- a/portail/app/Http/Middleware/PolitiqueSecuriteContenu.php +++ b/portail/app/Http/Middleware/PolitiqueSecuriteContenu.php @@ -82,7 +82,7 @@ class PolitiqueSecuriteContenu return false; } - if (in_array($route->getName(), ['login', 'register'], true)) { + if (in_array($route->getName(), ['login', 'register'], true) || str_starts_with((string) $route->getName(), 'rejoindre')) { return true; } diff --git a/portail/app/Models/Classe.php b/portail/app/Models/Classe.php index a6254a0..f249dce 100644 --- a/portail/app/Models/Classe.php +++ b/portail/app/Models/Classe.php @@ -50,6 +50,11 @@ class Classe extends Model return $this->hasMany(Eleve::class); } + public function demandesInscription(): HasMany + { + return $this->hasMany(DemandeInscription::class); + } + public function seances(): HasMany { return $this->hasMany(Seance::class); diff --git a/portail/app/Models/DemandeInscription.php b/portail/app/Models/DemandeInscription.php new file mode 100644 index 0000000..b38b20b --- /dev/null +++ b/portail/app/Models/DemandeInscription.php @@ -0,0 +1,52 @@ + 'encrypted', + ]; + + public function classe(): BelongsTo + { + return $this->belongsTo(Classe::class); + } + + public function eleve(): BelongsTo + { + return $this->belongsTo(Eleve::class); + } + + public static function empreinte(string $jeton): string + { + return hash('sha256', $jeton); + } + + public function prunable(): Builder + { + return static::where('created_at', '<', now()->subHour()); + } +} diff --git a/portail/app/Services/GenerateurCompteEleve.php b/portail/app/Services/GenerateurCompteEleve.php index 41e59f9..22372dd 100644 --- a/portail/app/Services/GenerateurCompteEleve.php +++ b/portail/app/Services/GenerateurCompteEleve.php @@ -2,7 +2,9 @@ namespace App\Services; +use App\Models\Classe; use App\Models\Eleve; +use Illuminate\Support\Facades\Hash; use Illuminate\Support\Str; /** @@ -40,4 +42,27 @@ class GenerateurCompteEleve return $code; } + + /** + * Crée le compte d'un élève et renvoie sa fiche (le code secret en clair + * n'existe que dans cette fiche : il est stocké haché). + * + * @return array{eleve: Eleve, identifiant: string, prenom: string, nom: ?string, code_secret: string, jeton_qr: string} + */ + public function creerDans(Classe $classe, string $prenom, ?string $nom): array + { + $identifiant = $this->genererIdentifiant($prenom, $nom); + $codeSecret = $this->genererCodeSecret(); + $jetonQr = Str::random(32); + + $eleve = $classe->eleves()->create([ + 'prenom' => $prenom, + 'nom' => $nom, + 'identifiant' => $identifiant, + 'code_secret' => Hash::make($codeSecret), + 'jeton_qr' => $jetonQr, + ]); + + return ['eleve' => $eleve, 'identifiant' => $identifiant, 'prenom' => $prenom, 'nom' => $nom, 'code_secret' => $codeSecret, 'jeton_qr' => $jetonQr]; + } } diff --git a/portail/database/migrations/2026_10_09_200000_create_demandes_inscription_table.php b/portail/database/migrations/2026_10_09_200000_create_demandes_inscription_table.php new file mode 100644 index 0000000..234d1f0 --- /dev/null +++ b/portail/database/migrations/2026_10_09_200000_create_demandes_inscription_table.php @@ -0,0 +1,32 @@ +id(); + $table->foreignId('classe_id')->constrained()->cascadeOnDelete(); + $table->string('prenom', 100); + $table->string('nom', 100)->nullable(); + // Empreinte du jeton gardé dans la session du navigateur de l'élève : + // seul ce navigateur peut suivre la demande et recevoir son code. + $table->char('jeton_empreinte', 64)->unique(); + $table->string('statut', 20)->default('en_attente'); + $table->foreignId('eleve_id')->nullable()->constrained('eleves')->cascadeOnDelete(); + // Code secret en clair (chiffré par APP_KEY), le temps que l'élève le + // voie une fois ; la demande est supprimée juste après. + $table->text('code_secret_chiffre')->nullable(); + $table->timestamps(); + }); + } + + public function down(): void + { + Schema::dropIfExists('demandes_inscription'); + } +}; diff --git a/portail/resources/views/auth/connexion.blade.php b/portail/resources/views/auth/connexion.blade.php index 285dddc..eb511e4 100644 --- a/portail/resources/views/auth/connexion.blade.php +++ b/portail/resources/views/auth/connexion.blade.php @@ -15,6 +15,7 @@

Ils sont sur ta fiche de connexion. Tu peux aussi scanner le QR.

+

Pas encore d'identifiant ? Rejoins ta classe avec son code.

diff --git a/portail/resources/views/confidentialite.blade.php b/portail/resources/views/confidentialite.blade.php index b7091c5..bef8264 100644 --- a/portail/resources/views/confidentialite.blade.php +++ b/portail/resources/views/confidentialite.blade.php @@ -47,6 +47,7 @@
  • aucune adresse e-mail ;
  • sessions de lab : date, état, machines virtuelles attribuées ;
  • résultats de la vérification automatique (réussi ou non, note, message par vérification).
    • +
  • s'il rejoint sa classe avec le code de classe : le prénom et le nom (facultatif) qu'il saisit, le temps que l'enseignant accepte ou refuse sa demande.

    • Ce que l’élève tape dans la console de sa machine virtuelle n’est pas enregistré par Syllabus. Son enseignant peut suivre la console en direct, en lecture seule, pendant la séance.

    Enseignants

    @@ -70,6 +71,7 @@
    • Machines virtuelles et leur contenu : supprimées à la fin de la session de lab (45 minutes par défaut).
    • Classe et données de ses élèves : jusqu’à l’archivage de la classe par l’enseignant, puis supprimées automatiquement 90 jours après l’archivage.
      • +
    • Demande pour rejoindre une classe : supprimée dès que l’élève a reçu son identifiant, et au plus tard une heure après son envoi.
    • Compte enseignant : tant qu’il est utilisé ; supprimé sur demande.
    • Journaux techniques : 14 jours.
    • Sauvegardes : chiffrées, conservées au plus 5 semaines (7 sauvegardes quotidiennes et 4 hebdomadaires). Une donnée supprimée disparaît donc des sauvegardes au plus tard 5 semaines après sa suppression.
      • diff --git a/portail/resources/views/mes-classes/show.blade.php b/portail/resources/views/mes-classes/show.blade.php index bd5a994..2b67cdf 100644 --- a/portail/resources/views/mes-classes/show.blade.php +++ b/portail/resources/views/mes-classes/show.blade.php @@ -18,6 +18,33 @@

      Classe créée. Ajoute maintenant tes élèves ci-dessous.

      @endif + @unless ($classe->estArchivee()) +
      +
      +

      Faire rejoindre tes élèves

      +

      Projette ce code ou ce QR. Chaque élève va sur {{ preg_replace('#^https?://#', '', route('rejoindre')) }}, tape le code et son prénom ; tu acceptes sa demande ici, et il reçoit à l'écran son identifiant et son code secret. Aucun e-mail.

      +

      {{ $classe->code_classe }}

      +

      Demandes en attente ({{ $demandes->count() }})

      + @if ($demandes->isEmpty()) +

      Aucune pour l'instant. Cette liste se met à jour toute seule.

      + @else +
        + @foreach ($demandes as $demande) +
      • + {{ $demande->prenom }}@if ($demande->nom) {{ $demande->nom }}@endif · {{ $demande->created_at->diffForHumans() }} + + @csrf +
        @csrf
        +         +
        • + @endforeach +
      + @endif +
      +
      {!! \App\Support\QrCode::svg(route('rejoindre', ['code' => $classe->code_classe])) !!}
      +
      + @endunless + @if (session('fiches'))
      @@ -69,16 +96,26 @@
      -

      Élèves ({{ $classe->eleves->count() }})

      +
      +

      Élèves ({{ $classe->eleves->count() }})

      + @if ($classe->eleves->isNotEmpty()) +
      + @csrf + +
      + @endif +
      + @error('eleves')

      Coche au moins un élève.

      @enderror @if ($classe->eleves->isEmpty())

      Aucun élève pour l'instant.

      @else
      - + @foreach ($classe->eleves as $eleve) + @@ -165,6 +202,22 @@ @endif +@endsection diff --git a/portail/resources/views/rejoindre/bienvenue.blade.php b/portail/resources/views/rejoindre/bienvenue.blade.php new file mode 100644 index 0000000..6d18152 --- /dev/null +++ b/portail/resources/views/rejoindre/bienvenue.blade.php @@ -0,0 +1,16 @@ +@extends('layouts.vitrine') + +@section('titre', 'Bienvenue') + +@section('contenu') +
      +

      Bienvenue, {{ $eleve->prenom }} !

      +

      Note ou photographie ces deux informations maintenant : elles ne s'afficheront plus jamais. Tu en auras besoin pour te reconnecter.

      +
      +

      Identifiant
      {{ $eleve->identifiant }}

      +

      Code secret
      {{ $codeSecret }}

      +
      +

      Si tu le perds, ton prof peut te donner un nouveau code.

      + C'est noté, aller à mon espace +
      +@endsection diff --git a/portail/resources/views/rejoindre/formulaire.blade.php b/portail/resources/views/rejoindre/formulaire.blade.php new file mode 100644 index 0000000..99adc30 --- /dev/null +++ b/portail/resources/views/rejoindre/formulaire.blade.php @@ -0,0 +1,28 @@ +@extends('layouts.vitrine') + +@section('titre', 'Rejoindre ma classe') + +@section('contenu') +
      +

      Rejoindre ma classe

      +

      Ton prof t'a donné un code de classe à 6 caractères. Ton prof va accepter ta demande, puis tu recevras ton identifiant et ton code secret sur cet écran.

      +
      + @csrf + + @error('code')

      {{ $message }}

      @enderror + + @error('prenom')

      {{ $message }}

      @enderror + +

      Aucune adresse e-mail n'est demandée.

      + + +

      Tu as déjà un identifiant ? Connecte-toi.

      +
      +@endsection diff --git a/portail/resources/views/rejoindre/refusee.blade.php b/portail/resources/views/rejoindre/refusee.blade.php new file mode 100644 index 0000000..73850b8 --- /dev/null +++ b/portail/resources/views/rejoindre/refusee.blade.php @@ -0,0 +1,10 @@ +@extends('layouts.vitrine') + +@section('titre', 'Demande refusée') + +@section('contenu') +
      +

      Ton prof n'a pas accepté la demande

      +

      Vérifie le code de la classe et ton prénom avec lui, puis réessaie.

      +
      +@endsection diff --git a/portail/routes/console.php b/portail/routes/console.php index cb37f9a..9461212 100644 --- a/portail/routes/console.php +++ b/portail/routes/console.php @@ -1,5 +1,6 @@ everyFiveMinutes(); // Étape 14 (RGPD) : classes archivées depuis 90 jours supprimées avec les // données de leurs élèves. Schedule::command('classes:purger-archivees')->dailyAt('02:30'); + +// Demandes pour rejoindre une classe non traitées depuis une heure : supprimées. +Schedule::command('model:prune', ['--model' => [DemandeInscription::class]])->everyFifteenMinutes(); diff --git a/portail/routes/web.php b/portail/routes/web.php index 525da8d..ef88f4d 100644 --- a/portail/routes/web.php +++ b/portail/routes/web.php @@ -7,6 +7,7 @@ use App\Http\Controllers\CorrectionController; use App\Http\Controllers\EleveTableauDeBordController; use App\Http\Controllers\InscriptionController; use App\Http\Controllers\RapportCspController; +use App\Http\Controllers\RejoindreClasseController; use App\Http\Controllers\SeanceController; use Illuminate\Foundation\Http\Middleware\ValidateCsrfToken; use Illuminate\Support\Facades\Route; @@ -71,6 +72,13 @@ Route::post('/connexion/prof', [ConnexionController::class, 'connexionProf'])->n Route::post('/connexion/eleve', [ConnexionController::class, 'connexionEleve'])->name('connexion.eleve'); Route::get('/connexion/qr/{jeton}', [ConnexionController::class, 'connexionParQr'])->name('connexion.qr'); Route::post('/inscription', [InscriptionController::class, 'store'])->name('inscription.store'); + +// Un élève rejoint une classe avec son code ; le prof accepte (aucun e-mail). +Route::get('/rejoindre', [RejoindreClasseController::class, 'formulaire'])->name('rejoindre'); +Route::post('/rejoindre', [RejoindreClasseController::class, 'demander'])->middleware('throttle:10,1')->name('rejoindre.demander'); +Route::get('/rejoindre/attente', [RejoindreClasseController::class, 'attente'])->name('rejoindre.attente'); +Route::get('/rejoindre/etat', [RejoindreClasseController::class, 'etat'])->name('rejoindre.etat'); +Route::get('/rejoindre/bienvenue', [RejoindreClasseController::class, 'bienvenue'])->name('rejoindre.bienvenue'); Route::post('/deconnexion', [ConnexionController::class, 'deconnexion'])->name('deconnexion'); // --- Espace élève (protégé, session unique) --- @@ -92,6 +100,10 @@ Route::middleware(['auth:web', 'session.unique:web'])->group(function () { Route::post('/mes-classes/{classe}/eleves', [ClasseController::class, 'storeEleve'])->name('mes-classes.eleves.store'); Route::post('/mes-classes/{classe}/eleves/import', [ClasseController::class, 'importerCsv'])->name('mes-classes.eleves.importer'); Route::post('/mes-classes/{classe}/eleves/{eleve}/regenerer-code', [ClasseController::class, 'regenererCode'])->name('mes-classes.eleves.regenererCode'); + Route::post('/mes-classes/{classe}/demandes/{demande}/accepter', [ClasseController::class, 'accepterDemande'])->name('mes-classes.demandes.accepter'); + Route::post('/mes-classes/{classe}/demandes/{demande}/refuser', [ClasseController::class, 'refuserDemande'])->name('mes-classes.demandes.refuser'); + Route::get('/mes-classes/{classe}/demandes/nombre', [ClasseController::class, 'nombreDemandes'])->name('mes-classes.demandes.nombre'); + Route::post('/mes-classes/{classe}/fiches', [ClasseController::class, 'nouvellesFiches'])->name('mes-classes.fiches'); Route::post('/mes-classes/{classe}/archiver', [ClasseController::class, 'archiver'])->name('mes-classes.archiver'); Route::post('/mes-classes/{classe}/desarchiver', [ClasseController::class, 'desarchiver'])->name('mes-classes.desarchiver'); Route::post('/mes-classes/{classe}/seances', [SeanceController::class, 'store'])->name('mes-classes.seances.store'); diff --git a/portail/tests/Feature/PagesPubliquesTest.php b/portail/tests/Feature/PagesPubliquesTest.php index 08248aa..dc2c516 100644 --- a/portail/tests/Feature/PagesPubliquesTest.php +++ b/portail/tests/Feature/PagesPubliquesTest.php @@ -13,8 +13,10 @@ test('chaque page publique s’affiche sans erreur', function () { && ! str_contains($r->uri(), '{') && ! collect($r->gatherMiddleware())->contains(fn ($m) => is_string($m) && str_starts_with($m, 'auth'))) ->reject(fn ($r) => str_starts_with($r->uri(), 'horizon') || str_starts_with($r->uri(), '_') || $r->uri() === 'up') - // Redirections voulues (ex. /statut vers le site de statut). - ->reject(fn ($r) => $r->getActionName() === '\\Illuminate\\Routing\\RedirectController'); + // Redirections voulues (ex. /statut vers le site de statut ; pages de + // suivi d'une demande pour rejoindre une classe, sans demande en cours). + ->reject(fn ($r) => $r->getActionName() === '\\Illuminate\\Routing\\RedirectController' + || in_array($r->getName(), ['rejoindre.attente', 'rejoindre.bienvenue'], true)); expect($pages)->not->toBeEmpty(); diff --git a/portail/tests/Feature/RejoindreClasseTest.php b/portail/tests/Feature/RejoindreClasseTest.php new file mode 100644 index 0000000..38ead42 --- /dev/null +++ b/portail/tests/Feature/RejoindreClasseTest.php @@ -0,0 +1,98 @@ +withSession(['jeton_session_web' => $prof->demarreNouvelleSession()])->actingAs($prof, 'web'); +} + +test('un élève rejoint la classe avec son code ; le prof accepte ; l’élève voit son code une seule fois et peut se reconnecter', function () { + $classe = Classe::factory()->create(); + + $this->post(route('rejoindre.demander'), ['code' => strtolower($classe->code_classe), 'prenom' => 'Inès', 'nom' => 'Martin']) + ->assertRedirect(route('rejoindre.attente')); + $this->get(route('rejoindre.attente'))->assertOk()->assertSee('Demande envoyée, Inès'); + $this->getJson(route('rejoindre.etat'))->assertJson(['statut' => 'en_attente']); + + $demande = DemandeInscription::sole(); + commeProf($this, $classe->prof)->get(route('mes-classes.show', $classe))->assertSee('Inès Martin')->assertSee($classe->code_classe); + commeProf($this, $classe->prof)->post(route('mes-classes.demandes.accepter', [$classe, $demande]))->assertRedirect(); + Auth::guard('web')->logout(); + + $this->getJson(route('rejoindre.etat'))->assertJson(['statut' => 'acceptee']); + $page = $this->get(route('rejoindre.bienvenue'))->assertOk()->assertSee('ines.m'); + preg_match('#Code secret
      ]*>([A-Z0-9]{6})#', $page->getContent(), $m); + $code = $m[1]; + + expect(DemandeInscription::count())->toBe(0) + ->and(Auth::guard('eleve')->user()?->identifiant)->toBe('ines.m'); + + // Une seule fois : un second affichage renvoie vers l'espace élève. + $this->get(route('rejoindre.bienvenue'))->assertRedirect(route('eleve')); + + Auth::guard('eleve')->logout(); + expect(Auth::guard('eleve')->attempt(['identifiant' => 'ines.m', 'password' => $code]))->toBeTrue(); +}); + +test('un code faux ou une classe archivée sont refusés', function () { + $this->post(route('rejoindre.demander'), ['code' => 'ZZZZZZ', 'prenom' => 'Léo'])->assertSessionHasErrors('code'); + + $archivee = Classe::factory()->create(); + $archivee->forceFill(['archivee_a' => now()])->save(); + $this->post(route('rejoindre.demander'), ['code' => $archivee->code_classe, 'prenom' => 'Léo'])->assertSessionHasErrors('code'); + + expect(DemandeInscription::count())->toBe(0); +}); + +test('une demande refusée l’annonce à l’élève et disparaît', function () { + $classe = Classe::factory()->create(); + $this->post(route('rejoindre.demander'), ['code' => $classe->code_classe, 'prenom' => 'Léo']); + + commeProf($this, $classe->prof)->post(route('mes-classes.demandes.refuser', [$classe, DemandeInscription::sole()])); + Auth::guard('web')->logout(); + + $this->get(route('rejoindre.attente'))->assertOk()->assertSee('pas accepté'); + expect(DemandeInscription::count())->toBe(0)->and(Eleve::count())->toBe(0); +}); + +test('seul le navigateur de l’élève voit son code, et seul le prof de la classe peut accepter', function () { + $classe = Classe::factory()->create(); + $this->post(route('rejoindre.demander'), ['code' => $classe->code_classe, 'prenom' => 'Léo']); + $demande = DemandeInscription::sole(); + + commeProf($this, User::factory()->create())->post(route('mes-classes.demandes.accepter', [$classe, $demande]))->assertForbidden(); + commeProf($this, $classe->prof)->post(route('mes-classes.demandes.accepter', [$classe, $demande])); + Auth::guard('web')->logout(); + + $this->flushSession(); + $this->get(route('rejoindre.bienvenue'))->assertRedirect(route('rejoindre')); + expect(DemandeInscription::sole()->statut)->toBe(DemandeInscription::ACCEPTEE); +}); + +test('une demande non traitée depuis une heure est supprimée', function () { + $classe = Classe::factory()->create(); + $ancienne = DemandeInscription::create(['classe_id' => $classe->id, 'prenom' => 'A', 'jeton_empreinte' => DemandeInscription::empreinte('a'), 'statut' => 'en_attente']); + $ancienne->forceFill(['created_at' => now()->subMinutes(61)])->save(); + DemandeInscription::create(['classe_id' => $classe->id, 'prenom' => 'B', 'jeton_empreinte' => DemandeInscription::empreinte('b'), 'statut' => 'en_attente']); + + $this->artisan('model:prune', ['--model' => [DemandeInscription::class]]); + + expect(DemandeInscription::pluck('prenom')->all())->toBe(['B']); +}); + +test('le prof génère de nouvelles fiches pour les élèves cochés, et seulement eux', function () { + $classe = Classe::factory()->create(); + [$a, $b] = Eleve::factory()->count(2)->create(['classe_id' => $classe->id]); + $ancienB = $b->code_secret; + + commeProf($this, $classe->prof)->post(route('mes-classes.fiches', $classe), ['eleves' => [$a->id]]) + ->assertSessionHas('fiches', fn ($fiches) => count($fiches) === 1 && $fiches[0]['identifiant'] === $a->identifiant); + + expect(Auth::guard('eleve')->attempt(['identifiant' => $a->identifiant, 'password' => '123456']))->toBeFalse() + ->and($b->refresh()->code_secret)->toBe($ancienB); +});
      ÉlèveIdentifiantDernière connexion
      ÉlèveIdentifiantDernière connexion
      {{ $eleve->nomAffiche() }} {{ $eleve->identifiant }} {{ $eleve->derniere_connexion_a?->diffForHumans() ?? 'jamais' }}