Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

DOCKER HEALTHCHECK ISSUE 43 #85

Closed
wants to merge 3 commits into from
Closed

DOCKER HEALTHCHECK ISSUE 43 #85

wants to merge 3 commits into from

Conversation

Ahsanadam
Copy link
Contributor

No description provided.

@Ahsanadam Ahsanadam changed the title Issue 43 DOCKER HEALTHCHECK ISSUE 43 Feb 9, 2022
@ReymartNarvaez ReymartNarvaez self-assigned this Feb 9, 2022
@Vimbayinashe
Copy link
Contributor

This looks good so far! The only thing outstanding is the link below. I am happy to help and work together to create this.

To make the system better we can add an url to our server like http://localhost:8080/status which can be queried and return the status of the server as a simple 200 OK or some other error message when the server isn't running as intended.

@Vimbayinashe Vimbayinashe linked an issue Feb 10, 2022 that may be closed by this pull request
@ReymartNarvaez ReymartNarvaez linked an issue Feb 11, 2022 that may be closed by this pull request
@kappsegla kappsegla marked this pull request as draft February 11, 2022 12:44
@kappsegla
Copy link
Contributor

This PR is too old. Closing it.

@kappsegla kappsegla closed this Feb 16, 2024
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.

Docker HEALTHCHECK
5 participants