[P2][config] Classer un rejet de plugin par zone comme policy_blocked #35

New issue

Closed

opened 2026-07-31 21:05:15 +00:00 by mathieu · 3 comments

mathieu commented

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

Owner

Constat

config.Validate classe un plugin local exclu de policy.plugin_zones en config_invalid (internal/config/validate.go:112-123), tandis que plugin.New classe le même rejet policy_blocked (internal/plugin/plugin.go:99-105). Le constructeur appelle d'abord Validate, rendant sa classification défensive inaccessible.

Tâche

Retourner l'erreur policy pour une zone valide mais interdite; garder config_invalid pour une zone inconnue ou une liste mal formée.

Critères d'acceptation

Rejet d'une zone valide => policy_blocked partout.
Nom de zone invalide => config_invalid.
Resolve, doctor, core et CLI sont cohérents.
Le rejet précède capture et exécution plugin.

Validation

go test -race ./internal/config ./internal/core ./internal/plugin ./internal/doctor ./internal/cli

## Constat `config.Validate` classe un plugin local exclu de `policy.plugin_zones` en `config_invalid` (`internal/config/validate.go:112-123`), tandis que `plugin.New` classe le même rejet `policy_blocked` (`internal/plugin/plugin.go:99-105`). Le constructeur appelle d'abord Validate, rendant sa classification défensive inaccessible. ## Tâche Retourner l'erreur policy pour une zone valide mais interdite; garder `config_invalid` pour une zone inconnue ou une liste mal formée. ## Critères d'acceptation - Rejet d'une zone valide => `policy_blocked` partout. - Nom de zone invalide => `config_invalid`. - Resolve, doctor, core et CLI sont cohérents. - Le rejet précède capture et exécution plugin. ## Validation `go test -race ./internal/config ./internal/core ./internal/plugin ./internal/doctor ./internal/cli`

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:06 +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:34 +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.

mathieu commented

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

Author

Owner

Résolu sur main par add0d10 (intégré dans e3896f5). Zone plugin valide interdite classée policy_blocked et zone inconnue config_invalid ; tests config/plugin/core/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). Zone plugin valide interdite classée policy_blocked et zone inconnue config_invalid ; tests config/plugin/core/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:40 +00:00

mathieu referenced this issue

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

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