Share unique DNS query tool instead of rebuild.

Indeed, before this patch, we were constantly rebuilding a new DNS query
tool over and over before any HTTP request.
This patch fixes that by building one single DNS Query tool for the
lifetime of the requester object.

Once a requester object is initialized the DNS query tool it generates
will be shared accross all possible HTTP Adapters.

Author: funilrys

PyFunceble/query/requests/adapter/base.py

@@ -80,7 +80,11 @@ def __init__(self, *args, **kwargs):
                 total=kwargs["max_retries"], respect_retry_after_header=False
             )
 
-        self.dns_query_tool = DNSQueryTool().guess_all_settings()
+        if "dns_query_tool" in kwargs:
+            self.dns_query_tool = kwargs["dns_query_tool"]
+            del kwargs["dns_query_tool"]
+        else:
+            self.dns_query_tool = DNSQueryTool()
 
         super().__init__(*args, **kwargs)
 

PyFunceble/query/requests/requester.py

@@ -61,6 +61,7 @@
 import PyFunceble.facility
 import PyFunceble.storage
 from PyFunceble.dataset.user_agent import UserAgentDataset
+from PyFunceble.query.dns.query_tool import DNSQueryTool
 from PyFunceble.query.requests.adapter.http import RequestHTTPAdapter
 from PyFunceble.query.requests.adapter.https import RequestHTTPSAdapter
 
@@ -91,6 +92,7 @@ class Requester:
     _max_redirects: int = 60
 
     session: Optional[requests.Session] = None
+    dns_query_tool: Optional[DNSQueryTool] = None
 
     def __init__(
         self,
@@ -99,6 +101,7 @@ def __init__(
         verify_certificate: Optional[bool] = None,
         timeout: Optional[float] = None,
         max_redirects: Optional[int] = None,
+        dns_query_tool: Optional[DNSQueryTool] = None,
     ) -> None:
         if max_retries is not None:
             self.max_retries = max_retries
@@ -116,6 +119,11 @@ def __init__(
         if max_redirects is not None:
             self.max_redirects = max_redirects
 
+        if dns_query_tool is not None:
+            self.dns_query_tool = dns_query_tool
+        else:
+            self.dns_query_tool = DNSQueryTool()
+
         self.session = self.get_session()
 
         warnings.simplefilter("ignore", urllib3.exceptions.InsecureRequestWarning)
@@ -408,11 +416,19 @@ def get_session(self) -> requests.Session:
 
         session.mount(
             "https://",
-            RequestHTTPSAdapter(max_retries=self.max_retries, timeout=self.timeout),
+            RequestHTTPSAdapter(
+                max_retries=self.max_retries,
+                timeout=self.timeout,
+                dns_query_tool=self.dns_query_tool,
+            ),
         )
         session.mount(
             "http://",
-            RequestHTTPAdapter(max_retries=self.max_retries, timeout=self.timeout),
+            RequestHTTPAdapter(
+                max_retries=self.max_retries,
+                timeout=self.timeout,
+                dns_query_tool=self.dns_query_tool,
+            ),
         )
 
         custom_headers = {"User-Agent": UserAgentDataset().get_latest()}