Skip to content

fix: compte administrateur, seed haché et API sans hash (#189) - #190

Merged
mortifia merged 1 commit into
mainfrom
189-compte-admin
Nov 1, 2026
Merged

mortifia merged 1 commit into
mainfrom
189-compte-admin

Conversation

@mortifia

@mortifia mortifia commented Nov 1, 2026

Copy link
Copy Markdown
Contributor

Corrige #189.

Changements

  • prisma/seed.js hache ADMIN_PASSWORD. Le compte créé peut se connecter, ce qui retire la raison d'être du contournement du README.
  • README : étape « retirer le await » et lien vers bcrypt.fr supprimés, installations locale et de production réécrites.
  • .env.example remplace .default, supprimé par 87747ef.
  • migrate reçoit .env, nécessaire pour lancer le seed dans le conteneur.
  • server/utils/publicUserFields.ts : liste unique des champs renvoyés par GET, POST et PUT sur les utilisateurs. Le hash et le jeton ne sortent plus de ces routes.

Vérification

Installation neuve, commandes du README exécutées telles quelles, base vide, mot de passe aléatoire :

  • Production : migrate, seed et up -d --build app réussis. La base contient 1 admin avec un hash bcrypt et 19 tâches.
  • API, 13 contrôles conformes : mot de passe faux ou absent refusé, compte du seed accepté avec le rôle ADMIN, création sans jeton refusée, création et modification sans hash ni jeton dans la réponse, liste sans hash ni jeton, ancien mot de passe refusé et nouveau accepté, suppression.
  • Local : npm install, prisma generate, migrate deploy et db seed réussis, même contenu de base.

Effet en production

Aucun sur la base : le seed ne tourne qu'à l'installation. Le changement de l'API prendra effet au prochain déploiement.

@mortifia
mortifia merged commit 8398a2c into main Nov 1, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant