Skip to content

Manage environment variables : Paramétrer les ports PostgreSQL hôte dans docker-compose via les variables d'environnement - #56

Merged
raynaldlao merged 1 commit into
masterfrom
manage-environment-variables
Aug 10, 2026
Merged

Manage environment variables : Paramétrer les ports PostgreSQL hôte dans docker-compose via les variables d'environnement#56
raynaldlao merged 1 commit into
masterfrom
manage-environment-variables

Conversation

@raynaldlao

Copy link
Copy Markdown
Owner

Description

Paramétrage des ports PostgreSQL hôte dans docker-compose.yml via les variables d'environnement, afin d'aligner le fichier compose sur la configuration déjà centralisée dans .env.

Motivation

Les ports hôtes étaient codés en dur (5435 et 5434), ce qui imposait une modification du fichier compose à chaque changement de port. Le .env contient déjà POSTGRES_PROD_PORT et POSTGRES_TEST_PORT : docker-compose doit les consommer.

Modifications apportées

  • Remplacement des ports hôtes en dur par les variables d'environnement correspondantes dans les services db_prod et db_test
  • Conservation des ports internes des conteneurs et des URLs Flyway sur 5432

Impact

Aucun changement de comportement : les valeurs par défaut du .env (5435 / 5434) sont conservées. Un seul point de vérité pour la configuration des ports.

…SQL host ports are now parameterized while keeping the internal 5432 defaults. docker‑compose uses **${POSTGRES_PROD_PORT}** and **${POSTGRES_TEST_PORT}** for host mappings, with container ports and Flyway URLs left on 5432
@raynaldlao raynaldlao self-assigned this Aug 2, 2026
@raynaldlao
raynaldlao requested a review from hlargitte August 2, 2026 09:16

@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 ae75d19 into master Aug 10, 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