]> git.ipfire.org Git - thirdparty/haproxy.git/commitdiff
REGTESTS: adjust the http-rules regtest to support window updates
authorWilly Tarreau <w@1wt.eu>
Thu, 13 Dec 2018 16:29:20 +0000 (17:29 +0100)
committerWilly Tarreau <w@1wt.eu>
Thu, 13 Dec 2018 16:29:20 +0000 (17:29 +0100)
Varnishtest is not happy to see the window update come before the
settings ACK, as by default it expects exactly tx/rx/txack/rxack.
One workaround could consist in making haproxy send the WU after
the settings ACK but this would be a real hack as the preface is
already finished when sending this ack. Instead, let's make the
initial sequence explicit in the tests.

reg-tests/http-rules/h00001.vtc
reg-tests/http-rules/h00002.vtc

index ca86f1b909eddcc5d57d4374cda415d61cd25d9d..2056ea732a7d91d274d3121804a7e19ec81e1d5e 100644 (file)
@@ -187,6 +187,15 @@ client c1h1 -connect ${h1_feh1_sock} {
 } -run
 
 client c1h2 -connect ${h1_feh2_sock} {
+       txpri
+       stream 0 {
+               txsettings
+               rxsettings
+               txsettings -ack
+               rxwinup
+               rxsettings
+               expect settings.ack == true
+       } -run
        stream 1 {
                # warning: -req, -scheme, -url MUST be placed first otherwise
                # the H2 protocol is invalid since they are pseudo-headers
index b1a6b7b13fde9935a1bb8ec1a59789cd9a94430c..1f3c29d97758f8fdfb1c0181108751aa92ef259f 100644 (file)
@@ -188,6 +188,15 @@ client c1h1 -connect ${h1_feh1_sock} {
 } -run
 
 client c1h2 -connect ${h1_feh2_sock} {
+       txpri
+       stream 0 {
+               txsettings
+               rxsettings
+               txsettings -ack
+               rxwinup
+               rxsettings
+               expect settings.ack == true
+       } -run
        stream 1 {
                # warning: -req, -scheme, -url MUST be placed first otherwise
                # the H2 protocol is invalid since they are pseudo-headers