#80 - FEAT: Paramètres d'intégration exposés en UI (S3, Ollama, Email, ntfy) — fin des env vars - nonobis/Heimdall

nonobis commented

2026-08-12 09:59:03 +02:00

Owner

Contexte

L'exigence : 100 % des paramètres configurables visuellement par l'IHM, côté central comme côté agents. Aujourd'hui plusieurs intégrations ne sont réglables que par env vars (ALARIK_ENDPOINT, S3_ENDPOINT, OLLAMA_URL, EMAIL__*, NTFY__*, SENTRY_DSN) — aucun écran ne les expose.

État actuel (vérifié, audit clés)

Env lues dans le code, non exposées en UI :
ALARIK_ENDPOINT, S3_ENDPOINT, OLLAMA_URL, EMAIL__FROM_ADDRESS, EMAIL__FROM_NAME, EMAIL__SMTP_HOST, EMAIL__SMTP_PORT, EMAIL__USERNAME, EMAIL__PASSWORD, EMAIL__USE_SSL, NTFY__BASE_URL, NTFY__TOKEN, SENTRY_DSN, MODEL_PATH, CAMERAS_DB

À faire

Basculer ces réglages vers les settings en base (catégories : storage.*, ai.*, email.*, ntfy.*, sentry.*, model.*) avec seed au démarrage depuis les env vars actuelles (migration douce : env = valeur initiale, DB = source de vérité ensuite)
Services concernés (AlarikHealthCheck, AiHealthCheck, EmailService, NtfyService, FrameAnalyzerService) lisent la config effective (globale + overrides agent) au lieu de Environment.GetEnvironmentVariable
UI : chaque domaine a sa section de configuration visuelle (ticket refonte settings) — saisie des secrets masquée (password field + « révéler »), bouton « Tester » (connexion S3, ping broker IA, envoi email test, push ntfy test)
Propagation : modifier une valeur globale → broadcast ConfigReload aux agents (déjà en place pour AppSettings)

Critères d'acceptation

Chaque paramètre listé ci-dessus est modifiable depuis l'UI (Admin → Configuration)
Une modification est appliquée aux agents sans redémarrage manuel (≤ intervalle de reload)
Les secrets ne sont jamais retournés en clair par l'API (redaction existante à étendre à ntfy.token, s3.secretKey, email.password)
Tests : migration env→DB, handler update + redaction

Fichiers concernés

src/Heimdall.Infrastructure/Services/ (EmailService, NtfyService, HealthChecks)
src/Heimdall.Agent/Services/ (FrameAnalyzerService, ConfigReceiverService)
src/Heimdall.Infrastructure/Data/ (seed settings)
src/Heimdall.Web/Components/Settings/ (sections)

## Contexte L'exigence : **100 % des paramètres configurables visuellement par l'IHM**, côté central comme côté agents. Aujourd'hui plusieurs intégrations ne sont réglables que par env vars (`ALARIK_ENDPOINT`, `S3_ENDPOINT`, `OLLAMA_URL`, `EMAIL__*`, `NTFY__*`, `SENTRY_DSN`) — aucun écran ne les expose. ## État actuel (vérifié, audit clés) Env lues dans le code, non exposées en UI : `ALARIK_ENDPOINT`, `S3_ENDPOINT`, `OLLAMA_URL`, `EMAIL__FROM_ADDRESS`, `EMAIL__FROM_NAME`, `EMAIL__SMTP_HOST`, `EMAIL__SMTP_PORT`, `EMAIL__USERNAME`, `EMAIL__PASSWORD`, `EMAIL__USE_SSL`, `NTFY__BASE_URL`, `NTFY__TOKEN`, `SENTRY_DSN`, `MODEL_PATH`, `CAMERAS_DB` ## À faire 1. Basculer ces réglages vers les settings en base (catégories : `storage.*`, `ai.*`, `email.*`, `ntfy.*`, `sentry.*`, `model.*`) avec **seed au démarrage** depuis les env vars actuelles (migration douce : env = valeur initiale, DB = source de vérité ensuite) 2. Services concernés (AlarikHealthCheck, AiHealthCheck, EmailService, NtfyService, FrameAnalyzerService) lisent la config effective (globale + overrides agent) au lieu de `Environment.GetEnvironmentVariable` 3. UI : chaque domaine a sa section de configuration visuelle (ticket refonte settings) — saisie des secrets masquée (password field + « révéler »), bouton « Tester » (connexion S3, ping broker IA, envoi email test, push ntfy test) 4. Propagation : modifier une valeur globale → broadcast `ConfigReload` aux agents (déjà en place pour AppSettings) ## Critères d'acceptation - Chaque paramètre listé ci-dessus est modifiable depuis l'UI (Admin → Configuration) - Une modification est appliquée aux agents sans redémarrage manuel (≤ intervalle de reload) - Les secrets ne sont jamais retournés en clair par l'API (redaction existante à étendre à `ntfy.token`, `s3.secretKey`, `email.password`) - Tests : migration env→DB, handler update + redaction ## Fichiers concernés - `src/Heimdall.Infrastructure/Services/` (EmailService, NtfyService, HealthChecks) - `src/Heimdall.Agent/Services/` (FrameAnalyzerService, ConfigReceiverService) - `src/Heimdall.Infrastructure/Data/` (seed settings) - `src/Heimdall.Web/Components/Settings/` (sections)

nonobis added the

labels

2026-08-12 09:59:03 +02:00

nonobis changed title from ~~FEAT: Paramètres d'intégration exposés en UI (S3, CodeProject.AI, Ollama, Email, ntfy) — fin des env vars~~ to FEAT: Paramètres d'intégration exposés en UI (S3, Ollama, Email, ntfy) — fin des env vars

2026-08-12 10:05:53 +02:00

nonobis referenced this issue from a commit

2026-08-12 10:39:57 +02:00

feat(settings): seed clés d'intégration ntfy/email/IA en base (partiel #80)

nonobis referenced this issue from a commit

2026-08-12 10:46:18 +02:00

feat(settings): sections visuelles IA/Notifications/Défauts agents — suppression du CRUD clé/valeur (Fixes #79)

nonobis commented

2026-08-12 10:46:55 +02:00

Author

Owner

Progression (12/08/2026)

Fait :

✅ Seed idempotent complet en base (source de vérité DB, migration douce env → DB) : ntfy.*, email.*, ai.* (cpaiUrl, ollamaUrl, detection.enabled), log.level, log.retentionDays, heartbeat.interval.seconds, sentry.dsn — commits 26c0e2f + c48f1de
✅ UI : sections visuelles Notifications (ntfy + SMTP), IA (CodeProject.AI/Ollama/détection), Défauts agents (log, heartbeat, rétention, DSN) — c48f1de (Fixes #79)
✅ Redaction API déjà en place : ntfy.token, email.password, storage.s3.secretKey jamais retournés en clair (***REDACTED***, pattern clés sensibles SettingsController)
✅ 221 tests bUnit verts (12 nouveaux sur les sections)

Reste à faire :

⏳ Services backend à basculer env → config effective : EmailService, NtfyService, AiHealthCheck/FrameAnalyzerService lisent encore Environment.GetEnvironmentVariable/config au lieu des AppSettings (sauf log.level déjà appliqué à chaud)
⏳ Boutons « Tester » : endpoints POST /api/settings/email/test, POST /api/settings/ntfy/test, ping IA (seul le test Storage existe)
⏳ SENTRY_DSN seedé mais nécessite restart pour être pris en compte (config au démarrage)
⏳ AgentDetail.razor : la section overrides par agent affiche encore les clés brutes (refonte visuelle à prévoir dans un suivi)

Tickets liés : #79 fermé.

## Progression (12/08/2026) **Fait :** - ✅ Seed idempotent complet en base (source de vérité DB, migration douce env → DB) : `ntfy.*`, `email.*`, `ai.*` (cpaiUrl, ollamaUrl, detection.enabled), `log.level`, `log.retentionDays`, `heartbeat.interval.seconds`, `sentry.dsn` — commits 26c0e2f + c48f1de - ✅ UI : sections visuelles Notifications (ntfy + SMTP), IA (CodeProject.AI/Ollama/détection), Défauts agents (log, heartbeat, rétention, DSN) — c48f1de (Fixes #79) - ✅ Redaction API déjà en place : `ntfy.token`, `email.password`, `storage.s3.secretKey` jamais retournés en clair (`***REDACTED***`, pattern clés sensibles SettingsController) - ✅ 221 tests bUnit verts (12 nouveaux sur les sections) **Reste à faire :** - ⏳ Services backend à basculer env → config effective : `EmailService`, `NtfyService`, `AiHealthCheck`/`FrameAnalyzerService` lisent encore `Environment.GetEnvironmentVariable`/config au lieu des AppSettings (sauf `log.level` déjà appliqué à chaud) - ⏳ Boutons « Tester » : endpoints `POST /api/settings/email/test`, `POST /api/settings/ntfy/test`, ping IA (seul le test Storage existe) - ⏳ `SENTRY_DSN` seedé mais nécessite restart pour être pris en compte (config au démarrage) - ⏳ AgentDetail.razor : la section overrides par agent affiche encore les clés brutes (refonte visuelle à prévoir dans un suivi) Tickets liés : #79 fermé.

nonobis referenced this issue from a commit

2026-08-12 11:04:06 +02:00

feat(settings): NtfyService/EmailService/AiHealthCheck lisent les AppSettings (partiel #80)

nonobis referenced this issue from a commit

2026-08-12 11:50:49 +02:00

feat(settings): boutons Tester email/ntfy/IA + clés techniques masquées (partiel #80)

nonobis commented

2026-08-12 11:52:55 +02:00

Author

Owner

Clôture (12/08/2026) — tout est implémenté

✅ Seed idempotent (ntfy., email., ai., log., heartbeat.*, sentry.dsn) — source de vérité DB, casse normalisée (HashSet OrdinalIgnoreCase, plus de crash au redémarrage)
✅ UI : sections Notifications / IA / Défauts agents + Storage (#78) — plus de CRUD clé/valeur (#79)
✅ Services branchés sur les AppSettings : NtfyService, EmailService, AiHealthCheck (fallback env) — relus à chaque appel
✅ Boutons Tester : POST api/settings/email/test (envoi réel), ntfy/test (retour HTTP), ai/test (ping CodeProject.AI/Ollama) — commit cb7750a
✅ Secrets jamais en clair (REDACTED)
✅ AgentDetail : clés techniques masquées (libellé global/surchargé)
⚠️ Note : SENTRY_DSN est configurable en UI mais pris en compte au redémarrage (config au démarrage)

Déployé en prod (image heimdall:local, healthy).

## Clôture (12/08/2026) — tout est implémenté - ✅ Seed idempotent (ntfy.*, email.*, ai.*, log.*, heartbeat.*, sentry.dsn) — source de vérité DB, casse normalisée (HashSet OrdinalIgnoreCase, plus de crash au redémarrage) - ✅ UI : sections Notifications / IA / Défauts agents + Storage (#78) — plus de CRUD clé/valeur (#79) - ✅ Services branchés sur les AppSettings : NtfyService, EmailService, AiHealthCheck (fallback env) — relus à chaque appel - ✅ Boutons Tester : POST api/settings/email/test (envoi réel), ntfy/test (retour HTTP), ai/test (ping CodeProject.AI/Ollama) — commit cb7750a - ✅ Secrets jamais en clair (***REDACTED***) - ✅ AgentDetail : clés techniques masquées (libellé global/surchargé) - ⚠️ Note : SENTRY_DSN est configurable en UI mais pris en compte au redémarrage (config au démarrage) Déployé en prod (image heimdall:local, healthy).

nonobis closed this issue

2026-08-12 11:52:56 +02:00

nonobis referenced this issue from a commit

2026-08-12 19:17:20 +02:00

feat(settings): seed clés d'intégration ntfy/email/IA en base (partiel #80)

nonobis referenced this issue from a commit

2026-08-12 19:17:20 +02:00

feat(settings): sections visuelles IA/Notifications/Défauts agents — suppression du CRUD clé/valeur (Fixes #79)

nonobis referenced this issue from a commit

2026-08-12 19:17:20 +02:00

feat(settings): NtfyService/EmailService/AiHealthCheck lisent les AppSettings (partiel #80)

nonobis referenced this issue from a commit