Add support for healthchecks #145

Closed
opened 2017-05-24 11:19:05 +00:00 by SamMousa · 2 comments
SamMousa commented 2017-05-24 11:19:05 +00:00 (Migrated from github.com)

It would be nice to support a healthcheck URL that is configured in NGINX to always return a http 200.

It would be nice to support a healthcheck URL that is configured in NGINX to always return a http `200`.
richarvey commented 2017-08-18 14:06:46 +00:00 (Migrated from github.com)

Surely always returning a 200 negates the point of a healthcheck. Your application could be erroring and you would want that to fail the healthcheck. I think this would be better to include a health check in your code, rather than a generic 200 block.

Surely always returning a 200 negates the point of a healthcheck. Your application could be erroring and you would want that to fail the healthcheck. I think this would be better to include a health check in your code, rather than a generic 200 block.
richarvey commented 2017-08-25 10:45:52 +00:00 (Migrated from github.com)

Closing after after 7 days

Closing after after 7 days
Sign in to join this conversation.
No milestone
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference
ric/nginx-php-fpm#145
No description provided.