Security-Review: Docker abschalten, Checkout nach dem Agenten entfernen

[Musiker15]

Härtet das Security-Review nach einer zweiten Kritik am gehärteten Stand.

Was sich ändert

• Docker ist abgeschaltet. Der Läufer gehört zur Gruppe docker, und docker run -v /:/host ist so gut wie root. Der Schritt heißt jetzt "sudo und Docker abschalten": er stoppt docker.socket, docker.service und containerd.service, solange sudo das noch kann, und bricht ab, wenn danach sudo -n true oder docker info noch gelingt.
• Der Checkout geht gleich nach dem Agenten. Er ist der einzige Ort, an dem der Agent schreiben kann, .git eingeschlossen. Der Post-Schritt von actions/checkout ruft dort am Ende des Jobs git und git submodule foreach auf, außerhalb der Sandbox und mit der Umgebung des Jobs. Ein neuer Schritt "Checkout entfernen" (if: always()) entfernt pr/; nach dem Agenten braucht ihn kein Schritt mehr.
• Kein hängender Status mehr. Lehnt die API das Review ab (Freigabe durch Actions abgeschaltet, Text zu lang, API gestört), brach der Schritt bisher vor dem Setzen des Status ab, und PR-Security-Review blieb auf "läuft". Jetzt steht er dann auf error mit einem Verweis auf den Lauf, und der Job wird rot.
• Restrisiko benannt. kernel.apparmor_restrict_unprivileged_userns=0 öffnet den Kern über User Namespaces wieder für jeden Prozess des Jobs. Das steht jetzt als Kommentar am Schritt "Sandbox vorbereiten": bewusst getragen, nur für PRs von Mitgliedern, ein PR von außen bekommt keine Shell.

Eigene Festlegungen, die Moritz noch nicht gesehen hat

• Die Freigabe (APPROVE) bei einem Review ohne Befund bleibt, wie Moritz es entschieden hat. Sie ist auf main kein Tor: das Ruleset verlangt keine Freigabe (required_approving_review_count: 0), und PR-Security-Review steht nicht unter den Pflichtprüfungen. Eine Freigabe, die ein Mitglied über einen eigenen Workflow erzeugt, öffnet deshalb nichts; einen Status mit demselben Namen könnte es genauso setzen. Wirksam dagegen wäre CODEOWNERS für .github/** mit Pflicht zur Freigabe durch den Eigentümer, das liegt bei Moritz.
• systemctl stop darf scheitern (|| true), etwa wenn ein Abbild Docker nicht mitbringt; maßgeblich ist die Prüfung mit docker info danach.

Geprüft

• YAML liest sich, die drei Textprüfungen der CI sind grün.
• Ein Test-PR in opengewerk-kanzlei (Test: Security-Review auf dem Prüfzweig von .github#36 opengewerk-kanzlei#51), dessen Aufruf auf 426c2cc zeigt: alle drei Jobs grün, "sudo und Docker abschalten" mit beiden Prüfungen durch, "Checkout entfernen" vor dem Post-Schritt von actions/checkout, der danach ohne Fehler endet, Status gesetzt. Rot war der Status allein am erwarteten Befund "Pin liegt nicht auf main"; den Inhalt dieses Zweigs bewertet das Review dort als Härtung ohne Befund.