fix(securite): python3 -I partout, runuser sans l'environnement de root #19

Merged

mathieu merged 1 commit from securite-python-runuser into main

2026-10-14 02:22:22 +02:00

mathieu commented

2026-10-14 02:09:04 +02:00

Owner

Deux constats d'une revue de sécurité du 2026-10-14, tous deux antérieurs à #18 :

Imports Python détournables. publier de code et fix tourne dans le dépôt extrait (working-directory: repo) avec ANTHROPIC_API_KEY et LANE_BOT_TOKEN, et lançait python3 - / python3 -c : le répertoire courant passe en tête de sys.path, un json.py livré par l'agent ou l'auteur de la PR aurait été importé en root. Correctif : python3 -I sur les 39 appels (exemples, lancer/).
runuser héritait de l'environnement de root. Un processus lancé sous node avec cet environnement (notamment le kill -KILL -1 du nettoyage) expose le jeton du job dans /proc/<pid>/environ aux processus de node encore en vie. Correctif : env -i PATH=… runuser -u node -- … partout.

Tests : classe Isolement (aucun python3 sans -I, aucun runuser sans env -i préalable, -I exclut le répertoire courant) ; installer.py signale aussi un runuser qui hérite. 49 tests OK ; les nouveaux échouent sur 606be5c. CONTRAT.md décrit les deux règles.

À éprouver sur essai-actions avant fusion. Les tests.yml de fabrique-pdf, mathieu.lecrivain.fr et atelier sont à corriger de même au réépinglage.

Deux constats d'une revue de sécurité du 2026-10-14, tous deux antérieurs à #18 : - **Imports Python détournables.** `publier` de `code` et `fix` tourne dans le dépôt extrait (`working-directory: repo`) avec `ANTHROPIC_API_KEY` et `LANE_BOT_TOKEN`, et lançait `python3 -` / `python3 -c` : le répertoire courant passe en tête de `sys.path`, un `json.py` livré par l'agent ou l'auteur de la PR aurait été importé en root. Correctif : `python3 -I` sur les 39 appels (exemples, `lancer/`). - **`runuser` héritait de l'environnement de root.** Un processus lancé sous `node` avec cet environnement (notamment le `kill -KILL -1` du nettoyage) expose le jeton du job dans `/proc/<pid>/environ` aux processus de `node` encore en vie. Correctif : `env -i PATH=… runuser -u node -- …` partout. Tests : classe `Isolement` (aucun `python3` sans `-I`, aucun `runuser` sans `env -i` préalable, `-I` exclut le répertoire courant) ; `installer.py` signale aussi un `runuser` qui hérite. 49 tests OK ; les nouveaux échouent sur `606be5c`. `CONTRAT.md` décrit les deux règles. À éprouver sur essai-actions avant fusion. Les `tests.yml` de fabrique-pdf, mathieu.lecrivain.fr et atelier sont à corriger de même au réépinglage.

mathieu added 1 commit

2026-10-14 02:09:04 +02:00

fix(securite): python3 -I partout, runuser sans l'environnement de root cf70beed55

Les étapes de root qui portent un secret lançaient python3 dans le dépôt extrait : le répertoire courant passait en tête des imports, un json.py livré aurait tourné avec ANTHROPIC_API_KEY et LANE_BOT_TOKEN. runuser gardait l'environnement de root, jeton du job compris, lisible dans /proc/<pid>/environ par les processus de node encore en vie (nettoyage de fin d'étape). Tests d'isolement ; installer.py signale aussi un runuser qui hérite.

mathieu merged commit 879a8a9677 into main

2026-10-14 02:22:22 +02:00

mathieu deleted branch securite-python-runuser

2026-10-14 02:22:22 +02:00

mathieu referenced this pull request from a commit

2026-10-14 02:22:24 +02:00

Merge pull request 'fix(securite): python3 -I partout, runuser sans l'environnement de root' (#19) from securite-python-runuser into main

mathieu referenced this pull request

2026-10-14 02:39:54 +02:00

Évasion par la CI aval : la PR de la lane code fait exécuter le code de l'agent avec le jeton du job #13

mathieu referenced this pull request

2026-10-14 04:03:18 +02:00