From: Artem Boldariev <artem@boldariev.com>
Date: Thu, 29 Jul 2021 07:46:34 +0000 (+0300)
Subject: When terminating a client session, mark it as closing
X-Git-Tag: v9.17.18~40^2~6
X-Git-Url: http://git.ipfire.org/gitweb.cgi?a=commitdiff_plain;h=5b52a7e37ed32e32fa671223b6d16e21f43436c4;p=thirdparty%2Fbind9.git

When terminating a client session, mark it as closing

When an HTTP/2 client terminates a session it means that it is about
to close the underlying connection. However, we were not doing that.
As a result, with the latest changes to the test suite, which made it
to limit amount of requests per a transport connection, the tests
using quota would hang for quite a while. This commit fixes that.
---

diff --git a/lib/isc/netmgr/http.c b/lib/isc/netmgr/http.c
index 11eefc165cf..b5247f58b9f 100644
--- a/lib/isc/netmgr/http.c
+++ b/lib/isc/netmgr/http.c
@@ -616,6 +616,9 @@ on_client_stream_close_callback(int32_t stream_id,
 			if (rv != 0) {
 				return (rv);
 			}
+			/* Mark the session as closing one to finish it on a
+			 * subsequent call to http_do_bio() */
+			session->closing = true;
 		}
 	} else {
 		return (NGHTTP2_ERR_CALLBACK_FAILURE);