Some checks are pending
verifier / verifier (push) Waiting to run
Cinq jours, cinq defauts, tous de la meme famille : « quelle instance, quel inventaire ? » Neuf modules portaient chacun leur reponse. - 18 aout : P03 comparait chaque instance a l'inventaire d'une AUTRE ; - 19 aout : verifier_ports codait `principal/` en dur ; verifier_intrants et _frontiere_absente lisaient le symlink au lieu de la variable ; - 20 aout : devis_placement rendait un verdict juste sur le mauvais tenant ; - 22 aout : P35, puis P36 — la dixieme, trouvee par la preuve elle-meme. Aucune n'etait une faute d'inattention : chacune avait ete ecrite de bonne foi, a un moment ou le besoin semblait local. C'est le mode de panne de la duplication — pas l'erreur, mais la DERIVE, invisible depuis l'interieur d'un fichier. LA RESOLUTION UNIQUE. `inventory_rules` porte instance_courante(), inventaire_de(), dossier_inventaire() et plan_de(). Trois niveaux de repli, dont le TROISIEME manquait a la moitie des copies : un hosts.yml existant, puis un REPERTOIRE existant (instance neuve — c'est ce qui faisait echouer `make instancier` sur le modele public), puis le defaut. Vingt-huit modules y sont branches. CE QUI REND CE REFACTOR SUR : avant de toucher quoi que ce soit, chaque module a ete interroge sur ce qu'il resolvait, pour les DEUX ecosystemes. Apres refactor, meme mesure : 17 modules x 2 instances, diff VIDE. Aucune resolution n'a change — prouve, pas suppose. P41 echoue des qu'un module reintroduit une copie. Eprouvee en negatif : une copie replacee dans genome.py est signalee avec son numero de ligne. Trois exemptions nommees : instances.py et inventory_gui.py manipulent le SYMLINK lui-meme (bascule d'instance), et devis_opnsense lit deliberement quelle instance est ACTIVE. Elles parlent du lien, pas de la resolution. make verifier 41 OK, 0 echec, 0 saute ; make ci idem ; lint vert. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
102 lines
4.4 KiB
Python
Executable file
102 lines
4.4 KiB
Python
Executable file
#!/usr/bin/env python3
|
|
"""Devis PostgreSQL : le chiffrement est-il réellement imposé, et à qui ?
|
|
|
|
Lit les relevés de `playbooks/maintenance/devis-postgresql.yml`. Sort en code 1 s'il y a
|
|
un écart.
|
|
|
|
Deux défauts déjà vécus dans ce dépôt, et que ce devis rend visibles :
|
|
|
|
- un réseau **écrit** dans `pg_hba.conf` au lieu d'être dérivé (`10.11.0.0/16` sous un
|
|
commentaire « AJUSTER » que personne n'a suivi) : la base acceptait un supernet qui
|
|
n'était pas le sien ;
|
|
- une ligne `host` (en clair) là où il faut `hostssl` : le verrou saute sans bruit,
|
|
puisque les clients en `verify-full` continuent de fonctionner.
|
|
"""
|
|
|
|
from __future__ import annotations
|
|
|
|
import glob
|
|
import ipaddress
|
|
import json
|
|
import sys
|
|
from pathlib import Path
|
|
|
|
from inventory_rules import instance_courante # noqa: E402
|
|
|
|
RACINE = Path(__file__).resolve().parent.parent
|
|
MOTIF = str(instance_courante() / "devis-postgresql.json.*")
|
|
LOCAL = (ipaddress.ip_network("127.0.0.0/8"), ipaddress.ip_network("::1/128"))
|
|
|
|
|
|
def analyser(releves: list[dict]) -> list[str]:
|
|
ecarts = []
|
|
for r in sorted(releves, key=lambda x: x["hote"]):
|
|
h = r["hote"]
|
|
tls = dict(l.split("|", 1) for l in r["tls"] if "|" in l)
|
|
|
|
if tls.get("ssl") != "on":
|
|
ecarts.append(f"{h} : `ssl` n'est pas actif — tout le trafic est en clair")
|
|
cert = tls.get("ssl_cert_file", "")
|
|
if "snakeoil" in cert or not cert:
|
|
ecarts.append(f"{h} : certificat serveur « {cert or 'aucun'} » — ce n'est pas "
|
|
f"celui de l'AC interne, la confiance est-ouest ne tient pas")
|
|
if not tls.get("ssl_ca_file"):
|
|
ecarts.append(f"{h} : aucun `ssl_ca_file` — le serveur ne peut pas verifier "
|
|
f"un certificat client")
|
|
|
|
supernet = ipaddress.ip_network(r["supernet"]) if r.get("supernet") else None
|
|
for ligne in r["hba"]:
|
|
champs = [c.strip() for c in ligne.split("|")]
|
|
if len(champs) < 4:
|
|
continue
|
|
type_, adresse, masque = champs[0], champs[1], champs[2]
|
|
if type_ == "local" or not adresse:
|
|
continue
|
|
# `address` peut valoir « all », « samehost », un nom d'hote, une adresse
|
|
# nue (`::1`) ou un CIDR ; `netmask` n'est renseigne que parfois. On essaie
|
|
# les deux recompositions avant de conclure que ce n'est pas un reseau.
|
|
res = None
|
|
for candidat in ((f"{adresse}/{masque}" if masque else None), adresse):
|
|
if not candidat:
|
|
continue
|
|
try:
|
|
res = ipaddress.ip_network(candidat, strict=False)
|
|
break
|
|
except ValueError:
|
|
res = None
|
|
if res is None:
|
|
if type_ != "hostssl":
|
|
ecarts.append(f"{h} : `{type_}` (et non `hostssl`) pour « {adresse} » "
|
|
f"— le chiffrement n'est pas IMPOSE, seulement possible")
|
|
continue
|
|
if any(res.subnet_of(l) for l in LOCAL if res.version == l.version):
|
|
continue
|
|
if type_ != "hostssl":
|
|
ecarts.append(f"{h} : `{type_}` (et non `hostssl`) pour {res} — le "
|
|
f"chiffrement n'est pas IMPOSE, seulement possible")
|
|
if supernet and res.version == supernet.version and not res.subnet_of(supernet):
|
|
ecarts.append(f"{h} : {res} est autorise mais n'appartient pas au "
|
|
f"supernet derive {supernet} — un reseau ECRIT au lieu "
|
|
f"d'etre derive")
|
|
return ecarts
|
|
|
|
|
|
def main(argv: list[str] | None = None) -> int:
|
|
fichiers = sorted(glob.glob(argv[0] if argv else MOTIF))
|
|
if not fichiers:
|
|
print("Relevés absents. Lancer d'abord : make postgresql-plan", file=sys.stderr)
|
|
return 2
|
|
releves = [json.loads(Path(f).read_text(encoding="utf-8")) for f in fichiers]
|
|
ecarts = analyser(releves)
|
|
print(f"Devis PostgreSQL — {len(releves)} instance(s)\n")
|
|
for e in ecarts:
|
|
print(f" [écart] {e}")
|
|
if not ecarts:
|
|
print("CONFORME : chiffrement imposé, et aucun réseau hors du supernet dérivé.")
|
|
return 0
|
|
print(f"\nÉCART : {len(ecarts)} divergence(s).")
|
|
return 1
|
|
|
|
|
|
if __name__ == "__main__":
|
|
raise SystemExit(main(sys.argv[1:]))
|