Browse Source

reverseproxy: Fix incorrect `health_headers` Caddyfile parsing (#4485)

Fixes #4481
master
Francis Lavoie 3 years ago
committed by GitHub
parent
commit
5333c3528b
No known key found for this signature in database GPG Key ID: 4AEE18F83AFDEB23
  1. 4
      caddytest/integration/caddyfile_adapt/reverse_proxy_health_headers.txt
  2. 4
      modules/caddyhttp/reverseproxy/caddyfile.go

4
caddytest/integration/caddyfile_adapt/reverse_proxy_health_headers.txt

@ -7,6 +7,7 @@ reverse_proxy 127.0.0.1:65535 {
X-Header-Keys VbG4NZwWnipo 335Q9/MhqcNU3s2TO
X-Empty-Value
}
health_uri /health
}
----------
{
@ -38,7 +39,8 @@ reverse_proxy 127.0.0.1:65535 {
"VbG4NZwWnipo",
"335Q9/MhqcNU3s2TO"
]
}
},
"uri": "/health"
}
},
"upstreams": [

4
modules/caddyhttp/reverseproxy/caddyfile.go

@ -361,8 +361,7 @@ func (h *Handler) UnmarshalCaddyfile(d *caddyfile.Dispenser) error {
case "health_headers":
healthHeaders := make(http.Header)
for d.Next() {
for d.NextBlock(0) {
for nesting := d.Nesting(); d.NextBlock(nesting); {
key := d.Val()
values := d.RemainingArgs()
if len(values) == 0 {
@ -370,7 +369,6 @@ func (h *Handler) UnmarshalCaddyfile(d *caddyfile.Dispenser) error {
}
healthHeaders[key] = values
}
}
if h.HealthChecks == nil {
h.HealthChecks = new(HealthChecks)
}

Loading…
Cancel
Save