Skip to content

chore(setup): add redis and postgres to docker compose - #22

Merged
augustin-laurent merged 5 commits into
mainfrom
feat/redis_postgres_dockercompose
Oct 10, 2025
Merged

chore(setup): add redis and postgres to docker compose#22
augustin-laurent merged 5 commits into
mainfrom
feat/redis_postgres_dockercompose

Conversation

@augustin-laurent

Copy link
Copy Markdown
Collaborator

No description provided.

@rltbg

rltbg commented Oct 8, 2025

Copy link
Copy Markdown

plusieurs points a corriger

  • ca ne sert a rien de charger un fichier env et de definir des variables dans environnement, cest l'un ou l'autre, a moins d'avoir beaucoup de variables d'environnement a traiter on prefere les environnements pour avoir les informations d'un coup d'oeil
  • lorsque l'on est en prod, on ne veut pas servir le postgres ni le redis vers l'exterieur, juste au sein du reseau
  • a reverifier mais il me semble que comme le backend est compile alors il ne prend pas en compte le fichier .env que tu lui donnes
  • il manque les addons postgres (?) et il manque son utilisateur admin

@augustin-laurent

Copy link
Copy Markdown
Collaborator Author

plusieurs points a corriger

  • ca ne sert a rien de charger un fichier env et de definir des variables dans environnement, cest l'un ou l'autre, a moins d'avoir beaucoup de variables d'environnement a traiter on prefere les environnements pour avoir les informations d'un coup d'oeil
  • lorsque l'on est en prod, on ne veut pas servir le postgres ni le redis vers l'exterieur, juste au sein du reseau
  • a reverifier mais il me semble que comme le backend est compile alors il ne prend pas en compte le fichier .env que tu lui donnes
  • il manque les addons postgres (?) et il manque son utilisateur admin

Yes en te taggant sur discord je pensais que t'allais continuer pas faire une review je reprend tes commentaires du coup pour finaliser la branche

@augustin-laurent

augustin-laurent commented Oct 8, 2025

Copy link
Copy Markdown
Collaborator Author

plusieurs points a corriger

  • ca ne sert a rien de charger un fichier env et de definir des variables dans environnement, cest l'un ou l'autre, a moins d'avoir beaucoup de variables d'environnement a traiter on prefere les environnements pour avoir les informations d'un coup d'oeil
  • lorsque l'on est en prod, on ne veut pas servir le postgres ni le redis vers l'exterieur, juste au sein du reseau
  • a reverifier mais il me semble que comme le backend est compile alors il ne prend pas en compte le fichier .env que tu lui donnes
  • il manque les addons postgres (?) et il manque son utilisateur admin
  • C'est corrigé
  • C'est basculé en expose pour garder le port accessible en interne
  • J'ai vérifié et oui si il est compilé, le .env n'est pas pris en compte
  • J'ai ajouté les 4 dont je t'avais parlé sur discord certaines ne seront peut-être pas utiles on avisera plus tard

@sonarqubecloud

sonarqubecloud Bot commented Oct 8, 2025

Copy link
Copy Markdown

@augustin-laurent
augustin-laurent merged commit 44d1124 into main Oct 10, 2025
8 checks passed
@augustin-laurent
augustin-laurent deleted the feat/redis_postgres_dockercompose branch October 10, 2025 10:29
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