# Corrections effectuées - Audit Backend API
**Date:** 26 novembre 2025  
**Contexte:** Corrections suite à audit ciblé routes/contrôleurs/services + migrations

## ✅ Problèmes corrigés

### 1. Colonne `allow_comments` manquante dans la table `fails`
**Problème:** 
- FailsController.createFail insère `allow_comments`
- CommentsController le lit
- Conséquence: Erreur SQL `Unknown column 'allow_comments' in 'field list'`

**Solution:**
- ✅ Ajouté `allow_comments TINYINT(1) NOT NULL DEFAULT 1` dans `backend-api/migrations/faildaily.sql`
- ✅ Créé migration `013_add_allow_comments_to_fails.sql` (déjà appliquée manuellement sur serveur)
- ✅ Colonne existe maintenant en production

---

### 2. Endpoint `getAnonymeFails` forçait `1 as allowComments`
**Problème:**
- Ligne 272 dans failsController.js: `SELECT ... 1 as allowComments`
- Les fails listés ignoraient l'état réel
- Frontend voyait toujours commentaires actifs (impossible d'afficher "commentaires désactivés")

**Solution:**
- ✅ Remplacé `1 as allowComments` par `f.allow_comments` (ligne 272)
- ✅ Lecture de la vraie colonne depuis la base de données

---

### 3. Validation incohérente dans `validators.js`
**Problème:**
- Commentaire "allow_comments a été supprimé" alors que le champ est utilisé partout
- Incohérence entre documentation et code réel

**Solution:**
- ✅ Réactivé validation: `body('allow_comments').optional().isBoolean()`
- ✅ Supprimé commentaire trompeur

---

### 4. Catégories de données de démo invalides
**Problème:**
- Données d'exemple utilisaient "personal/professional"
- Validation n'accepte que: apprentissage, bricolage, cuisine, travail, etc.
- Incohérence si on relit ces données de démo

**Solution:**
- ✅ "personal" → "cuisine" (gâteau carbonisé)
- ✅ "professional" → "travail" (présentation ratée)
- ✅ Ajouté colonne `allow_comments` aux INSERT de démo

---

### 5. Duplication `connectionLimit` dans `database.js`
**Problème:**
- `connectionLimit: 10` défini deux fois (lignes 21 et 29)
- Redondance inutile dans la configuration du pool

**Solution:**
- ✅ Supprimé la duplication (gardé uniquement ligne 21)
- ✅ Configuration du pool désormais propre

---

## 📝 Points notés (non bloquants)

### 6. TODO non traité - Tags
**Localisation:** FailsController  
**Note:** "TODO: Ajouter le champ tags à la table ou table séparée"  
**Action:** Fonctionnalité entamée mais non finalisée - à traiter dans une future sprint

### 7. Absence de Foreign Keys
**Localisation:** database.js et schema SQL  
**Note:** Aucune FK définie explicitement  
**Impact:** Pas bloquant mais attention à l'intégrité référentielle  
**Recommandation:** Ajouter des FK pour `user_id`, `fail_id`, etc. dans une future migration

### 8. Tables auxiliaires de commentaires
**Localisation:** CommentsController  
**Note:** `ensureAuxTables` appelé au premier commentaire  
**Recommandation:** Envisager d'appeler au boot du serveur pour éviter latence première insertion

---

## 🚀 Déploiement

### Fichiers modifiés:
```
✅ backend-api/migrations/faildaily.sql (schéma principal)
✅ backend-api/migrations/013_add_allow_comments_to_fails.sql (nouvelle migration)
✅ backend-api/src/controllers/failsController.js (ligne 272)
✅ backend-api/src/middleware/validators.js (validation allow_comments)
✅ backend-api/src/config/database.js (suppression duplication)
```

### Actions serveur:
```bash
# Migration appliquée
✅ docker exec -i faildaily_db_ovh mysql faildaily < /tmp/013_add_allow_comments_to_fails.sql

# Backend redémarré
✅ docker restart faildaily_backend_ovh

# Vérification
✅ docker logs faildaily_backend_ovh → "🚀 FailDaily API Server démarré !"
```

---

## 🧪 Tests à effectuer

1. **Créer un fail avec commentaires désactivés:**
   ```json
   POST /api/fails
   {
     "title": "Test fail",
     "description": "Test commentaires désactivés",
     "category": "technologie",
     "allow_comments": false
   }
   ```
   ✅ Devrait réussir (plus d'erreur SQL)

2. **Lister les fails:**
   ```
   GET /api/fails/anonymes
   ```
   ✅ Devrait afficher `allowComments: false` pour les fails concernés

3. **Tenter de commenter un fail avec commentaires désactivés:**
   ```json
   POST /api/fails/{failId}/comments
   {
     "content": "Test commentaire"
   }
   ```
   ✅ Devrait retourner erreur "Les commentaires sont désactivés"

---

## 📊 Impact

- **Avant:** Erreur 500 à chaque création de fail
- **Après:** Création de fail fonctionne normalement
- **Avant:** Impossible de désactiver les commentaires (valeur ignorée)
- **Après:** Fonctionnalité allow_comments pleinement opérationnelle
- **Avant:** Incohérence entre code et validation
- **Après:** Alignement complet backend + validation + base de données

---

## 🔄 Prochaines étapes recommandées

1. Tester création de fail avec et sans commentaires
2. Valider affichage frontend du statut "commentaires désactivés"
3. Considérer ajout des Foreign Keys dans une migration future
4. Finaliser ou supprimer la fonctionnalité "tags" (TODO)
5. Déplacer `ensureAuxTables` au boot si besoin de performance