wild-cloud/wild-central
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases 1 Wiki Activity

fix: Handle dnsmasq local=/ directive in DNS filter parser

Browse Source 
The parser only recognized address=/ and server=/ prefixes but not
local=/, which is the format used by Hagezi blocklists. This caused
~270k domains to be stored with the raw "local=/domain/" string as
the domain name, breaking allow-list matching.
This commit is contained in:
Paul Payne
2026-08-19 23:11:50 +00:00
parent 03ce1f3c42
commit a90af10932
2 changed files with 3 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
internal/dnsfilter/parser.go
View File

@@ -16,8 +16,8 @@ func ParseLine(line string) (string, bool) {
return "", false return "", false
} }
// dnsmasq format: address=/domain/0.0.0.0 or server=/domain/ // dnsmasq format: address=/domain/... or server=/domain/ or local=/domain/
if strings.HasPrefix(line, "address=/") || strings.HasPrefix(line, "server=/") { if strings.HasPrefix(line, "address=/") || strings.HasPrefix(line, "server=/") || strings.HasPrefix(line, "local=/") {
parts := strings.SplitN(line, "/", 4) parts := strings.SplitN(line, "/", 4)
if len(parts) >= 3 && parts[1] != "" { if len(parts) >= 3 && parts[1] != "" {
return validateDomain(parts[1]) return validateDomain(parts[1])

1
internal/dnsfilter/parser_test.go
View File

@@ -39,6 +39,7 @@ func TestParseLine_DnsmasqFormat(t *testing.T) {
{"address=/ads.example.com/0.0.0.0", "ads.example.com", true}, {"address=/ads.example.com/0.0.0.0", "ads.example.com", true},
{"address=/tracker.example.com/", "tracker.example.com", true}, {"address=/tracker.example.com/", "tracker.example.com", true},
{"server=/blocked.example.com/", "blocked.example.com", true}, {"server=/blocked.example.com/", "blocked.example.com", true},
{"local=/blocked.example.com/", "blocked.example.com", true},
{"address=//", "", false}, {"address=//", "", false},
} }