Marown
/
hosts
mirror of https://github.com/StevenBlack/hosts.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

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.
This commit is contained in:
funilrys 2023-09-08 18:45:30 +02:00
parent 16f0e134ab
commit 870ade3522
No known key found for this signature in database
GPG Key ID: 0D8BFEF5515C00C6
2 changed files with 3 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
testUpdateHostsFile.py
View File

@ -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..",
]:

2
updateHostsFile.py
View File

@ -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.