From: funilrys <redacted>
Date: Wed, 6 Sep 2023 19:52:52 +0000 (+0200)
Subject: Fix tests and logic.
X-Git-Url: http://git.99rst.org/?a=commitdiff_plain;h=65ebac6bf8e54d1f12cfa9ffdf02e8ef372d1c2f;p=stevenblack-hosts.git

Fix tests and logic.

Indeed, from on:
  1. We strip out IPs.
  2. We strip out "potential" INVALID that:
     - doesn't contains dots
     - contains at least 2 consecutive dots
     - looks like an IP.

From now on an acceptable subject shall:
  1. have at least 1 dot.
  2. NOT be an IPv4 or IPv6
  3. NOT look like an IP. (Example: 258.300.10.3)
---

diff --git a/testUpdateHostsFile.py b/testUpdateHostsFile.py
index a0392cd42..b519dc69c 100644
--- a/testUpdateHostsFile.py
+++ b/testUpdateHostsFile.py
@@ -842,8 +842,7 @@ class TestNormalizeRule(BaseStdout):
             "128.0.0.1",
             "::1",
             "0.0.0.0 128.0.0.2",
-            "0.0.0 google",
-            "0.1.2.3.4 foo/bar",
+            "0.1.2.3 foo/bar",
             "0.0.0.0 https",
             "0.0.0.0 https..",
         ]:
@@ -894,16 +893,16 @@ class TestNormalizeRule(BaseStdout):
     def test_two_ips(self):
         for target_ip in ("0.0.0.0", "127.0.0.1", "8.8.8.8"):
             rule = "127.0.0.1 11.22.33.44 foo"
-            expected = ("11.22.33.44", str(target_ip) + " 11.22.33.44\n")
 
             actual = normalize_rule(
                 rule, target_ip=target_ip, keep_domain_comments=False
             )
-            self.assertEqual(actual, expected)
+            self.assertEqual(actual, (None, None))
 
-            # Nothing gets printed if there's a match.
             output = sys.stdout.getvalue()
-            self.assertEqual(output, "")
+
+            expected = "==>" + rule + "<=="
+            self.assertIn(expected, output)
 
             sys.stdout = StringIO()
 
diff --git a/updateHostsFile.py b/updateHostsFile.py
index db3f3fb98..b925665bb 100755
--- a/updateHostsFile.py
+++ b/updateHostsFile.py
@@ -1127,6 +1127,7 @@ def normalize_rule(rule, target_ip, keep_domain_comments):
             is_ip(hostname)
             or re.search(static_ip_regex, hostname)
             or "." not in hostname
+            or ".." in hostname
             or ":" in hostname
         ):
             # Example: 0.0.0.0 127.0.0.1
@@ -1143,9 +1144,10 @@ def normalize_rule(rule, target_ip, keep_domain_comments):
     if (
         not re.search(static_ip_regex, split_rule[0])
         and ":" not in split_rule[0]
+        and ".." not in split_rule[0]
         and "." in split_rule[0]
     ):
-        # Deny anything that looks like an IP & doesn't container dots.
+        # Deny anything that looks like an IP; doesn't container dots or INVALID.
 
         try:
             hostname, suffix = split_rule