[P2][config] Valider chaque provider une seule fois dans un ordre déterministe #4

New issue

Closed

opened 2026-07-31 21:05:10 +00:00 by mathieu · 4 comments

mathieu commented

2026-07-31 21:05:10 +00:00

Owner

Constat

Le provider STT primaire et le fallback sont validés séparément puis à nouveau pendant le parcours de map (internal/config/validate.go:130-180). Les providers inactifs STT et cleanup sont parcourus dans l'ordre aléatoire des maps, donc une même configuration peut produire des diagnostics différents.

Tâche

Préserver la priorité primaire/fallback, puis valider une seule fois les entrées restantes triées par nom. Appliquer la même règle au cleanup.

Critères d'acceptation

Chaque provider est validé exactement une fois.
Primaire et fallback gardent leur priorité d'erreur.
Les erreurs des entrées inactives sont stables sur 100 exécutions.
Les configurations valides se résolvent à l'identique.

Validation

go test -count=100 ./internal/config ./internal/doctor && go test -race ./internal/config ./internal/doctor

## Constat Le provider STT primaire et le fallback sont validés séparément puis à nouveau pendant le parcours de map (`internal/config/validate.go:130-180`). Les providers inactifs STT et cleanup sont parcourus dans l'ordre aléatoire des maps, donc une même configuration peut produire des diagnostics différents. ## Tâche Préserver la priorité primaire/fallback, puis valider une seule fois les entrées restantes triées par nom. Appliquer la même règle au cleanup. ## Critères d'acceptation - Chaque provider est validé exactement une fois. - Primaire et fallback gardent leur priorité d'erreur. - Les erreurs des entrées inactives sont stables sur 100 exécutions. - Les configurations valides se résolvent à l'identique. ## Validation `go test -count=100 ./internal/config ./internal/doctor && go test -race ./internal/config ./internal/doctor`

hermes referenced this issue from a pull request that will close it,

2026-08-07 17:14:04 +00:00

fix(config): aligner validation providers et politiques réseau #38

hermes commented

2026-08-07 17:14:05 +00:00

Member

Correctif proposé dans la PR #38 (mathieu/murmur#38). Les validations ciblées et sous race décrites dans la PR passent.

Correctif proposé dans la PR #38 (https://git.2027a.net/mathieu/murmur/pulls/38). Les validations ciblées et sous race décrites dans la PR passent.

hermes commented

2026-08-07 18:02:33 +00:00

Member

Correctif proposé et validé dans la PR #38 : mathieu/murmur#38.

Correctif proposé et validé dans la PR #38 : https://git.2027a.net/mathieu/murmur/pulls/38.

silex commented

2026-09-23 12:36:32 +00:00

Owner

Corrigé sur main par add0d10 (intégré dans e3896f5). Validation unique dans l’ordre primaire/fallback puis entrées triées ; tests config/doctor sous -race validés. Fermeture du ticket devenu obsolète ; la PR #38 revendique aussi ce correctif mais n'est pas requise pour celui-ci.

Corrigé sur main par add0d10 (intégré dans e3896f5). Validation unique dans l’ordre primaire/fallback puis entrées triées ; tests config/doctor sous -race validés. Fermeture du ticket devenu obsolète ; la PR #38 revendique aussi ce correctif mais n'est pas requise pour celui-ci.

mathieu commented

2026-09-23 12:37:38 +00:00

Author

Owner

Résolu sur main par add0d10 (intégré dans e3896f5). Validation unique, primaire/fallback puis noms triés ; tests config/doctor sous -race validés. La PR #38 revendique également des corrections déjà intégrées ; son traitement reste indépendant.

Résolu sur main par add0d10 (intégré dans e3896f5). Validation unique, primaire/fallback puis noms triés ; tests config/doctor sous -race validés. La PR #38 revendique également des corrections déjà intégrées ; son traitement reste indépendant.

mathieu closed this issue

2026-09-23 12:37:38 +00:00

mathieu referenced this issue

2026-09-23 12:58:40 +00:00

fix(config): aligner validation providers et politiques réseau #38