]> git.ipfire.org Git - thirdparty/dnspython.git/commitdiff
automated format to f-string conversion + black
authorBob Halley <halley@dnspython.org>
Wed, 24 Jul 2024 01:45:32 +0000 (18:45 -0700)
committerBob Halley <halley@dnspython.org>
Wed, 24 Jul 2024 01:45:32 +0000 (18:45 -0700)
19 files changed:
dns/asyncquery.py
dns/edns.py
dns/ipv6.py
dns/query.py
dns/rdata.py
dns/rdataset.py
dns/rdtypes/ANY/GPOS.py
dns/rdtypes/ANY/HINFO.py
dns/rdtypes/ANY/ISDN.py
dns/rdtypes/ANY/LOC.py
dns/rdtypes/ANY/NSEC.py
dns/rdtypes/ANY/RP.py
dns/rdtypes/ANY/WALLET.py
dns/rdtypes/txtbase.py
dns/resolver.py
dns/zonefile.py
pyproject.toml
tests/test_async.py
tests/test_doh.py

index f7d4df44cf4cdf19a314eb282b586f256a649603..717f43b4707daf0c5247714452443789841156ee 100644 (file)
@@ -554,13 +554,15 @@ async def https(
         af = None
     if af is not None and dns.inet.is_address(where):
         if af == socket.AF_INET:
-            url = "https://{}:{}{}".format(where, port, path)
+            url = f"https://{where}:{port}{path}"
         elif af == socket.AF_INET6:
-            url = "https://[{}]:{}{}".format(where, port, path)
+            url = f"https://[{where}]:{port}{path}"
     else:
         url = where
 
-    if http_version == HTTPVersion.H3 or (http_version == HTTPVersion.DEFAULT and not have_doh):
+    if http_version == HTTPVersion.H3 or (
+        http_version == HTTPVersion.DEFAULT and not have_doh
+    ):
         if bootstrap_address is None:
             parsed = urllib.parse.urlparse(url)
             resolver = _maybe_get_resolver(resolver)
@@ -621,9 +623,7 @@ async def https(
     if client:
         cm: contextlib.AbstractAsyncContextManager = NullContext(client)
     else:
-        cm = httpx.AsyncClient(
-            http1=h1, http2=h2, verify=verify, transport=transport
-        )
+        cm = httpx.AsyncClient(http1=h1, http2=h2, verify=verify, transport=transport)
 
     async with cm as the_client:
         # see https://tools.ietf.org/html/rfc8484#section-4.1.1 for DoH
@@ -649,10 +649,8 @@ async def https(
     # status codes
     if response.status_code < 200 or response.status_code > 299:
         raise ValueError(
-            "{} responded with status code {}"
-            "\nResponse body: {!r}".format(
-                where, response.status_code, response.content
-            )
+            f"{where} responded with status code {response.status_code}"
+            f"\nResponse body: {response.content!r}"
         )
     r = dns.message.from_wire(
         response.content,
index af90827a514a0c90531f859ea70d3a026911de2e..cbb7c443f77c783d7dd200a89117d8ecb727da9c 100644 (file)
@@ -222,7 +222,7 @@ class ECSOption(Option):  # lgtm[py/missing-equals]
             self.addrdata = self.addrdata[:-1] + last
 
     def to_text(self) -> str:
-        return "ECS {}/{} scope/{}".format(self.address, self.srclen, self.scopelen)
+        return f"ECS {self.address}/{self.srclen} scope/{self.scopelen}"
 
     @staticmethod
     def from_text(text: str) -> Option:
@@ -255,10 +255,10 @@ class ECSOption(Option):  # lgtm[py/missing-equals]
             ecs_text = tokens[0]
         elif len(tokens) == 2:
             if tokens[0] != optional_prefix:
-                raise ValueError('could not parse ECS from "{}"'.format(text))
+                raise ValueError(f'could not parse ECS from "{text}"')
             ecs_text = tokens[1]
         else:
-            raise ValueError('could not parse ECS from "{}"'.format(text))
+            raise ValueError(f'could not parse ECS from "{text}"')
         n_slashes = ecs_text.count("/")
         if n_slashes == 1:
             address, tsrclen = ecs_text.split("/")
@@ -266,18 +266,16 @@ class ECSOption(Option):  # lgtm[py/missing-equals]
         elif n_slashes == 2:
             address, tsrclen, tscope = ecs_text.split("/")
         else:
-            raise ValueError('could not parse ECS from "{}"'.format(text))
+            raise ValueError(f'could not parse ECS from "{text}"')
         try:
             scope = int(tscope)
         except ValueError:
-            raise ValueError(
-                "invalid scope " + '"{}": scope must be an integer'.format(tscope)
-            )
+            raise ValueError("invalid scope " + f'"{tscope}": scope must be an integer')
         try:
             srclen = int(tsrclen)
         except ValueError:
             raise ValueError(
-                "invalid srclen " + '"{}": srclen must be an integer'.format(tsrclen)
+                "invalid srclen " + f'"{tsrclen}": srclen must be an integer'
             )
         return ECSOption(address, srclen, scope)
 
index 44a1063936891d417cc8e354acd32dc5b9f70eb1..4dd1d1cade2ddc126234825a75eef66a7c574896 100644 (file)
@@ -143,9 +143,7 @@ def inet_aton(text: Union[str, bytes], ignore_scope: bool = False) -> bytes:
     if m is not None:
         b = dns.ipv4.inet_aton(m.group(2))
         btext = (
-            "{}:{:02x}{:02x}:{:02x}{:02x}".format(
-                m.group(1).decode(), b[0], b[1], b[2], b[3]
-            )
+            f"{m.group(1).decode()}:{b[0]:02x}{b[1]:02x}:{b[2]:02x}{b[3]:02x}"
         ).encode()
     #
     # Try to turn '::<whatever>' into ':<whatever>'; if no match try to
index 8f8f97cdc15bdd6f0823473584e9b11ca3bc4fcb..050aeca44ae370060ef61103149ca325e22f1ded 100644 (file)
@@ -443,13 +443,15 @@ def https(
     )
     if af is not None and dns.inet.is_address(where):
         if af == socket.AF_INET:
-            url = "https://{}:{}{}".format(where, port, path)
+            url = f"https://{where}:{port}{path}"
         elif af == socket.AF_INET6:
-            url = "https://[{}]:{}{}".format(where, port, path)
+            url = f"https://[{where}]:{port}{path}"
     else:
         url = where
 
-    if http_version == HTTPVersion.H3 or (http_version == HTTPVersion.DEFAULT and not have_doh):
+    if http_version == HTTPVersion.H3 or (
+        http_version == HTTPVersion.DEFAULT and not have_doh
+    ):
         if bootstrap_address is None:
             parsed = urllib.parse.urlparse(url)
             resolver = _maybe_get_resolver(resolver)
@@ -533,8 +535,8 @@ def https(
     # status codes
     if response.status_code < 200 or response.status_code > 299:
         raise ValueError(
-            "{} responded with status code {}"
-            "\nResponse body: {}".format(where, response.status_code, response.content)
+            f"{where} responded with status code {response.status_code}"
+            f"\nResponse body: {response.content}"
         )
     r = dns.message.from_wire(
         response.content,
index 1b4a6a76a3a4eaab2762f712b719c05aba2fbc70..5f9d47134ce6906ccf2b690447c61c9c9dcd81aa 100644 (file)
@@ -434,15 +434,11 @@ class Rdata:
                 continue
             if key not in parameters:
                 raise AttributeError(
-                    "'{}' object has no attribute '{}'".format(
-                        self.__class__.__name__, key
-                    )
+                    f"'{self.__class__.__name__}' object has no attribute '{key}'"
                 )
             if key in ("rdclass", "rdtype"):
                 raise AttributeError(
-                    "Cannot overwrite '{}' attribute '{}'".format(
-                        self.__class__.__name__, key
-                    )
+                    f"Cannot overwrite '{self.__class__.__name__}' attribute '{key}'"
                 )
 
         # Construct the parameter list.  For each field, use the value in
index 8bff58d7a5afde491accad2e9ce6d04e86febc6f..b5bd40a3043507d3527b1b9b9e3dd6997d03a64d 100644 (file)
@@ -248,12 +248,7 @@ class Rdataset(dns.set.Set):
             # (which is meaningless anyway).
             #
             s.write(
-                "{}{}{} {}\n".format(
-                    ntext,
-                    pad,
-                    dns.rdataclass.to_text(rdclass),
-                    dns.rdatatype.to_text(self.rdtype),
-                )
+                f"{ntext}{pad}{dns.rdataclass.to_text(rdclass)} {dns.rdatatype.to_text(self.rdtype)}\n"
             )
         else:
             for rd in self:
index 312338f9d1ff98cff633a2d33fbdddf358fa20bf..0aa33798a53400f38ea100abe84caefd230be119 100644 (file)
@@ -75,9 +75,7 @@ class GPOS(dns.rdata.Rdata):
             raise dns.exception.FormError("bad longitude")
 
     def to_text(self, origin=None, relativize=True, **kw):
-        return "{} {} {}".format(
-            self.latitude.decode(), self.longitude.decode(), self.altitude.decode()
-        )
+        return f"{self.latitude.decode()} {self.longitude.decode()} {self.altitude.decode()}"
 
     @classmethod
     def from_text(
index c2c45de01d220808d529bbaf0bbfe973090aff8f..06ad3487cf8c37a48f4ab53240986287798b9e79 100644 (file)
@@ -37,9 +37,7 @@ class HINFO(dns.rdata.Rdata):
         self.os = self._as_bytes(os, True, 255)
 
     def to_text(self, origin=None, relativize=True, **kw):
-        return '"{}" "{}"'.format(
-            dns.rdata._escapify(self.cpu), dns.rdata._escapify(self.os)
-        )
+        return f'"{dns.rdata._escapify(self.cpu)}" "{dns.rdata._escapify(self.os)}"'
 
     @classmethod
     def from_text(
index fb01eab3bbdee80abf6faa2bda2775f5ee2e4cbf..d2c44ed60e722cc81e28db4ba7df25dad1ff3821 100644 (file)
@@ -38,9 +38,7 @@ class ISDN(dns.rdata.Rdata):
 
     def to_text(self, origin=None, relativize=True, **kw):
         if self.subaddress:
-            return '"{}" "{}"'.format(
-                dns.rdata._escapify(self.address), dns.rdata._escapify(self.subaddress)
-            )
+            return f'"{dns.rdata._escapify(self.address)}" "{dns.rdata._escapify(self.subaddress)}"'
         else:
             return '"%s"' % dns.rdata._escapify(self.address)
 
index a36a2c109c4b18508f35b4cf5305b3536acaadb9..ced5dc0360edc4cb0461d47c4ed7ad4b0fd2b012 100644 (file)
@@ -184,11 +184,7 @@ class LOC(dns.rdata.Rdata):
             or self.horizontal_precision != _default_hprec
             or self.vertical_precision != _default_vprec
         ):
-            text += " {:0.2f}m {:0.2f}m {:0.2f}m".format(
-                self.size / 100.0,
-                self.horizontal_precision / 100.0,
-                self.vertical_precision / 100.0,
-            )
+            text += f" {self.size / 100.0:0.2f}m {self.horizontal_precision / 100.0:0.2f}m {self.vertical_precision / 100.0:0.2f}m"
         return text
 
     @classmethod
index 340525a6ebd2ac5075887c8c8f5fd2b5f3b69be6..3c78b72288b75a65b7fe3eaaaf7177d296f33b5d 100644 (file)
@@ -44,7 +44,7 @@ class NSEC(dns.rdata.Rdata):
     def to_text(self, origin=None, relativize=True, **kw):
         next = self.next.choose_relativity(origin, relativize)
         text = Bitmap(self.windows).to_text()
-        return "{}{}".format(next, text)
+        return f"{next}{text}"
 
     @classmethod
     def from_text(
index 9b74549db90e97a990070423cb1b1554fb24b28f..a66cfc50f5e9854f556a727544ee0114f887b45d 100644 (file)
@@ -37,7 +37,7 @@ class RP(dns.rdata.Rdata):
     def to_text(self, origin=None, relativize=True, **kw):
         mbox = self.mbox.choose_relativity(origin, relativize)
         txt = self.txt.choose_relativity(origin, relativize)
-        return "{} {}".format(str(mbox), str(txt))
+        return f"{str(mbox)} {str(txt)}"
 
     @classmethod
     def from_text(
index 4781ebc44e30e13e8a0ef5d5bf1e71971425b5b5..ff4647632bdf1418d515c58afb37e8a071ef042a 100644 (file)
@@ -3,6 +3,7 @@
 import dns.immutable
 import dns.rdtypes.txtbase
 
+
 @dns.immutable.immutable
 class WALLET(dns.rdtypes.txtbase.TXTBase):
     """WALLET record"""
index 44d6df57e291164db75e74848bb081807a458c55..3e35b1400f5ef092116a0e8098ff4e42ea0914e0 100644 (file)
@@ -60,7 +60,7 @@ class TXTBase(dns.rdata.Rdata):
         txt = ""
         prefix = ""
         for s in self.strings:
-            txt += '{}"{}"'.format(prefix, dns.rdata._escapify(s))
+            txt += f'{prefix}"{dns.rdata._escapify(s)}"'
             prefix = " "
         return txt
 
index 1358c4c00e6557f898caed7c2a914b18519d2eb3..da63eaf87e8d7a4f42002ec76fb0b3fa22b02550 100644 (file)
@@ -83,7 +83,7 @@ class NXDOMAIN(dns.exception.DNSException):
         else:
             msg = "The DNS query name does not exist"
         qnames = ", ".join(map(str, qnames))
-        return "{}: {}".format(msg, qnames)
+        return f"{msg}: {qnames}"
 
     @property
     def canonical_name(self):
@@ -154,7 +154,7 @@ def _errors_to_text(errors: List[ErrorTuple]) -> List[str]:
     """Turn a resolution errors trace into a list of text."""
     texts = []
     for err in errors:
-        texts.append("Server {} answered {}".format(err[0], err[3]))
+        texts.append(f"Server {err[0]} answered {err[3]}")
     return texts
 
 
@@ -1205,9 +1205,7 @@ class BaseResolver:
                 enriched_nameservers.append(enriched_nameserver)
         else:
             raise ValueError(
-                "nameservers must be a list or tuple (not a {})".format(
-                    type(nameservers)
-                )
+                f"nameservers must be a list or tuple (not a {type(nameservers)})"
             )
         return enriched_nameservers
 
index 1836adf1543fa0ed0528b4cb66554f3e09b1b915..2bb44a524b77916db087e4a4e9ef4344b3477570 100644 (file)
@@ -251,9 +251,7 @@ class Reader:
             # We convert them to syntax errors so that we can emit
             # helpful filename:line info.
             (ty, va) = sys.exc_info()[:2]
-            raise dns.exception.SyntaxError(
-                "caught exception {}: {}".format(str(ty), str(va))
-            )
+            raise dns.exception.SyntaxError(f"caught exception {str(ty)}: {str(va)}")
 
         if not self.default_ttl_known and rdtype == dns.rdatatype.SOA:
             # The pre-RFC2308 and pre-BIND9 behavior inherits the zone default
index 9c7ab9880d0672a838f4353d793c4cbdf03c2819..71316241cfdf1f71c277a01a3d22d3f0dc6d48f0 100644 (file)
@@ -113,7 +113,6 @@ lint.ignore = [
     "B904",
     "B011",
     "UP031",
-    "UP032",
 ]
 lint.exclude = ["tests/*"]
 
index f0c227de08e884fa742b251f6dc00c452b7a80d6..b6e7a451e56a081f1690d575ce0ef4c4b6a5e63c 100644 (file)
@@ -596,7 +596,7 @@ class AsyncTests(unittest.TestCase):
     @unittest.skipIf(not dns.quic.have_quic, "aioquic not available")
     def TestDoH3QueryIP(self):
         async def run():
-            nameserver_ip = '8.8.8.8'
+            nameserver_ip = "8.8.8.8"
             q = dns.message.make_query("example.com.", dns.rdatatype.A)
             r = dns.asyncquery.https(
                 q,
index 900a3fae414ea611da7485da01094958a3751605..f5d413085e935398183aa9841ea5ccc1376ce482 100644 (file)
@@ -248,5 +248,6 @@ class DNSOverHTTP3TestCase(unittest.TestCase):
             )
             self.assertTrue(q.is_response(r))
 
+
 if __name__ == "__main__":
     unittest.main()