MINOR: checks: Add a way to send custom headers and payload during http chekcs
authorChristopher Faulet <cfaulet@haproxy.com>
Thu, 9 Apr 2020 06:44:06 +0000 (08:44 +0200)
committerWilly Tarreau <w@1wt.eu>
Fri, 1 May 2020 10:13:57 +0000 (12:13 +0200)
commit104d66a2caa12ba8eb528320e2e9b0136d754ccd
tree0963ecdeaea0a99afb8bd24af96fa78e47b02b19
parentaa60aae00ee0a21493f11a59bdaa0f040b507ab7
MINOR: checks: Add a way to send custom headers and payload during http chekcs

The 'http-check send' directive have been added to add headers and optionnaly a
payload to the request sent during HTTP healthchecks. The request line may be
customized by the "option httpchk" directive but there was not official way to
add extra headers. An old trick consisted to hide these headers at the end of
the version string, on the "option httpchk" line. And it was impossible to add
an extra payload with an "http-check expect" directive because of the
"Connection: close" header appended to the request (See issue #16 for details).

So to make things official and fully support payload additions, the "http-check
send" directive have been added :

    option httpchk POST /status HTTP/1.1

    http-check send hdr Content-Type "application/json;charset=UTF-8" \
        hdr X-test-1 value1 hdr X-test-2 value2 \
        body "{id: 1, field: \"value\"}"

When a payload is defined, the Content-Length header is automatically added. So
chunk-encoded requests are not supported yet. For now, there is no special
validity checks on the extra headers.

This patch is inspired by Kiran Gavali's work. It should fix the issue #16 and
as far as possible, it may be backported, at least as far as 1.8.

(cherry picked from commit 8acb1284bcd47301fce0aa45a7083dbcef94cb5f)
Signed-off-by: Willy Tarreau <w@1wt.eu>
doc/configuration.txt
include/types/proxy.h
reg-tests/checks/http-check-send.vtc [new file with mode: 0644]
src/cfgparse-listen.c
src/checks.c
src/haproxy.c