From: Steven Black <redacted>
Date: Mon, 26 Apr 2021 21:55:39 +0000 (-0400)
Subject: Issue #1628: fix — parse out @ characters from lists.
X-Git-Url: http://git.99rst.org/?a=commitdiff_plain;h=023eb7faa79ffe6060c8f6350367a02509fa184f;p=stevenblack-hosts.git

Issue #1628: fix — parse out @ characters from lists.
---

diff --git a/testUpdateHostsFile.py b/testUpdateHostsFile.py
index d93028d6d..95a275645 100644
--- a/testUpdateHostsFile.py
+++ b/testUpdateHostsFile.py
@@ -634,7 +634,7 @@ class TestMatchesExclusions(Base):
             self.assertTrue(matches_exclusions(domain, exclusion_regexes))
 
     def test_match_raw_list(self):
-        exclusion_regexes = [r".*\.com", r".*\.org", r".*\.edu"]
+        exclusion_regexes = [r".*\.com", r".*\.org", r".*\.edu", r".*@.*"]
         exclusion_regexes = [re.compile(regex) for regex in exclusion_regexes]
 
         for domain in [
@@ -642,6 +642,7 @@ class TestMatchesExclusions(Base):
             "yahoo.com",
             "adaway.org",
             "education.edu",
+            "a.stro.lo.gy@45.144.225.135",
         ]:
             self.assertTrue(matches_exclusions(domain, exclusion_regexes))
 
diff --git a/updateHostsFile.py b/updateHostsFile.py
index 3656c5f71..1bb937c1b 100644
--- a/updateHostsFile.py
+++ b/updateHostsFile.py
@@ -943,6 +943,10 @@ def remove_dups_and_excl(merge_file, exclusion_regexes, output_file=None):
         if not stripped_rule or matches_exclusions(stripped_rule, exclusion_regexes):
             continue
 
+        # Issue #1628
+        if ("@" in stripped_rule):
+            continue
+
         # Normalize rule
         hostname, normalized_rule = normalize_rule(
             stripped_rule,