Skip to content
This repository has been archived by the owner on Oct 8, 2024. It is now read-only.

Dockerfile #110

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Dockerfile #110

wants to merge 3 commits into from

Conversation

bendeckdavid
Copy link

  • Added dockerfile for the frontend
  • Install netcat in the backend to avoid infinite loop restart of the docker container
  • Some useful comments

@johan-fvh
Copy link
Contributor

Hi @bendeckdavid & thanks for the PR! The dockerfiles have been sadly neglected, good to see you making the effort to bring them up to date! Looks like there should maybe be some change to entrypoint.sh included as well?

Also, if you'd like feel free to tell us your interests in this project in case we have any similar or diverging development aims in the future?

@bendeckdavid
Copy link
Author

Hello @johan-fvh , thank you very much for your comments.
I have not made any changes to the entrypoint file, the reason for the netcat installation is that the entrypoint calls this library (which is not installed by default)

while ! nc -z $SQL_HOST $SQL_PORT; do sleep 0.1 done

I am a full stack developer, my languages of greatest knowledge are nodejs, vue, reactjs, java, ruby and php, but I am always interested in contributing to projects and increasing my knowledge. I am very interested in contributing as much as I can to open source projects, I would love to help you develop this project.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants