[master] af1a69838 vtc: Add http_max_hdr coverage to r3709

Dridi Boukelmoune dridi.boukelmoune at gmail.com
Thu Apr 4 14:29:12 UTC 2024


commit af1a698380da6c46c2e15d80327f00af91c011a5
Author: Dridi Boukelmoune <dridi.boukelmoune at gmail.com>
Date:   Thu Mar 28 16:22:59 2024 +0100

    vtc: Add http_max_hdr coverage to r3709
    
    For the sole purpose of having these limits tested in a single place.

diff --git a/bin/varnishtest/tests/r03709.vtc b/bin/varnishtest/tests/r03709.vtc
index 242afe2f1..65038fcce 100644
--- a/bin/varnishtest/tests/r03709.vtc
+++ b/bin/varnishtest/tests/r03709.vtc
@@ -3,6 +3,7 @@ varnishtest "h2 req limits"
 varnish v1 -cliok "param.set feature +http2"
 varnish v1 -cliok "param.set http_req_hdr_len 40b"
 varnish v1 -cliok "param.set http_req_size 512b"
+varnish v1 -cliok "param.set http_max_hdr 32"
 varnish v1 -vcl {
 	backend be none;
 } -start
@@ -10,6 +11,7 @@ varnish v1 -vcl {
 logexpect l1 -v v1 -g raw -q BogoHeader,LostHeader {
 	expect 0 1001 BogoHeader "Header too large: :path"
 	expect 0 1002 LostHeader "Header list too large"
+	expect 0 1003 LostHeader "Too many headers"
 } -start
 
 client c1 {
@@ -39,6 +41,41 @@ client c1 {
 		rxrst
 		expect rst.err == ENHANCE_YOUR_CALM
 	} -run
+
+	stream next {
+		txreq -url "/http_max_hdr" \
+			-hdr hdr1  val1  \
+			-hdr hdr2  val2  \
+			-hdr hdr3  val3  \
+			-hdr hdr4  val4  \
+			-hdr hdr4  val4  \
+			-hdr hdr5  val5  \
+			-hdr hdr6  val6  \
+			-hdr hdr7  val7  \
+			-hdr hdr8  val8  \
+			-hdr hdr9  val9  \
+			-hdr hdr10 val10 \
+			-hdr hdr11 val11 \
+			-hdr hdr11 val11 \
+			-hdr hdr11 val11 \
+			-hdr hdr12 val12 \
+			-hdr hdr13 val13 \
+			-hdr hdr14 val14 \
+			-hdr hdr15 val15 \
+			-hdr hdr16 val16 \
+			-hdr hdr17 val17 \
+			-hdr hdr18 val18 \
+			-hdr hdr19 val19 \
+			-hdr hdr20 val20 \
+			-hdr hdr20 val20 \
+			-hdr hdr21 val21 \
+			-hdr hdr22 val22 \
+			-hdr hdr23 val23 \
+			-hdr hdr24 val24 \
+			-hdr hdr25 val25
+		rxrst
+		expect rst.err == ENHANCE_YOUR_CALM
+	} -run
 } -run
 
 logexpect l1 -wait


More information about the varnish-commit mailing list