vtc: Add http_max_hdr coverage to r3709

For the sole purpose of having these limits tested in a single place.
varnishcache · Mar 29, 2024 · af1a698 · af1a698
1 parent 7ccffe7
commit af1a698
Showing 1 changed file with 37 additions and 0 deletions.
diff --git a/bin/varnishtest/tests/r03709.vtc b/bin/varnishtest/tests/r03709.vtc
@@ -3,13 +3,15 @@ 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
 
 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