cedricAbonnel/varlog
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

[Nettoyage] Supprimer AuthService — dead code incompatible avec le système de session actuel #30

New Issue
Closed
opened 2026-05-08 21:32:53 +00:00 by cedricAbonnel · 1 comment
cedricAbonnel commented 2026-05-08 21:32:53 +00:00
Owner
Copy Link
Copy Source

Problème

src/Service/AuthService.php implémente un système d'authentification par mot de passe qui n'est pas utilisé dans le projet. De plus, il est architecturalement incompatible avec le reste :

  • AuthService::login() stocke $_SESSION['uid']
  • auth.php (système actif) lit $_SESSION['user_email']

Ces deux clés de session coexistent sans jamais se parler. AuthService est du dead code qui :

  • crée une fausse impression de fonctionnalité disponible
  • suppose des tables (users, login_attempts) peut-être absentes
  • ajoute une surface de maintenance inutile

Correction

  • Supprimer src/Service/AuthService.php
  • Vérifier et supprimer src/Repository/UserRepository.php si uniquement utilisé par AuthService
  • Vérifier si les tables users et login_attempts sont utilisées ailleurs ; si non, les documenter comme orphelines

Critères d'acceptation

  • AuthService.php supprimé
  • Aucune référence restante dans le code actif
  • PHPStan passe sans erreur après suppression
## Problème `src/Service/AuthService.php` implémente un système d'authentification par mot de passe qui n'est **pas utilisé** dans le projet. De plus, il est architecturalement incompatible avec le reste : - `AuthService::login()` stocke `$_SESSION['uid']` - `auth.php` (système actif) lit `$_SESSION['user_email']` Ces deux clés de session coexistent sans jamais se parler. `AuthService` est du dead code qui : - crée une fausse impression de fonctionnalité disponible - suppose des tables (`users`, `login_attempts`) peut-être absentes - ajoute une surface de maintenance inutile ## Correction - Supprimer `src/Service/AuthService.php` - Vérifier et supprimer `src/Repository/UserRepository.php` si uniquement utilisé par `AuthService` - Vérifier si les tables `users` et `login_attempts` sont utilisées ailleurs ; si non, les documenter comme orphelines ## Critères d'acceptation - [ ] `AuthService.php` supprimé - [ ] Aucune référence restante dans le code actif - [ ] PHPStan passe sans erreur après suppression
cedricAbonnel commented 2026-05-13 22:09:34 +00:00
Author
Owner
Copy Link
Copy Source

Ticket migré vers le dépôt Folio : cedricAbonnel/folio#19

Ticket migré vers le dépôt Folio : https://git.abonnel.fr/cedricAbonnel/folio/issues/19
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
No items
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
cedricAbonnel (Cédric Abonnel)
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: cedricAbonnel/varlog#30
Delete Branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?