Création Audit

2026-05-26 16:29:10 +02:00
parent 1c89546b61
commit eb7959eb09
1 changed files with 82 additions and 0 deletions
@@ -0,0 +1,82 @@
+# Audit PTT Live — 2026-05-26
+
+## Structure & Documentation
+
+La structure réelle **diverge de CLAUDE.md** — l'implémentation a avancé au-delà du plan initial sans mise à jour de la doc.
+
+Fichiers documentés mais absents : `server/api/routes.js`, `client/src/utils/audio.js`
+Fichiers présents mais non documentés : `AudioBridgeManager.js`, `LiveKitServerBridge.js`, `AudioLevelsServer.js`, + plusieurs composants React.
+
+---
+
+## Problèmes critiques
+
+**1. Chaîne audio serveur incomplète**
+Le bridge audio ne transmet pas l'audio capturé vers LiveKit. Les TODOs sont explicites :
+- `server/bridge/AudioBridge.js:368` : `// TODO: Envoyer opusData à LiveKit pour ce groupe spécifique`
+- `server/bridge/AudioBridge.js:439` : `// TODO: Implémenter réception bas niveau Opus depuis LiveKit`
+
+Le flux `carte son → LiveKit → clients` n'est pas fonctionnel côté serveur.
+
+**2. `LiveKitServerBridge.js` jamais utilisé**
+Fichier créé, jamais importé ni appelé. Contient lui-même un `// TODO: Implémenter l'envoi réel vers LiveKit`. Code mort qui confond les responsabilités avec `LiveKitClient.js`.
+
+**3. Pas d'authentification sur `/admin`**
+N'importe qui sur le réseau peut modifier la configuration (groupes, routing, devices). Critique en production.
+Note du dev : c'est normal et non critique pour le moment. 
+
+---
+
+## Problèmes de sécurité
+
+| Sévérité | Problème |
+|----------|----------|
+| Haute | CORS `*` dans `server/index.js` — accès depuis n'importe quel domaine |
+| Haute | API `/admin` sans authentification |
+| Moyenne | Clés LiveKit hardcodées en fallback `'devkey'/'secret'` |
+
+Note du dev : l'app a pour vocation à être utilisée sur un réseau local.
+---
+
+## Qualité du code
+
+**Points positifs**
+- Architecture modulaire solide
+- EventEmitter bien utilisé pour la réactivité et le hot-reload
+- Gestion d'erreurs gracieuse (fallback sans crash si pas de carte son)
+- OpusCodec robuste avec presets configurables
+- JitterBuffer avec stats adaptatives
+
+**Points faibles**
+- Logging DEBUG non retiré dans `server/bridge/LiveKitClient.js:93`
+- Device IDs hardcodés dans `config.yaml` (`inputDeviceId: 4`, `outputDeviceId: 0`) — non portable
+- Création de `Float32Array` à chaque frame audio → pression GC potentielle à 30+ clients
+
+---
+
+## État des phases
+
+| Phase | Avancement | Bloquant |
+|-------|-----------|---------|
+| Phase 1 MVP | ~80% | Bridge audio serveur incomplet |
+| Phase 2 Fonctionnalités | ~95% | Authentification manquante |
+| Phase 3 Intégrations | ~85% | Tests réels manquants |
+
+---
+
+## Recommandations par priorité
+
+### Priorité 1 — Bloquant
+1. Implémenter la connexion `AudioBridge → LiveKitClient` (TODOs lignes 368/439)
+2. Ajouter authentification sur `/admin` (token Bearer ou session)
+3. Supprimer ou intégrer `LiveKitServerBridge.js`
+
+### Priorité 2 — Important
+4. CORS : remplacer `*` par origin explicite du client
+5. Retirer les `console.log` DEBUG de `LiveKitClient.js`
+6. Device IDs : auto-détection plutôt que valeurs hardcodées
+
+### Priorité 3 — Amélioration
+7. Pool de buffers audio pré-alloués pour tenir 30+ clients
+8. Mettre à jour `CLAUDE.md` avec la structure réelle du code
+9. Tests d'intégration E2E (latence mesurée, scénario multi-clients)