Merge pull request #28 from R0m1k3/claude/combohr-app-recreation-plan-yu7wre

Créer le rôle applicatif à chaque démarrage, pas seulement au premier

docker-compose.yml

@@ -34,18 +34,30 @@ services:
     # vrai problème.
     networks:
       - interne
-    # Le conteneur reste sur 5432 pour l'application (réseau interne), mais le
+    # Le conteneur reste sur 5432 pour l'application, et le port hôte est publié
-    # port hôte est publié sur un port non commun, configurable : plusieurs
+    # sur un port non commun, configurable : plusieurs instances PostgreSQL
-    # instances PostgreSQL coexistent sur le même serveur, et s'en tenir au
+    # coexistent sur le même serveur, et s'en tenir au 5432 par défaut les
-    # 5432 par défaut les ferait entrer en collision.
+    # ferait entrer en collision.
+    #
+    # **Lié à la boucle locale**, et c'est important : la publication sert à se
+    # connecter depuis le serveur (sauvegarde, psql), pas depuis le réseau. Sans
+    # `127.0.0.1`, Docker ouvre le port sur toutes les interfaces — un jeu de
+    # données RH derrière un mot de passe par défaut, joignable depuis
+    # l'extérieur. Pour un accès distant, passer par un tunnel.
     ports:
-      - '${POSTGRES_PORT:-55432}:5432'
+      - '${POSTGRES_BIND:-127.0.0.1}:${POSTGRES_PORT:-55432}:5432'
 
-  # Provisionne le rôle applicatif à chaude occurrence de `up`, pas seulement à
+  # Pose le rôle applicatif à **chaque** démarrage de la pile.
-  # la première initialisation du volume : inversement, une base existante dont
+  #
-  # le rôle est absent (ou dont le mot de passe a changé) laisserait l'app en
+  # `/docker-entrypoint-initdb.d` ne s'exécute qu'à la toute première
-  # échec d'authentification P1000 sans recours autre que du SQL à la main.
+  # initialisation du volume : une base dont le volume existait déjà — ou dont
-  # Idempotent : rejouable autant de fois que nécessaire.
+  # le mot de passe a changé — laisserait l'application en échec
+  # d'authentification P1000, sans recours autre que du SQL à la main. Le script
+  # est rejouable, ce service le rejoue.
+  #
+  # Les identifiants du superutilisateur restent ici : les confier au conteneur
+  # applicatif lui donnerait de quoi contourner la row-level security, ce que
+  # tout ce montage cherche précisément à empêcher.
   db-init:
     image: postgres:16-alpine
     restart: 'no'
@@ -53,15 +65,17 @@ services:
       db:
         condition: service_healthy
     environment:
-      POSTGRES_USER: ${POSTGRES_USER:-planflow}
+      PGHOST: db
       PGPASSWORD: ${POSTGRES_PASSWORD:-planflow-interne}
+      POSTGRES_USER: ${POSTGRES_USER:-planflow}
       POSTGRES_DB: ${POSTGRES_DB:-planflow}
       APP_DB_USER: ${APP_DB_USER:-planflow_app}
       APP_DB_PASSWORD: ${APP_DB_PASSWORD:-planflow-app-interne}
-      PGHOST: db
     volumes:
-      - ./docker/init-app-role.sh:/docker-entrypoint-initdb.d/10-init-app-role.sh:ro
+      - ./docker/init-app-role.sh:/init-app-role.sh:ro
-    command: ['sh', '/docker-entrypoint-initdb.d/10-init-app-role.sh']
+    # `entrypoint` et non `command` : l'image postgres a son propre point
+    # d'entrée, qui tenterait de démarrer un serveur.
+    entrypoint: ['sh', '/init-app-role.sh']
     networks:
       - interne
 

docker/init-app-role.sh

@@ -14,19 +14,23 @@ set -eu
 # ni superutilisateur ni BYPASSRLS. C'est précisément pourquoi les politiques
 # sont déclarées en FORCE : elles s'appliquent aussi au propriétaire.
 #
-# Playé par le conteneur db à la **première** initialisation du volume, puis par
+# Joué par le conteneur `db` à la **première** initialisation du volume, puis
-# le service `db-init` à chaque `docker compose up`. Idempotent : la branche
+# par le service `db-init` à chaque `docker compose up`. Les deux sont
-# `ELSE` re-synchronise le mot de passe d'une base déjà en place.
+# nécessaires : `/docker-entrypoint-initdb.d` ne s'exécute qu'une fois, et une
+# installation dont le volume existait déjà n'aurait jamais vu passer ce rôle.
+# Le script est donc rejouable de bout en bout.
 
 APP_ROLE="${APP_DB_USER:-planflow_app}"
 APP_PASSWORD="${APP_DB_PASSWORD:-planflow-app-interne}"
+DB_NAME="${POSTGRES_DB:-planflow}"
+DB_USER="${POSTGRES_USER:-planflow}"
 
-# En officiation dans le conteneur db, l'hôte est local ; depuis le service
+# Dans le conteneur `db`, l'hôte est local ; depuis `db-init`, c'est `db`.
-# db-init, il est `db`. L'un et l'autre doivent aboutir au même SQL.
+# L'un et l'autre doivent aboutir au même SQL.
 if [ -n "${PGHOST:-}" ]; then
-  set -- -h "$PGHOST" -p "${PGPORT:-5432}" -U "$POSTGRES_USER" -d "$POSTGRES_DB" -v ON_ERROR_STOP=1
+  set -- -h "$PGHOST" -p "${PGPORT:-5432}" -U "$DB_USER" -d "$DB_NAME" -v ON_ERROR_STOP=1
 else
-  set -- -U "$POSTGRES_USER" -d "$POSTGRES_DB" -v ON_ERROR_STOP=1
+  set -- -U "$DB_USER" -d "$DB_NAME" -v ON_ERROR_STOP=1
 fi
 
 psql "$@" <<SQL
@@ -36,15 +40,36 @@ BEGIN
     CREATE ROLE ${APP_ROLE} LOGIN PASSWORD '${APP_PASSWORD}'
       NOSUPERUSER NOCREATEDB NOCREATEROLE NOBYPASSRLS;
   ELSE
+    -- Le mot de passe peut avoir changé dans la configuration de la pile ;
+    -- laisser l'ancien produirait un refus d'authentification illisible.
     ALTER ROLE ${APP_ROLE} WITH LOGIN PASSWORD '${APP_PASSWORD}'
       NOSUPERUSER NOCREATEDB NOCREATEROLE NOBYPASSRLS;
   END IF;
 END
 \$\$;
 
-ALTER DATABASE ${POSTGRES_DB} OWNER TO ${APP_ROLE};
+ALTER DATABASE ${DB_NAME} OWNER TO ${APP_ROLE};
 ALTER SCHEMA public OWNER TO ${APP_ROLE};
 GRANT ALL ON SCHEMA public TO ${APP_ROLE};
+
+-- Objets déjà créés par un compte d'amorçage : sans ce transfert, le rôle
+-- applicatif ne pourrait ni migrer ni lire ce qui existe déjà. Le cas se
+-- produit dès qu'une base a tourné avant que ce rôle n'existe.
+DO \$\$
+DECLARE
+  statement text;
+BEGIN
+  FOR statement IN
+    SELECT format('ALTER TABLE %I.%I OWNER TO ${APP_ROLE}', schemaname, tablename)
+    FROM pg_tables WHERE schemaname = 'public'
+    UNION ALL
+    SELECT format('ALTER SEQUENCE %I.%I OWNER TO ${APP_ROLE}', sequence_schema, sequence_name)
+    FROM information_schema.sequences WHERE sequence_schema = 'public'
+  LOOP
+    EXECUTE statement;
+  END LOOP;
+END
+\$\$;
 SQL
 
-echo "Rôle applicatif ${APP_ROLE} prêt — NOSUPERUSER NOBYPASSRLS, propriétaire de ${POSTGRES_DB}."
+echo "Rôle applicatif ${APP_ROLE} prêt — NOSUPERUSER NOBYPASSRLS, propriétaire de ${DB_NAME}."

tests/e2e/roles.spec.ts

@@ -9,7 +9,13 @@ import { slugifyRoleKey } from '../../src/domain/access/role-editing';
  * changement de code. » Le vérifier suppose d'aller jusqu'au bout : créer le
  * rôle, l'attribuer, et constater qu'un écran s'ouvre ou se ferme en
  * conséquence. Un test qui s'arrêterait à « la case est cochée » ne dirait rien.
+ *
+ * En série, et il le faut : l'un de ces tests retire momentanément « Gérer les
+ * rôles » aux rôles semés pour atteindre le refus de verrouillage. Joué en
+ * parallèle des autres, qui s'appuient sur cette même capacité, il les fait
+ * échouer par intermittence — et la cause n'est visible nulle part.
  */
+test.describe.configure({ mode: 'serial' });
 
 const OWNER_PASSWORD = 'planflow-demo-2026';