http: set consistency header properly
A recent change in 1.9.x inverted the order of these two lines, which caused the X-Consul-Effective-Consistency header to be missing for the servie health endpoints
This commit is contained in:
parent
5c7b0e6339
commit
72f52255ff
|
@ -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
|
||||||
|
```
|
|
@ -227,8 +227,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
|
||||||
|
|
Loading…
Reference in New Issue