fmtr.tools 1.3.14__py3-none-any.whl → 1.3.16__py3-none-any.whl

fmtr/tools/dns_tools/client.py

@@ -3,6 +3,7 @@ from dataclasses import dataclass
 from dns import query
 from functools import cached_property
 from httpx_retries import Retry, RetryTransport
+from typing import Optional
 
 from fmtr.tools import http_tools as http
 from fmtr.tools.dns_tools.dm import Exchange, Response
@@ -38,14 +39,17 @@ class Plain:
     """
     host: str
     port: int = 53
+    ttl_min: Optional[int] = None
 
     def resolve(self, exchange: Exchange):
 
         with logger.span(f'UDP {self.host}:{self.port}'):
             response_plain = query.udp(q=exchange.query_last, where=self.host, port=self.port)
             response = Response.from_message(response_plain)
+            for answer in response.message.answer:
+                answer.ttl = max(answer.ttl, self.ttl_min or answer.ttl)
 
-        exchange.response.message.answer += response.message.answer
+        exchange.response = response
 
 
 @dataclass
@@ -86,7 +90,7 @@ class HTTP:
             response_doh = self.CLIENT.post(url, headers=headers, content=exchange.query_last.to_wire())
             response_doh.raise_for_status()
             response = Response.from_http(response_doh)
-            exchange.response.message.answer += response.message.answer
+            exchange.response = response
 
         except Exception as exception:
             exchange.response.message.set_rcode(dnspython.rcode.SERVFAIL)

fmtr/tools/dns_tools/dm.py

@@ -1,15 +1,28 @@
 import dns
 import httpx
-from dataclasses import dataclass
+from dataclasses import dataclass, field
 from dns import rcode as dnsrcode
 from dns import reversename
 from dns.message import Message, QueryMessage
 from dns.rrset import RRset
 from functools import cached_property
-from typing import Self, Optional
+from typing import Self, Optional, List
 
 from fmtr.tools.string_tools import join
 
+TTL_CODE_DEFAULTS = {
+    dnsrcode.NOERROR: 300,  # Successful query
+    dnsrcode.FORMERR: 60,  # Format error
+    dnsrcode.SERVFAIL: 10,  # Server failure
+    dnsrcode.NXDOMAIN: 60 * 60,  # Non-existent domain
+    dnsrcode.NOTIMP: 60,  # Not implemented
+    dnsrcode.REFUSED: 60,  # Refused
+    dnsrcode.YXDOMAIN: 600,  # Name exists when it should not
+    dnsrcode.YXRRSET: 600,  # RR Set exists when it should not
+    dnsrcode.NXRRSET: 300,  # RR Set that should exist does not
+    dnsrcode.NOTAUTH: 60,  # Not authorized
+    dnsrcode.NOTZONE: 60  # Name not contained in zone
+}
 
 @dataclass
 class BaseDNSData:
@@ -37,7 +50,6 @@ class Response(BaseDNSData):
     """
 
     http: Optional[httpx.Response] = None
-    is_complete: bool = False
     blocked_by: Optional[str] = None
 
     @classmethod
@@ -69,6 +81,24 @@ class Response(BaseDNSData):
     def rcode_text(self) -> str:
         return dnsrcode.to_text(self.rcode)
 
+    @property
+    def ttl(self) -> int:
+        """
+
+        Get median TTL from answers, falling back to authority, then to error-code defaults.
+
+        """
+        answers = self.message.answer or self.message.authority
+        if answers:
+            ttls = [answer.ttl for answer in answers]
+            ttl = min(ttls)
+            return ttl
+
+        ttl = TTL_CODE_DEFAULTS.get(self.rcode, dnsrcode.NXDOMAIN)
+        return ttl
+
+
+
     def __str__(self):
         """
 
@@ -144,9 +174,10 @@ class Exchange:
 
     request: Request
     response: Optional[Response] = None
+    answers_pre: List[RRset] = field(default_factory=list)
     is_internal: bool = False
     client_name: Optional[str] = None
-
+    is_complete: bool = False
 
 
     @classmethod
@@ -163,27 +194,25 @@ class Exchange:
     def question_last(self) -> RRset:
         """
 
-        Contrive an RRset representing the latest/current question. This can be the original question - or a hybrid one if we've injected our own answers into the exchange.
+        Create an RRset surrogate representing the latest/current question. This can be the original question - or a hybrid one if we've injected our own answers into the Exchange.
 
         """
-        if self.response.answer:
-            rrset = self.response.answer
-            ty = self.request.type
+        if self.answers_pre:
+            rrset = self.answers_pre[-1]
+            rdtype = self.request.type
             ttl = self.request.question.ttl
             rdclass = self.request.question.rdclass
             name = next(iter(rrset.items.keys())).to_text()
-
-            rrset_contrived = dns.rrset.from_text(
+            rrset_surrogate = dns.rrset.from_text(
                 name=name,
                 ttl=ttl,
-                rdtype=ty,
+                rdtype=rdtype,
                 rdclass=rdclass,
-
             )
 
-            return rrset_contrived
+            return rrset_surrogate
         else:
-            return self.request.question  # Solves the issue of digging out the name.
+            return self.request.question
 
     @property
     def query_last(self) -> QueryMessage:
@@ -194,7 +223,7 @@ class Exchange:
         """
 
         question_last = self.question_last
-        query = dns.message.make_query(qname=question_last.name, rdclass=question_last.rdclass, rdtype=question_last.rdtype)
+        query = dns.message.make_query(qname=question_last.name, rdclass=question_last.rdclass, rdtype=question_last.rdtype, id=self.request.message.id)
         return query
 
     @property

fmtr/tools/dns_tools/proxy.py

@@ -31,6 +31,14 @@ class Proxy(server.Plain):
         """
         return
 
+    def finalize(self, exchange: Exchange):
+        """
+
+        Finalize a still open exchange.
+
+        """
+        exchange.is_complete = True
+
     def resolve(self, exchange: Exchange) -> Exchange:
         """
 
@@ -38,20 +46,21 @@ class Proxy(server.Plain):
         Subclasses can override the relevant processing methods to implement custom behaviour.
 
         """
-
         with logger.span(f'Processing question...'):
             self.process_question(exchange)
-        if exchange.response.is_complete:
+        if exchange.is_complete:
             return exchange
 
         with logger.span(f'Making upstream request...'):
             self.client.resolve(exchange)
-        if exchange.response.is_complete:
+        if exchange.is_complete:
             return exchange
 
         with logger.span(f'Processing upstream response...'):
             self.process_upstream(exchange)
-        if exchange.response.is_complete:
+        if exchange.is_complete:
             return exchange
 
+        self.finalize(exchange)
+
         return exchange

fmtr/tools/dns_tools/server.py

@@ -1,3 +1,4 @@
+import dns
 import socket
 from dataclasses import dataclass
 from datetime import timedelta
@@ -66,7 +67,7 @@ class Plain:
             logger.info(f'Request found in cache.')
             exchange.response = self.cache[exchange.key]
             exchange.response.message.id = exchange.request.message.id
-            exchange.response.is_complete = True
+            exchange.is_complete = True
 
     def get_span(self, exchange: Exchange):
         """
@@ -90,10 +91,20 @@ class Plain:
         response = exchange.response
 
         logger.info(
-            f'Resolution complete {exchange.client_name=} {request.message.id=} {request.type_text} {request.name_text} {request.question=} {response.is_complete=} {response.rcode=} {response.rcode_text=} {response.answer=} {response.blocked_by=}...'
+            f'Resolution complete {exchange.client_name=} {request.message.id=} {request.type_text} {request.name_text} {request.question=} {exchange.is_complete=} {response.rcode=} {response.rcode_text=} {response.answer=} {response.blocked_by=}...'
         )
 
+    def log_dns_errors(self, exchange: Exchange):
+        """
+
+        Warn about any errors
+
+        """
+
+        if exchange.response.rcode == dns.rcode.NOERROR:
+            return
 
+        logger.warning(f'Error {exchange.response.rcode_text=}')
 
     def handle(self, exchange: Exchange):
         """
@@ -116,11 +127,11 @@ class Plain:
             with logger.span(f'Checking cache...'):
                 self.check_cache(exchange)
 
-            if not exchange.response.is_complete:
+            if not exchange.is_complete:
                 exchange = self.resolve(exchange)
-                exchange.response.is_complete = True
+                self.cache[exchange.key] = exchange.response
 
-            self.cache[exchange.key] = exchange.response
+            self.log_dns_errors(exchange)
             self.log_response(exchange)
 
         return exchange

fmtr/tools/version

@@ -1 +1 @@
-1.3.14
+1.3.16

{fmtr_tools-1.3.14.dist-info → fmtr_tools-1.3.16.dist-info}/METADATA

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: fmtr.tools
-Version: 1.3.14
+Version: 1.3.16
 Summary: Collection of high-level tools to simplify everyday development tasks, with a focus on AI/ML
 Home-page: https://github.com/fmtr/fmtr.tools
 Author: Frontmatter
@@ -126,61 +126,61 @@ Requires-Dist: logfire[httpx]; extra == "http"
 Provides-Extra: setup
 Requires-Dist: setuptools; extra == "setup"
 Provides-Extra: all
-Requires-Dist: sentence_transformers; extra == "all"
-Requires-Dist: Unidecode; extra == "all"
-Requires-Dist: peft; extra == "all"
-Requires-Dist: tinynetrc; extra == "all"
-Requires-Dist: contexttimer; extra == "all"
+Requires-Dist: pydantic-ai[logfire,openai]; extra == "all"
 Requires-Dist: torchaudio; extra == "all"
-Requires-Dist: uvicorn[standard]; extra == "all"
-Requires-Dist: openpyxl; extra == "all"
-Requires-Dist: pydantic-settings; extra == "all"
-Requires-Dist: sre_yield; extra == "all"
-Requires-Dist: distributed; extra == "all"
-Requires-Dist: semver; extra == "all"
-Requires-Dist: tokenizers; extra == "all"
+Requires-Dist: flet-webview; extra == "all"
 Requires-Dist: pydevd-pycharm~=251.25410.159; extra == "all"
-Requires-Dist: fastapi; extra == "all"
-Requires-Dist: pydantic-ai[logfire,openai]; extra == "all"
-Requires-Dist: httpx_retries; extra == "all"
-Requires-Dist: flet[all]; extra == "all"
-Requires-Dist: pymupdf4llm; extra == "all"
+Requires-Dist: appdirs; extra == "all"
+Requires-Dist: transformers[sentencepiece]; extra == "all"
+Requires-Dist: openai; extra == "all"
+Requires-Dist: google-auth-httplib2; extra == "all"
+Requires-Dist: docker; extra == "all"
+Requires-Dist: faker; extra == "all"
+Requires-Dist: torchvision; extra == "all"
+Requires-Dist: flet-video; extra == "all"
+Requires-Dist: pandas; extra == "all"
 Requires-Dist: filetype; extra == "all"
-Requires-Dist: pytest-cov; extra == "all"
-Requires-Dist: flet-webview; extra == "all"
+Requires-Dist: tokenizers; extra == "all"
+Requires-Dist: logfire[fastapi]; extra == "all"
+Requires-Dist: google-auth-oauthlib; extra == "all"
+Requires-Dist: pymupdf4llm; extra == "all"
+Requires-Dist: openpyxl; extra == "all"
+Requires-Dist: fastapi; extra == "all"
+Requires-Dist: uvicorn[standard]; extra == "all"
+Requires-Dist: peft; extra == "all"
 Requires-Dist: pymupdf; extra == "all"
+Requires-Dist: bokeh; extra == "all"
+Requires-Dist: httpx_retries; extra == "all"
 Requires-Dist: json_repair; extra == "all"
-Requires-Dist: openai; extra == "all"
-Requires-Dist: google-auth; extra == "all"
-Requires-Dist: huggingface_hub; extra == "all"
 Requires-Dist: ollama; extra == "all"
-Requires-Dist: faker; extra == "all"
+Requires-Dist: html2text; extra == "all"
 Requires-Dist: dnspython[doh]; extra == "all"
-Requires-Dist: logfire[fastapi]; extra == "all"
-Requires-Dist: httpx; extra == "all"
+Requires-Dist: pytest-cov; extra == "all"
+Requires-Dist: logfire; extra == "all"
 Requires-Dist: google-api-python-client; extra == "all"
-Requires-Dist: docker; extra == "all"
-Requires-Dist: google-auth-oauthlib; extra == "all"
-Requires-Dist: torchvision; extra == "all"
-Requires-Dist: regex; extra == "all"
-Requires-Dist: dask[bag]; extra == "all"
-Requires-Dist: appdirs; extra == "all"
-Requires-Dist: html2text; extra == "all"
-Requires-Dist: deepmerge; extra == "all"
 Requires-Dist: logfire[httpx]; extra == "all"
-Requires-Dist: pandas; extra == "all"
-Requires-Dist: flet-video; extra == "all"
+Requires-Dist: deepmerge; extra == "all"
+Requires-Dist: google-auth; extra == "all"
+Requires-Dist: sentence_transformers; extra == "all"
+Requires-Dist: pyyaml; extra == "all"
+Requires-Dist: sre_yield; extra == "all"
+Requires-Dist: distributed; extra == "all"
+Requires-Dist: dask[bag]; extra == "all"
+Requires-Dist: huggingface_hub; extra == "all"
+Requires-Dist: pydantic; extra == "all"
+Requires-Dist: regex; extra == "all"
+Requires-Dist: pydantic-settings; extra == "all"
+Requires-Dist: contexttimer; extra == "all"
+Requires-Dist: flet[all]; extra == "all"
+Requires-Dist: semver; extra == "all"
+Requires-Dist: setuptools; extra == "all"
+Requires-Dist: tinynetrc; extra == "all"
 Requires-Dist: tabulate; extra == "all"
-Requires-Dist: google-auth-httplib2; extra == "all"
-Requires-Dist: bokeh; extra == "all"
-Requires-Dist: logfire; extra == "all"
+Requires-Dist: diskcache; extra == "all"
 Requires-Dist: cachetools; extra == "all"
-Requires-Dist: setuptools; extra == "all"
+Requires-Dist: httpx; extra == "all"
+Requires-Dist: Unidecode; extra == "all"
 Requires-Dist: yamlscript; extra == "all"
-Requires-Dist: pydantic; extra == "all"
-Requires-Dist: transformers[sentencepiece]; extra == "all"
-Requires-Dist: pyyaml; extra == "all"
-Requires-Dist: diskcache; extra == "all"
 Dynamic: author
 Dynamic: author-email
 Dynamic: description

{fmtr_tools-1.3.14.dist-info → fmtr_tools-1.3.16.dist-info}/RECORD

@@ -44,16 +44,16 @@ fmtr/tools/tabular_tools.py,sha256=tpIpZzYku1HcJrHZJL6BC39LmN3WUWVhFbK2N7nDVmE,1
 fmtr/tools/tokenization_tools.py,sha256=me-IBzSLyNYejLybwjO9CNB6Mj2NYfKPaOVThXyaGNg,4268
 fmtr/tools/tools.py,sha256=CAsApa1YwVdNE6H66Vjivs_mXYvOas3rh7fPELAnTpk,795
 fmtr/tools/unicode_tools.py,sha256=yS_9wpu8ogNoiIL7s1G_8bETFFO_YQlo4LNPv1NLDeY,52
-fmtr/tools/version,sha256=GG4QPlY_pdWNhfbn575pBPL9Qt2zJ_Kt1m89Ii4ieks,6
+fmtr/tools/version,sha256=BGvCUIvG-zhY8XPNIcDdYZpZkRSi1ad9zjGx1qmjmTU,6
 fmtr/tools/yaml_tools.py,sha256=Bhhyd6GQVKO72Lp8ky7bAUjIB_65Hdh0Q45SKIEe6S8,1901
 fmtr/tools/ai_tools/__init__.py,sha256=JZrLuOFNV1A3wvJgonxOgz_4WS-7MfCuowGWA5uYCjs,372
 fmtr/tools/ai_tools/agentic_tools.py,sha256=acSEPFS-aguDXanWGs3fAAlRyJSYPZW7L-Kb2qDLm-I,4300
 fmtr/tools/ai_tools/inference_tools.py,sha256=2UP2gXEyOJUjyyV6zmFIYmIxUsh1rXkRH0IbFvr2bRs,11908
 fmtr/tools/dns_tools/__init__.py,sha256=PjD3Og6D5yvDVpKmsUsrnSpz_rjXpl4zBtvMqm8xKWU,237
-fmtr/tools/dns_tools/client.py,sha256=zAPJbQZIuNJPTA-HDBtrVZHvHRP_7QYWLgH7D73g5LU,2598
-fmtr/tools/dns_tools/dm.py,sha256=_G3ELId69SDMm0YciWQCW9-cTwt1l6cMVCt5-3T9V58,5177
-fmtr/tools/dns_tools/proxy.py,sha256=b3TdSwRO7IwcNjrWg1e8jVQb-YxJhT377rdVkeDc8_I,1466
-fmtr/tools/dns_tools/server.py,sha256=yL76rbYesdr1kHqLis5q1bxc3nzq68Utx-_PWA3Vu9U,3656
+fmtr/tools/dns_tools/client.py,sha256=uUFoFRwoPtetPVH6PZ3ssHmx3WEquT6UW4FCBKq4n74,2722
+fmtr/tools/dns_tools/dm.py,sha256=KKIcxJ5Dni2PFtOEHNNSi1Y4NCZ0bp3tifpSZRXtMTM,6261
+fmtr/tools/dns_tools/proxy.py,sha256=0lgn1pq5KLoGA4644ZXYs2lPjXjRB-ibFb7JHzlMY9o,1618
+fmtr/tools/dns_tools/server.py,sha256=c2ZxhF3EUgYLl1fnBEfLbM_URP3t7GdXeYOc6PQEmd0,3890
 fmtr/tools/entrypoints/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
 fmtr/tools/entrypoints/cache_hfh.py,sha256=fQNs4J9twQuZH_Yj98-oOvEX7-LrSUP3kO8nzw2HrHs,60
 fmtr/tools/entrypoints/ep_test.py,sha256=B8HfWISfSgw_xVX475CbJGh_QnpOe9MH65H8qGjTWbY,46
@@ -76,9 +76,9 @@ fmtr/tools/tests/test_path.py,sha256=AkZQa6_8BQ-VaCyL_J-iKmdf2ZaM-xFYR37Kun3k4_g
 fmtr/tools/tests/test_yaml.py,sha256=jc0TwwKu9eC0LvFGNMERdgBue591xwLxYXFbtsRwXVM,287
 fmtr/tools/version_tools/__init__.py,sha256=pg4iLtmIr5HtyEW_j0fMFoIdzqi_w9xH8-grQaXLB28,318
 fmtr/tools/version_tools/version_tools.py,sha256=Hcc6yferZS1hHbugRTdiHhSNmXEEG0hjCiTTXKna-YY,1127
-fmtr_tools-1.3.14.dist-info/licenses/LICENSE,sha256=FW9aa6vVN5IjRQWLT43hs4_koYSmpcbIovlKeAJ0_cI,10757
-fmtr_tools-1.3.14.dist-info/METADATA,sha256=Yj9yReion57e01vzQDF85BNGZVVLXHOUxfv8KQWYj1o,15938
-fmtr_tools-1.3.14.dist-info/WHEEL,sha256=_zCd3N1l69ArxyTb8rzEoP9TpbYXkqRFSNOD5OuxnTs,91
-fmtr_tools-1.3.14.dist-info/entry_points.txt,sha256=h-r__Xh5njtFqreMLg6cGuTFS4Qh-QqJPU1HB-_BS-Q,357
-fmtr_tools-1.3.14.dist-info/top_level.txt,sha256=LXem9xCgNOD72tE2gRKESdiQTL902mfFkwWb6-dlwEE,5
-fmtr_tools-1.3.14.dist-info/RECORD,,
+fmtr_tools-1.3.16.dist-info/licenses/LICENSE,sha256=FW9aa6vVN5IjRQWLT43hs4_koYSmpcbIovlKeAJ0_cI,10757
+fmtr_tools-1.3.16.dist-info/METADATA,sha256=MunCjEPB35a78qpSMM10MHjcuJjl3gcGHjC3JvVwA4I,15938
+fmtr_tools-1.3.16.dist-info/WHEEL,sha256=_zCd3N1l69ArxyTb8rzEoP9TpbYXkqRFSNOD5OuxnTs,91
+fmtr_tools-1.3.16.dist-info/entry_points.txt,sha256=h-r__Xh5njtFqreMLg6cGuTFS4Qh-QqJPU1HB-_BS-Q,357
+fmtr_tools-1.3.16.dist-info/top_level.txt,sha256=LXem9xCgNOD72tE2gRKESdiQTL902mfFkwWb6-dlwEE,5
+fmtr_tools-1.3.16.dist-info/RECORD,,