From: funilrys <redacted>
Date: Fri, 8 Sep 2023 16:45:30 +0000 (+0200)
Subject: Correct 2nd Iter: Ignore subject with slashes (/).
X-Git-Url: http://git.99rst.org/?a=commitdiff_plain;h=870ade35220303f1ab8d2f6dcbdbae4731d1da31;p=stevenblack-hosts.git

Correct 2nd Iter: Ignore subject with slashes (/).

Upon checking the lastest release after merging #2433, I noticed
that some subjects with slashes goes through.

This patch fixes that by ignoring subjects which contains slashes.
---

diff --git a/testUpdateHostsFile.py b/testUpdateHostsFile.py
index 69a90984c..4f580f2e2 100644
--- a/testUpdateHostsFile.py
+++ b/testUpdateHostsFile.py
@@ -843,6 +843,7 @@ class TestNormalizeRule(BaseStdout):
             "::1",
             "0.0.0.0 128.0.0.2",
             "0.1.2.3 foo/bar",
+            "0.3.4.5 example.org/hello/world",
             "0.0.0.0 https",
             "0.0.0.0 https..",
         ]:
diff --git a/updateHostsFile.py b/updateHostsFile.py
index 291faa0a4..77e9cc844 100755
--- a/updateHostsFile.py
+++ b/updateHostsFile.py
@@ -1129,6 +1129,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
             or ":" in hostname
         ):
@@ -1147,6 +1148,7 @@ def normalize_rule(rule, target_ip, keep_domain_comments):
         not re.search(static_ip_regex, split_rule[0])
         and ":" not in 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 or INVALID.