docs: ADR 0003 — accès base par tunnel SSH, jamais par le réseau (#65) #97

Merged

gabriel merged 2 commits from lenaic/65-adr-acces-base-tunnel-ssh into develop 2026-09-03 13:20:16 +00:00

Conversation 4 Commits 2 Files changed 2 +43

lenaic commented 2026-09-02 19:22:29 +00:00

Owner

Copy link

Ce que ça change

La décision « accès base par tunnel SSH uniquement » (journal du 01/09) est maintenant tracée en ADR, avec ses trois clôtures et le geste d'accès documenté. Aucun code ne change.

Closes #65

Preuve

$ ss -lnt | grep 5433                      # sur le serveur
LISTEN 0  200  127.0.0.1:5433  0.0.0.0:*
$ bash -c "</dev/tcp/10.105.200.41/5433"   # depuis l'extérieur (VPN)
→ fermé (timeout)
$ grep -E "^(listen_addresses|port)" infra/compose/postgres/conf/postgresql.conf
listen_addresses = '127.0.0.1'
port = 5433

Relecture

• Un pair a relu et laissé un commentaire, même court
• Ses remarques sont traitées, ou une réponse explique pourquoi elles ne le sont pas

Ce qui suit le code

• docs/adr/ complété, une décision structurante a été prise

Où regarder en priorité

La section Conséquences : elle engage les futurs services (connexion en 127.0.0.1:5433, réseau hôte).

## Ce que ça change La décision « accès base par tunnel SSH uniquement » (journal du 01/09) est maintenant tracée en ADR, avec ses trois clôtures et le geste d'accès documenté. Aucun code ne change. Closes #65 ## Preuve ``` $ ss -lnt | grep 5433 # sur le serveur LISTEN 0 200 127.0.0.1:5433 0.0.0.0:* $ bash -c "</dev/tcp/10.105.200.41/5433" # depuis l'extérieur (VPN) → fermé (timeout) $ grep -E "^(listen_addresses|port)" infra/compose/postgres/conf/postgresql.conf listen_addresses = '127.0.0.1' port = 5433 ``` ## Relecture - [ ] Un pair a relu et laissé un commentaire, même court - [ ] Ses remarques sont traitées, ou une réponse explique pourquoi elles ne le sont pas ## Ce qui suit le code - [x] `docs/adr/` complété, une décision structurante a été prise ## Où regarder en priorité La section Conséquences : elle engage les futurs services (connexion en `127.0.0.1:5433`, réseau hôte).

lenaic added 1 commit 2026-09-02 19:22:30 +00:00

docs: ADR 0003 — accès base par tunnel SSH, jamais par le réseau (#65) 0191acd34a

lenaic referenced this pull request 2026-09-02 19:22:48 +00:00

[décision] Fermer l'accès distant à PostgreSQL et passer par un tunnel SSH #65

florian approved these changes 2026-09-03 07:35:22 +00:00

Dismissed

olivier referenced this pull request 2026-09-03 07:58:29 +00:00

infra: pose le registre de modèles MLflow sur PostgreSQL et MinIO #100

lenaic dismissed florian's review 2026-09-03 09:45:39 +00:00

lenaic requested reviews from gabriel, justine 2026-09-03 09:45:44 +00:00

gabriel requested changes 2026-09-03 10:24:49 +00:00

Dismissed

gabriel left a comment

Copy link

Sur le fond, rien à redire : les trois clôtures (listen_addresses / pg_hba / UFW en réseau hôte) sont cohérentes avec l'infra réelle, la remarque sur DOCKER-USER passant avant UFW est juste, et le geste ssh -L est correct.

Deux points de forme bloquants :

1. Collision de numéro. docs/adr/0003-* existe déjà sur develop (0003-duckdb-plutot-que-spark-pour-l-etl.md). Cette PR a été ouverte avant le merge de renumérotation. Il faut rebaser sur develop et renuméroter. Attention : 0008 est aussi visé par #110 et #109 pose une ADR — je coordonne l'attribution des numéros, on se cale sur l'ordre de merge.

2. Définition de terminé incomplète. Le docs/adr/README.md dit qu'une décision « n'existe que si elle est ici ou dans le tableau §1 de EXIGENCES-collectives.md ». Ajouter la ligne dans le tableau d'index de l'ADR README (a minima).

Sur le fond, rien à redire : les trois clôtures (listen_addresses / pg_hba / UFW en réseau hôte) sont cohérentes avec l'infra réelle, la remarque sur DOCKER-USER passant avant UFW est juste, et le geste `ssh -L` est correct. Deux points de forme bloquants : 1. **Collision de numéro.** `docs/adr/0003-*` existe déjà sur `develop` (`0003-duckdb-plutot-que-spark-pour-l-etl.md`). Cette PR a été ouverte avant le merge de renumérotation. Il faut rebaser sur `develop` et renuméroter. Attention : 0008 est aussi visé par #110 et #109 pose une ADR — je coordonne l'attribution des numéros, on se cale sur l'ordre de merge. 2. **Définition de terminé incomplète.** Le `docs/adr/README.md` dit qu'une décision « n'existe que si elle est ici ou dans le tableau §1 de EXIGENCES-collectives.md ». Ajouter la ligne dans le tableau d'index de l'ADR README (a minima).

lenaic added 1 commit 2026-09-03 12:56:50 +00:00

Merge branch 'develop' into lenaic/65-adr-acces-base-tunnel-ssh 9973c8a11b

lenaic added 1 commit 2026-09-03 13:09:19 +00:00

Merge branch 'develop' into lenaic/65-adr-acces-base-tunnel-ssh f756f0b63c

lenaic force-pushed lenaic/65-adr-acces-base-tunnel-ssh from f756f0b63c to 06f086755f 2026-09-03 13:14:13 +00:00 Compare

lenaic commented 2026-09-03 13:14:50 +00:00

Author

Owner

Copy link

Les deux points sont traités.

Le numéro. Tu as raison, et la collision était pire que 0003 : la série est allée
jusqu'à 0009 depuis l'ouverture de cette demande, avec le collecteur en cron (#109) et
le schéma de la zone or (#110). La fiche devient 0010, premier libre. J'ai rebasé sur
develop et vérifié qu'aucun document ne renvoie à l'ancien nom.

L'index. Ligne ajoutée au tableau du README des fiches. Tu as raison de le compter
comme bloquant plutôt que comme un détail : le README dit lui-même qu'une décision qui n'y
figure pas n'existe pas, et une fiche orpheline est exactement ce qu'on ne retrouve plus
au jour 9.

Sur ta coordination des numéros : elle a déjà servi deux fois aujourd'hui, la fiche de
Justine et celle d'Olivier se disputaient le 0008. Si tu veux, je peux ajouter un contrôle
à la chaîne qui refuse deux fiches au même numéro — ça éviterait de compter à la main à
chaque fusion.

Les deux points sont traités. **Le numéro.** Tu as raison, et la collision était pire que 0003 : la série est allée jusqu'à **0009** depuis l'ouverture de cette demande, avec le collecteur en cron (#109) et le schéma de la zone or (#110). La fiche devient **0010**, premier libre. J'ai rebasé sur `develop` et vérifié qu'aucun document ne renvoie à l'ancien nom. **L'index.** Ligne ajoutée au tableau du `README` des fiches. Tu as raison de le compter comme bloquant plutôt que comme un détail : le README dit lui-même qu'une décision qui n'y figure pas n'existe pas, et une fiche orpheline est exactement ce qu'on ne retrouve plus au jour 9. Sur ta coordination des numéros : elle a déjà servi deux fois aujourd'hui, la fiche de Justine et celle d'Olivier se disputaient le 0008. Si tu veux, je peux ajouter un contrôle à la chaîne qui refuse deux fiches au même numéro — ça éviterait de compter à la main à chaque fusion.

lenaic requested review from gabriel 2026-09-03 13:14:59 +00:00

gabriel approved these changes 2026-09-03 13:19:19 +00:00

gabriel merged commit 9a9c89bed3 into develop 2026-09-03 13:20:16 +00:00

gabriel deleted branch lenaic/65-adr-acces-base-tunnel-ssh 2026-09-03 13:20:16 +00:00

gabriel referenced this pull request from a commit 2026-09-03 13:20:18 +00:00

Merge pull request 'docs: ADR 0003 — accès base par tunnel SSH, jamais par le réseau (#65)' (#97) from lenaic/65-adr-acces-base-tunnel-ssh into develop

gabriel referenced this pull request 2026-09-04 08:10:06 +00:00

docs: réaligne PRD et BACKLOG sur l'état réel de la forge, ajoute les comptes-rendus J1→J4 (#48) #144

Sign in to join this conversation.

Reviewers

No reviewers

justine

gabriel

Labels

Clear labels   

Compat/Breaking

Breaking change that won't be backward compatible

  

EC01

Preuve pour l'épreuve Architecture

  

EC02

Preuve pour l'épreuve Management

  

EC03

Preuve pour l'épreuve CI/CD

  

EC04

Preuve pour l'épreuve Cloud

  

EC05

Preuve pour l'épreuve Data et BI

  

EC06

Preuve pour l'épreuve IA

  

Kind/BDD

BDD

  

Kind/Back

Back

  

Kind/Bug

Something is not working

  

Kind/CICD

CICD

  

Kind/Cloud

Cloud

  

Kind/Contenu

Contenu

  

Kind/Data

Data

  

Kind/Documentation

Documentation changes

  

Kind/Enhancement

Improve existing functionality

  

Kind/Feature

New functionality

  

Kind/Front

Front

  

Kind/IA

IA

  

Kind/Infra

Serveur, déploiement, chaîne, supervision

  

Kind/Monitoring

Monitoring

  

Kind/Security

This is security issue

  

Kind/Testing

Issue or pull request related to testing

  

Portée/Post-jury

Périmètre documenté, hors de la fenêtre du 11/09. Repris après le jury. Rien nest supprimé.

  

Priority

Critical

The priority is critical

  

Priority

High

The priority is high

  

Priority

Low

The priority is low

  

Priority

Medium

The priority is medium

  

Reviewed

Confirmed

Issue has been confirmed

  

Reviewed

Duplicate

This issue or pull request already exists

  

Reviewed

Invalid

Invalid issue

  

Reviewed

Won't Fix

This issue won't be fixed

  

Status

Abandoned

Somebody has started to work on this but abandoned work

  

Status

Blocked

Something is blocking this issue or pull request

  

Status

Need More Info

Feedback is required to reproduce issue or to continue work

  

ops/alerte

Alerte d'exploitation (relais Grafana, #42)

No labels

Compat/Breaking

EC01

EC02

EC03

EC04

EC05

EC06

Kind/BDD

Kind/Back

Kind/Bug

Kind/CICD

Kind/Cloud

Kind/Contenu

Kind/Data

Kind/Documentation

Kind/Enhancement

Kind/Feature

Kind/Front

Kind/IA

Kind/Infra

Kind/Monitoring

Kind/Security

Kind/Testing

Portée/Post-jury

Priority

Critical

Priority

High

Priority

Low

Priority

Medium

Reviewed

Confirmed

Reviewed

Duplicate

Reviewed

Invalid

Reviewed

Won't Fix

Status

Abandoned

Status

Blocked

Status

Need More Info

ops/alerte

Milestone

Clear milestone

No items

No milestone

Projects

Clear projects

No items

No project

Assignees

Clear assignees

No assignees

3 participants

Notifications

Subscribe

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference

g2/enervision!97

No description provided.