]> git.ipfire.org Git - thirdparty/bind9.git/commitdiff
test for growth of compressed pipelined responses
authorEvan Hunt <each@isc.org>
Wed, 17 Aug 2022 19:46:02 +0000 (12:46 -0700)
committerMichal Nowak <mnowak@isc.org>
Tue, 18 Oct 2022 15:16:00 +0000 (17:16 +0200)
add a test to compare the Content-Length of successive compressed
messages on a single HTTP connection that should contain the same
data; fail if the size grows by more than 100 bytes from one query
to the next.

bin/tests/system/statschannel/tests.sh

index ec899e1b3db0d688ae27f8d2abd66f32292c1340..6c741776c46c069333692f6ba443a246fa2793f3 100644 (file)
@@ -397,9 +397,9 @@ else
     echo_i "skipping test as nc not found"
 fi
 
+echo_i "Check HTTP/1.1 pipelined requests are handled (POST) ($n)"
+ret=0
 if [ -x "${NC}" ] ; then
-    echo_i "Check HTTP/1.1 pipelined requests are handled (POST) ($n)"
-    ret=0
     ${NC} 10.53.0.3 ${EXTRAPORT1} << EOF > nc.out$n || ret=1
 POST /xml/v3/status HTTP/1.1
 Host: 10.53.0.3:${EXTRAPORT1}
@@ -417,12 +417,12 @@ Connection: close
 EOF
     lines=$(grep -c "^HTTP/1.1" nc.out$n)
     test "$lines" = 2 || ret=1
-    if [ $ret != 0 ]; then echo_i "failed"; fi
-    status=$((status + ret))
-    n=$((n + 1))
 else
     echo_i "skipping test as nc not found"
 fi
+if [ $ret != 0 ]; then echo_i "failed"; fi
+status=$((status + ret))
+n=$((n + 1))
 
 echo_i "Check HTTP with more than 10 headers ($n)"
 ret=0
@@ -476,5 +476,28 @@ if [ $ret != 0 ]; then echo_i "failed"; fi
 status=$((status + ret))
 n=$((n + 1))
 
+echo_i "Check pipelined responses do not grow excessively ($n)"
+ret=0
+i=0
+if [ -x "${NC}" ] ; then
+    {
+        while test $i -lt 10; do
+            printf "GET /json/v1 HTTP/1.1\r\nHost: 10.53.0.3:%s\r\nAccept-Encoding: deflate, gzip, br, zstd\r\n\r\n" "${EXTRAPORT1}"
+            i=$((i + 1))
+        done
+    } | ${NC} 10.53.0.3 ${EXTRAPORT1} | grep -a Content-Length |
+        awk 'BEGIN { prev=0; }
+             { if (prev != 0 && $2 - prev > 100) {
+                   exit(1);
+               }
+               prev = $2;
+             }' || ret=1
+else
+    echo_i "skipping test as nc not found"
+fi
+if [ $ret != 0 ]; then echo_i "failed"; fi
+status=$((status + ret))
+n=$((n + 1))
+
 echo_i "exit status: $status"
 [ $status -eq 0 ] || exit 1