|
 |
711f45 |
From 9a33838407f244e481523fe643bc0626874e8b1a Mon Sep 17 00:00:00 2001
|
|
|
711f45 |
From: Rafael Guterres Jeffman <rjeffman@redhat.com>
|
|
|
711f45 |
Date: Mon, 19 Dec 2022 14:57:03 -0300
|
|
|
711f45 |
Subject: [PATCH] Revert "DNSResolver: Fix use of nameservers with ports"
|
|
|
711f45 |
|
|
|
711f45 |
This reverts commit 5e2e4664aec641886923c2bec61ce25b96edb62a.
|
|
|
711f45 |
---
|
|
|
711f45 |
ipapython/dnsutil.py | 41 -------------------------
|
|
|
711f45 |
ipatests/test_ipapython/test_dnsutil.py | 40 ------------------------
|
|
|
711f45 |
2 files changed, 81 deletions(-)
|
|
|
711f45 |
|
|
|
711f45 |
diff --git a/ipapython/dnsutil.py b/ipapython/dnsutil.py
|
|
|
711f45 |
index 58de365ab..4baeaf8cc 100644
|
|
|
711f45 |
--- a/ipapython/dnsutil.py
|
|
|
711f45 |
+++ b/ipapython/dnsutil.py
|
|
|
711f45 |
@@ -144,47 +144,6 @@ class DNSResolver(dns.resolver.Resolver):
|
|
|
711f45 |
nameservers.remove(ipv4_loopback)
|
|
|
711f45 |
self.nameservers = nameservers
|
|
|
711f45 |
|
|
|
711f45 |
- @dns.resolver.Resolver.nameservers.setter
|
|
|
711f45 |
- def nameservers(self, nameservers):
|
|
|
711f45 |
- """
|
|
|
711f45 |
- *nameservers*, a ``list`` of nameservers with optional ports:
|
|
|
711f45 |
- "SERVER_IP port PORT_NUMBER".
|
|
|
711f45 |
-
|
|
|
711f45 |
- Overloads dns.resolver.Resolver.nameservers setter to split off ports
|
|
|
711f45 |
- into nameserver_ports after setting nameservers successfully with the
|
|
|
711f45 |
- setter in dns.resolver.Resolver.
|
|
|
711f45 |
- """
|
|
|
711f45 |
- # Get nameserver_ports if it is already set
|
|
|
711f45 |
- if hasattr(self, "nameserver_ports"):
|
|
|
711f45 |
- nameserver_ports = self.nameserver_ports
|
|
|
711f45 |
- else:
|
|
|
711f45 |
- nameserver_ports = {}
|
|
|
711f45 |
-
|
|
|
711f45 |
- # Check nameserver items in list and split out converted port number
|
|
|
711f45 |
- # into nameserver_ports: { nameserver: port }
|
|
|
711f45 |
- if isinstance(nameservers, list):
|
|
|
711f45 |
- _nameservers = []
|
|
|
711f45 |
- for nameserver in nameservers:
|
|
|
711f45 |
- splits = nameserver.split()
|
|
|
711f45 |
- if len(splits) == 3 and splits[1] == "port":
|
|
|
711f45 |
- nameserver = splits[0]
|
|
|
711f45 |
- try:
|
|
|
711f45 |
- port = int(splits[2])
|
|
|
711f45 |
- if port < 0 or port > 65535:
|
|
|
711f45 |
- raise ValueError()
|
|
|
711f45 |
- except ValueError:
|
|
|
711f45 |
- raise ValueError(
|
|
|
711f45 |
- "invalid nameserver: %s is not a valid port" %
|
|
|
711f45 |
- splits[2])
|
|
|
711f45 |
- nameserver_ports[nameserver] = port
|
|
|
711f45 |
- _nameservers.append(nameserver)
|
|
|
711f45 |
- nameservers = _nameservers
|
|
|
711f45 |
-
|
|
|
711f45 |
- # Call dns.resolver.Resolver.nameservers setter
|
|
|
711f45 |
- dns.resolver.Resolver.nameservers.__set__(self, nameservers)
|
|
|
711f45 |
- # Set nameserver_ports after successfull call to setter
|
|
|
711f45 |
- self.nameserver_ports = nameserver_ports
|
|
|
711f45 |
-
|
|
|
711f45 |
|
|
|
711f45 |
class DNSZoneAlreadyExists(dns.exception.DNSException):
|
|
|
711f45 |
supp_kwargs = {'zone', 'ns'}
|
|
|
711f45 |
diff --git a/ipatests/test_ipapython/test_dnsutil.py b/ipatests/test_ipapython/test_dnsutil.py
|
|
|
711f45 |
index 09463c69d..5e7a46197 100644
|
|
|
711f45 |
--- a/ipatests/test_ipapython/test_dnsutil.py
|
|
|
711f45 |
+++ b/ipatests/test_ipapython/test_dnsutil.py
|
|
|
711f45 |
@@ -101,43 +101,3 @@ class TestSortURI:
|
|
|
711f45 |
assert dnsutil.sort_prio_weight([h3, h2, h1]) == [h1, h2, h3]
|
|
|
711f45 |
assert dnsutil.sort_prio_weight([h3, h3, h3]) == [h3]
|
|
|
711f45 |
assert dnsutil.sort_prio_weight([h2, h2, h1, h1]) == [h1, h2]
|
|
|
711f45 |
-
|
|
|
711f45 |
-
|
|
|
711f45 |
-class TestDNSResolver:
|
|
|
711f45 |
- def test_nameservers(self):
|
|
|
711f45 |
- res = dnsutil.DNSResolver()
|
|
|
711f45 |
- res.nameservers = ["4.4.4.4", "8.8.8.8"]
|
|
|
711f45 |
- assert res.nameservers == ["4.4.4.4", "8.8.8.8"]
|
|
|
711f45 |
-
|
|
|
711f45 |
- def test_nameservers_with_ports(self):
|
|
|
711f45 |
- res = dnsutil.DNSResolver()
|
|
|
711f45 |
- res.nameservers = ["4.4.4.4 port 53", "8.8.8.8 port 8053"]
|
|
|
711f45 |
- assert res.nameservers == ["4.4.4.4", "8.8.8.8"]
|
|
|
711f45 |
- assert res.nameserver_ports == {"4.4.4.4": 53, "8.8.8.8": 8053}
|
|
|
711f45 |
-
|
|
|
711f45 |
- res.nameservers = ["4.4.4.4 port 53", "8.8.8.8 port 8053"]
|
|
|
711f45 |
- assert res.nameservers == ["4.4.4.4", "8.8.8.8"]
|
|
|
711f45 |
- assert res.nameserver_ports == {"4.4.4.4": 53, "8.8.8.8": 8053}
|
|
|
711f45 |
-
|
|
|
711f45 |
- def test_nameservers_with_bad_ports(self):
|
|
|
711f45 |
- res = dnsutil.DNSResolver()
|
|
|
711f45 |
- try:
|
|
|
711f45 |
- res.nameservers = ["4.4.4.4 port a"]
|
|
|
711f45 |
- except ValueError:
|
|
|
711f45 |
- pass
|
|
|
711f45 |
- else:
|
|
|
711f45 |
- pytest.fail("No fail on bad port a")
|
|
|
711f45 |
-
|
|
|
711f45 |
- try:
|
|
|
711f45 |
- res.nameservers = ["4.4.4.4 port -1"]
|
|
|
711f45 |
- except ValueError:
|
|
|
711f45 |
- pass
|
|
|
711f45 |
- else:
|
|
|
711f45 |
- pytest.fail("No fail on bad port -1")
|
|
|
711f45 |
-
|
|
|
711f45 |
- try:
|
|
|
711f45 |
- res.nameservers = ["4.4.4.4 port 65536"]
|
|
|
711f45 |
- except ValueError:
|
|
|
711f45 |
- pass
|
|
|
711f45 |
- else:
|
|
|
711f45 |
- pytest.fail("No fail on bad port 65536")
|
|
|
711f45 |
--
|
|
|
711f45 |
2.38.1
|
|
|
711f45 |
|