From 5110278008fdf2689fe37515da214e6faa0f29d7 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Micha=C5=82=20K=C4=99pie=C5=84?= Date: Fri, 30 May 2025 18:08:54 +0200 Subject: [PATCH] Drop unused AsyncDnsServer constructor argument The constructor for the AsyncDnsServer class takes a 'load_zones' argument that is not used anywhere and is not expected to be useful in the future: zone files are not required for an AsyncDnsServer instance to start and, if necessary, zone-based answers can be suppressed or modified by installing a custom response handler. --- bin/tests/system/isctest/asyncserver.py | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/bin/tests/system/isctest/asyncserver.py b/bin/tests/system/isctest/asyncserver.py index bcf26a41a1..522537b9d4 100644 --- a/bin/tests/system/isctest/asyncserver.py +++ b/bin/tests/system/isctest/asyncserver.py @@ -533,14 +533,13 @@ class AsyncDnsServer(AsyncServer): response from scratch, without using zone data at all. """ - def __init__(self, load_zones: bool = True): + def __init__(self): super().__init__(self._handle_udp, self._handle_tcp, "ans.pid") self._zone_tree: _ZoneTree = _ZoneTree() self._response_handlers: List[ResponseHandler] = [] - if load_zones: - self._load_zones() + self._load_zones() def install_response_handler( self, handler: ResponseHandler, prepend: bool = False