Minor bug fix
This commit is contained in:
@@ -104,7 +104,7 @@ func filterDNSResponse(conn *network.Connection, rrCache *resolver.RRCache) *res
|
|||||||
rrCache.Answer, filteredRecords, validIPs = filterDNSSection(rrCache.Answer, p, rrCache.ServerScope)
|
rrCache.Answer, filteredRecords, validIPs = filterDNSSection(rrCache.Answer, p, rrCache.ServerScope)
|
||||||
rrCache.FilteredEntries = append(rrCache.FilteredEntries, filteredRecords...)
|
rrCache.FilteredEntries = append(rrCache.FilteredEntries, filteredRecords...)
|
||||||
|
|
||||||
// FIXME(ppacher): should we consider valid IPs from the extra section?
|
// we don't count the valid IPs in the extra section
|
||||||
rrCache.Extra, filteredRecords, _ = filterDNSSection(rrCache.Extra, p, rrCache.ServerScope)
|
rrCache.Extra, filteredRecords, _ = filterDNSSection(rrCache.Extra, p, rrCache.ServerScope)
|
||||||
rrCache.FilteredEntries = append(rrCache.FilteredEntries, filteredRecords...)
|
rrCache.FilteredEntries = append(rrCache.FilteredEntries, filteredRecords...)
|
||||||
|
|
||||||
|
|||||||
@@ -83,7 +83,7 @@ func (br ListBlockReason) ToRRs() []dns.RR {
|
|||||||
wouldBeBlockedBy, err := dns.NewRR(fmt.Sprintf(
|
wouldBeBlockedBy, err := dns.NewRR(fmt.Sprintf(
|
||||||
"%s-wouldBeBlockedBy. 0 IN TXT %q",
|
"%s-wouldBeBlockedBy. 0 IN TXT %q",
|
||||||
strings.TrimRight(lm.Entity, "."),
|
strings.TrimRight(lm.Entity, "."),
|
||||||
strings.Join(lm.ActiveLists, ","),
|
strings.Join(lm.InactiveLists, ","),
|
||||||
))
|
))
|
||||||
if err == nil {
|
if err == nil {
|
||||||
rrs = append(rrs, wouldBeBlockedBy)
|
rrs = append(rrs, wouldBeBlockedBy)
|
||||||
|
|||||||
Reference in New Issue
Block a user