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

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

New Issue
Closed
opened 2026-05-13 22:09:33 +00:00 by cedricAbonnel · 1 comment
cedricAbonnel commented 2026-05-13 22:09:33 +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

Migré depuis varlog#30

## 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 --- *Migré depuis [varlog#30](https://git.abonnel.fr/cedricAbonnel/varlog/issues/30)*
cedricAbonnel commented 2026-05-15 21:35:12 +00:00
Author
Owner
Copy Link
Copy Source

Traité : src/Service/AuthService.php, src/Repository/UserRepository.php et src/Domain/User.php supprimés. Aucune référence active ne pointait vers ces fichiers. Les tables login_attempts et users restent en base (non supprimées — cf. UserRepository.createFromOidc utilisé par la connexion OIDC).

Traité : `src/Service/AuthService.php`, `src/Repository/UserRepository.php` et `src/Domain/User.php` supprimés. Aucune référence active ne pointait vers ces fichiers. Les tables `login_attempts` et `users` restent en base (non supprimées — cf. UserRepository.createFromOidc utilisé par la connexion OIDC).
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/folio#19
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?