status-im
/
consul
mirror of https://github.com/status-im/consul.git
2
0
Fork 0
Code Issues Projects Releases Wiki Activity

Merge pull request #10189 from hashicorp/dnephin/http-api-health-query-meta 

http: set consistency header properly for health endpoint
This commit is contained in:
Daniel Nephin 2021-05-06 14:05:12 -04:00 committed by hc-github-team-consul-core
parent dd586af4ef
commit cd55f89f84
2 changed files with 5 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
.changelog/10189.txt Normal file
View File

@ -0,0 +1,4 @@
```release-note:bug
http: fix a bug that caused the `X-Consul-Effective-Consistency` header to be missing on
request for service health
```

2
agent/health_endpoint.go
View File

@ -234,8 +234,8 @@ func (s *HTTPHandlers) healthServiceNodes(resp http.ResponseWriter, req *http.Re
if args.QueryOptions.UseCache { if args.QueryOptions.UseCache {
setCacheMeta(resp, &md) setCacheMeta(resp, &md)
} }
out.QueryMeta.ConsistencyLevel = args.QueryOptions.ConsistencyLevel()
setMeta(resp, &out.QueryMeta) setMeta(resp, &out.QueryMeta)
out.ConsistencyLevel = args.QueryOptions.ConsistencyLevel()
// FIXME: argument parsing should be done before performing the rpc // FIXME: argument parsing should be done before performing the rpc
// Filter to only passing if specified // Filter to only passing if specified