haproxy/reg-tests/checks/http-monitor-uri.vtc
Christopher Faulet 6072beb214 MINOR: http-ana: Match on the path if the monitor-uri starts by a /
if the monitor-uri starts by a slash ('/'), the matching is performed against
the request's path instead of the request's uri. It is a workaround to let the
HTTP/2 requests match the monitor-uri. Indeed, in HTTP/2, clients are encouraged
to send absolute URIs only.

This patch is not tagged as a bug, because the previous behavior matched exactly
what the doc describes. But it may surprise that HTTP/2 requests don't match the
monitor-uri.

This patch may be backported to 2.1 because URIs of HTTP/2 are stored using the
absolute-form starting this version. For previous versions, this patch will only
helps explicitely absolute HTTP/1 requests (and only the HTX part because on the
legacy HTTP, all the URI is matched).

It should fix the issue #509.
2020-02-18 16:29:29 +01:00

57 lines
1.3 KiB
Plaintext

varnishtest "Test the HTTP directive monitor-uri"
#REQUIRE_VERSION=2.2
# This config tests the HTTP directive monitor-uri. Especially the path matching
# when an absolute-form uri is received from the client. But also the
# case-sensitivity of the matching.
feature ignore_unknown_macro
haproxy h1 -conf {
defaults
mode http
timeout connect 1s
timeout client 1s
timeout server 1s
frontend fe1
bind "fd@${fe1}"
monitor-uri /health
frontend fe2
bind "fd@${fe2}"
monitor-uri http://www.haproxy.org/health
} -start
client c1 -connect ${h1_fe1_sock} {
txreq -req GET -url /health
rxresp
expect resp.status == 200
} -run
client c2 -connect ${h1_fe1_sock} {
txreq -req GET -url http://www.haproxy.org/health \
-hdr "Host: www.haproxy.org"
rxresp
expect resp.status == 200
} -run
client c3 -connect ${h1_fe1_sock} {
txreq -req GET -url /hEAlth
rxresp
expect resp.status == 503
} -run
client c4 -connect ${h1_fe2_sock} {
txreq -req GET -url http://www.haproxy.org/health \
-hdr "Host: www.haproxy.org"
rxresp
expect resp.status == 200
} -run
client c5 -connect ${h1_fe2_sock} {
txreq -req GET -url /health
rxresp
expect resp.status == 503
} -run