[P2][cleanup] Appliquer la politique HTTPS dans le constructeur défensif #18

New issue

Closed

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

mathieu commented

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

Owner

Constat

La validation config interdit HTTP en zone Internet sans opt-in, mais cleanup.NewOpenAICompatible ne reçoit pas AllowInsecure et n'applique aucun contrôle (internal/cleanup/cleanup.go:27-85). Les callers directs divergent du constructeur STT.

Tâche

Ajouter l'option minimale et refuser Internet+HTTP avant toute requête ou transmission de bearer, puis la propager depuis core.BuildCleanup.

Critères d'acceptation

Internet HTTP est refusé par défaut avant réseau.
L'opt-in explicite fonctionne.
Internet HTTPS et local/LAN HTTP restent valides.
Préflight et constructeur donnent le même verdict.

Validation

go test -race ./internal/cleanup ./internal/config ./internal/core

## Constat La validation config interdit HTTP en zone Internet sans opt-in, mais `cleanup.NewOpenAICompatible` ne reçoit pas `AllowInsecure` et n'applique aucun contrôle (`internal/cleanup/cleanup.go:27-85`). Les callers directs divergent du constructeur STT. ## Tâche Ajouter l'option minimale et refuser Internet+HTTP avant toute requête ou transmission de bearer, puis la propager depuis `core.BuildCleanup`. ## Critères d'acceptation - Internet HTTP est refusé par défaut avant réseau. - L'opt-in explicite fonctionne. - Internet HTTPS et local/LAN HTTP restent valides. - Préflight et constructeur donnent le même verdict. ## Validation `go test -race ./internal/cleanup ./internal/config ./internal/core`

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.

mathieu commented

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

Author

Owner

Résolu sur main par add0d10 (intégré dans e3896f5). HTTP Internet refusé sans allow_insecure dans le constructeur cleanup ; propagation depuis core ; tests cleanup/core 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). HTTP Internet refusé sans allow_insecure dans le constructeur cleanup ; propagation depuis core ; tests cleanup/core 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:39 +00:00

mathieu referenced this issue

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

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