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.

Sign up for GitHub

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

Jump to bottom

Support HEAD request for http check #2473

Closed
kamaradclimber opened this issue Nov 4, 2016 · 1 comment
Closed

Support HEAD request for http check #2473

kamaradclimber opened this issue Nov 4, 2016 · 1 comment
Labels
theme/health-checks Health Check functionality type/enhancement Proposed improvement or new feature

Comments

@kamaradclimber
Copy link
Contributor

kamaradclimber commented Nov 4, 2016 •
edited
Loading

If you have a question, please direct it to the
consul mailing list if it hasn't been
addressed in either the FAQ or in one
of the Consul Guides.

When filing a bug, please include the following:

consul version for both Client and Server

Client: 0.7.0

Description of the Issue (and unexpected/desired result)

When using http health checks (https://www.consul.io/docs/agent/checks.html), many application have large variable output leading to #1244.
Most application we use (couchbase for instance) don't have a clean health endpoint so we use some statistics endpoint (/pools/default/buckets/ for instance).

It would be convenient to be able to configure the http health check to perform HEAD requests instead of GET.
@slackpad slackpad added the type/enhancement Proposed improvement or new feature label Nov 4, 2016
@kamaradclimber kamaradclimber mentioned this issue Nov 4, 2016
Closed
kamaradclimber added a commit to criteo-forks/consul that referenced this issue Dec 27, 2016
@kamaradclimber
Support HEAD request for http check
d5a492a 
This will allow to avoid http checks whose output churn a lot (statistics
endpoint for instance). Such checks are quite common when the checked
service does not provide a `/health` endpoint.

Fix: hashicorp#2473

Change-Id: Ice13ccf0ecb58be13e17a2843cdac7871e6579ac
kamaradclimber added a commit to criteo-forks/consul that referenced this issue Feb 3, 2017
@kamaradclimber
Support HEAD request for http check
4ed001d 
This will allow to avoid http checks whose output churn a lot (statistics
endpoint for instance). Such checks are quite common when the checked
service does not provide a `/health` endpoint.

Fix: hashicorp#2473

Change-Id: Ice13ccf0ecb58be13e17a2843cdac7871e6579ac
@slackpad slackpad added the theme/health-checks Health Check functionality label May 25, 2017
@slackpad
Copy link
Contributor

slackpad commented Jan 5, 2018

This was added by #3106.

@slackpad slackpad closed this as completed Jan 5, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
theme/health-checks Health Check functionality type/enhancement Proposed improvement or new feature
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@kamaradclimber @slackpad