Skip to content

docker-compose for nodejs and postgres connection #321

Open
MathurUtkarsh wants to merge 4 commits intodocker:masterfrom
MathurUtkarsh:master
Open

docker-compose for nodejs and postgres connection #321
MathurUtkarsh wants to merge 4 commits intodocker:masterfrom
MathurUtkarsh:master

Conversation

@MathurUtkarsh
Copy link

This project is a simple NodeJS application using Express framework and Postgres database. The environment is set up using Docker Compose and the connection between NodeJS and Postgres is established through networks in the compose file.

@MathurUtkarsh MathurUtkarsh changed the title docker-compose for nodejs and postgres connection #200 docker-compose for nodejs and postgres connection Feb 2, 2023
MathurUtkarsh and others added 4 commits February 3, 2023 02:48
Signed-off-by: Utkarsh Mathur <utkarshmathur05@gmail.com>
Signed-off-by: Utkarsh Mathur <utkarshmathur05@gmail.com>
Signed-off-by: Utkarsh Mathur <78642104+MathurUtkarsh@users.noreply.github.com>
Signed-off-by: Utkarsh Mathur <utkarshmathur05@gmail.com>
Signed-off-by: Utkarsh Mathur <78642104+MathurUtkarsh@users.noreply.github.com>
Signed-off-by: Utkarsh Mathur <utkarshmathur05@gmail.com>
@MathurUtkarsh MathurUtkarsh changed the title docker-compose for nodejs and postgres connection docker-compose for nodejs and postgres connection #200 Feb 2, 2023
@MathurUtkarsh MathurUtkarsh changed the title docker-compose for nodejs and postgres connection #200 docker-compose for nodejs and postgres connection Feb 2, 2023
Copy link

@stewartmbofana stewartmbofana left a comment

Choose a reason for hiding this comment

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

This is nice example but the quality can be improved a little bit. You should perhaps try to follow the format of other postgres samples. For example take a look at the structure in the nginx-golang-postgres or the react-rust-postgres examples. Otherwise the concept is good.

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.

3 participants