Skip to content

Fix DB port and test access#35

Merged
raynaldlao merged 1 commit into
masterfrom
fix-prod-db-port-and-test-access
Jul 2, 2026
Merged

Fix DB port and test access#35
raynaldlao merged 1 commit into
masterfrom
fix-prod-db-port-and-test-access

Conversation

@raynaldlao

Copy link
Copy Markdown
Owner

Description

Correction du conflit de port PostgreSQL et amélioration du workflow de développement

Problème

La machine hôte a un PostgreSQL 16 natif installé sur le port 5433, qui entre en conflit avec le mapping de port du conteneur Docker db_prod (5433:5432). Le mapping Docker échoue silencieusement → l'application se connecte au PostgreSQL natif (qui a des identifiants différents) au lieu du conteneur Docker → password authentication failed.

Modifications

  1. docker-compose.yml — Passage du port db_prod de 5433 à 5435 pour éliminer le conflit avec le PostgreSQL natif. Aucune perte de données (volume pgdata_prod inchangé).

  2. sqlalchemy_setup_database.py — Ajout du support FLASK_ENV=test. Quand FLASK_ENV=test est défini, l'application se connecte à TEST_DATABASE_URL (BDD test sur le port 5434) au lieu de DATABASE_URL (BDD prod sur le port 5435). Permet un développement local sûr sans toucher aux données de production.

Utilisation

flask run                  → BDD prod (port 5435)
FLASK_ENV=test flask run   → BDD test (port 5434)
Fichiers modifiés
- docker-compose.yml — port 5433 → 5435
- src/infrastructure/output_adapters/sqlalchemy/sqlalchemy_setup_database.py
— routage conditionnel via FLASK_ENV
- .env (gitignoré, à mettre à jour après clone) : port DATABASE_URL
5433 → 5435

…by moving db_prod to 5435, keeps db_test on 5434, and adds FLASK_ENV=test handling in sqlalchemy_setup_database.py to switch cleanly between prod (DATABASE_URL) and test (TEST_DATABASE_URL) connections
@raynaldlao
raynaldlao requested a review from hlargitte July 1, 2026 23:31
@raynaldlao raynaldlao self-assigned this Jul 1, 2026

@hlargitte hlargitte left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@raynaldlao
raynaldlao merged commit 985b3c9 into master Jul 2, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants