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

Open
opened 2026-07-31 21:05:15 +00:00 by mathieu · 2 comments
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`

Correctif proposé dans la PR #38 (#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.

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

Correctif proposé et validé dans la PR #38 : https://git.2027a.net/mathieu/murmur/pulls/38.
Sign in to join this conversation.
No labels
No milestone
No project
No assignees
2 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference
mathieu/murmur#35
No description provided.