fix(logging): refuser les FIFO sans bloquer le démarrage (#60) #66

Merged

silex merged 1 commit from fix/nonblocking-log-open-60 into main

2026-09-29 22:06:14 +00:00

silex commented

2026-09-29 21:45:12 +00:00

Owner

Fixes #60

Correctif

Ajouter O_NONBLOCK à l'ouverture du log avant validation du descripteur ; conserver O_NOFOLLOW, O_CLOEXEC et toutes les vérifications fichier régulier/EUID/lien unique/mode 0600. Cela couvre OpenDestination, NewRotatingFile et ConfigureFile.

Régressions

Un sous-processus de test appelle chaque point d'entrée sur un FIFO sans lecteur. Avant le correctif, les trois appels bloquent et sont tués/récoltés par leur délai de garde, sans goroutine orpheline. Après le correctif, rejet immédiat.

Tests supplémentaires de non-modification des hardlinks, symlinks, FIFO avec lecteur et répertoires ; contrôle nominal de conservation du contenu, append et normalisation 0600. Les tests existants de rotation bornée restent verts.

Vérification

50 répétitions de tout internal/logging sous -race.
make lint test GO_TEST_FLAGS='-race -count=1'.
git diff --check.

Documentation de configuration actualisée. PR indépendante de #65 ; binaire local inchangé.

Fixes #60 ## Correctif Ajouter O_NONBLOCK à l'ouverture du log avant validation du descripteur ; conserver O_NOFOLLOW, O_CLOEXEC et toutes les vérifications fichier régulier/EUID/lien unique/mode 0600. Cela couvre OpenDestination, NewRotatingFile et ConfigureFile. ## Régressions Un sous-processus de test appelle chaque point d'entrée sur un FIFO sans lecteur. Avant le correctif, les trois appels bloquent et sont tués/récoltés par leur délai de garde, sans goroutine orpheline. Après le correctif, rejet immédiat. Tests supplémentaires de non-modification des hardlinks, symlinks, FIFO avec lecteur et répertoires ; contrôle nominal de conservation du contenu, append et normalisation 0600. Les tests existants de rotation bornée restent verts. ## Vérification - 50 répétitions de tout internal/logging sous -race. - make lint test GO_TEST_FLAGS='-race -count=1'. - git diff --check. Documentation de configuration actualisée. PR indépendante de #65 ; binaire local inchangé.

Rows
Columns