]> git.ipfire.org Git - thirdparty/bind9.git/commitdiff
Adjust the resolver-query-timeout test
authorAram Sargsyan <aram@isc.org>
Fri, 20 Dec 2024 10:39:26 +0000 (10:39 +0000)
committerArаm Sаrgsyаn <aram@isc.org>
Wed, 22 Jan 2025 13:40:45 +0000 (13:40 +0000)
Since the read timeout now works, the resolver time outs from the
dispatch level instead of from the "hung fetch" timer, and so the
EDE value in 'fctx_expired()' is not being set. Remove the expected
EDE value from the test.

bin/tests/system/resolver/tests.sh

index ab94c17c10a87fb03b744244c21d2a94c53876f7..c2c2fd4d6f874fc8b75f786186748f39c8888033 100755 (executable)
@@ -53,16 +53,14 @@ grep -F "no servers could be reached" dig.out.ns1.test${n} >/dev/null || ret=1
 if [ $ret != 0 ]; then echo_i "failed"; fi
 status=$((status + ret))
 
-# 'resolver-query-timeout' is set to 5 seconds in ns1, which is lower than the
-# current single query timeout value MAX_SINGLE_QUERY_TIMEOUT of 9 seconds, so
-# the "hung fetch" timer should kick in, interrupt the non-responsive query and
-# send a SERVFAIL answer.
+# 'resolver-query-timeout' is set to 5 seconds in ns1, so named should
+# interrupt the non-responsive query and send a SERVFAIL answer before dig's
+# own timeout fires, which is set to 7 seconds.
 n=$((n + 1))
 echo_i "checking no response handling with a longer than resolver-query-timeout timeout ($n)"
 ret=0
 dig_with_opts +tcp +tries=1 +timeout=7 noresponse.example.net @10.53.0.1 a >dig.out.ns1.test${n} || ret=1
 grep -F "status: SERVFAIL" dig.out.ns1.test${n} >/dev/null || ret=1
-grep -F "EDE: 22 (No Reachable Authority)" dig.out.ns1.test${n} >/dev/null || ret=1
 if [ $ret != 0 ]; then echo_i "failed"; fi
 status=$((status + ret))