Take nameservers that are not IPv4/v6 addresses or https URLs out of the

mix.

This also fixes the UnboundLocalError from [Issue #509].

1 files changed, 4 insertions, 5 deletions

diff --git a/dns/resolver.py b/dns/resolver.py
index 4948d96..1fe9224 100644
--- a/dns/resolver.py
+++ b/dns/resolver.py
@@ -1083,11 +1083,10 @@ class Resolver:
                                                      raise_on_truncation=True)
                     else:
                         protocol = urlparse(nameserver).scheme
-                        if protocol == 'https':
-                            response = dns.query.https(request, nameserver,
-                                                       timeout=timeout)
-                        elif protocol:
-                            continue
+                        if protocol != 'https':
+                            raise NotImplementedError
+                        response = dns.query.https(request, nameserver,
+                                                   timeout=timeout)
                 except Exception as ex:
                     (_, done) = resolution.query_result(None, ex)
                     continue