[Bug] 在经过 resp_ip 判断后 black_hole 无法替换 IPv4 或 IPv6 地址 about mosdns

[qingtian110]

可以在 GitHub 搜索下 openwrt mosdns luci 的配置文件，resp_ip + black_hole 它是工作的

[Journalist-HK]

可以在 GitHub 搜索下 openwrt mosdns luci 的配置文件，resp_ip + black_hole 它是工作的

如过你指的是这个
https://github.com/sbwml/luci-app-mosdns/blob/3ba579cf805795c407852d77607efa5fa1ac0e51/luci-app-mosdns/root/usr/share/mosdns/default.yaml#L141

那你可以尝试将图片中的“Cloudflare IP 范围”设置为只有 IPv4 或 IPv6 再试一试。

[qingtian110]

匹配范围 ipv4 only，这测试结果应该是对的吧。因为匹配 resp_ip 的 ipv4 地址，那么理所当然的也是更替应答结果的 ipv4 地址。

如果它连同 ipv6 也被更替了，我会觉得它是一个 bug。@IrineSistiana @sbwml 大佬们是不是这样的？

[Journalist-HK]

匹配范围 ipv4 only，这测试结果应该是对的吧。因为匹配 resp_ip 的 ipv4 地址，那么理所当然的也是更替应答结果的 ipv4 地址。

如果它连同 ipv6 也被更替了，我会觉得它是一个 bug。

按照你的说法，如果我只匹配 IPv6 地址，是不是应该只替换 IPv6 地址，而不替换 IPv4 地址？但是 mosdns 并没有这样工作，你可以再试一试。

[kkkgo]

1、A记录和AAAA记录是不同的qtype，他们的查询并不在同一个请求里面。
2、// Response returns a response with given ips if query has corresponding qtypes.
// Otherwise, it returns nil.
如果提供了blackhole的地址，就会替换，否则不替换。

[kkkgo]

3、如果说可能的bug的话，在Match方法里面，::/0是包含了0.0.0.0/0，0.0.0.0/0不包含::/0。
而Contains是先把IP统一转换成IPv6格式再匹配：

// Contains reports whether the list includes the given netip.Addr.
func (list *List) Contains(addr netip.Addr) bool {
	if !list.sorted {
		panic("list is not sorted")
	}
	if !addr.IsValid() {
		return false
	}

	addr = to6(addr)

	i, j := 0, len(list.e)
	for i < j {
		h := int(uint(i+j) >> 1) // avoid overflow when computing h
		if list.e[h].Addr().Compare(addr) <= 0 {
			i = h + 1
		} else {
			j = h
		}
	}

	if i == 0 {
		return false
	}

	return list.e[i-1].Contains(addr)
}

所以这也算不上bug，如果你想匹配全部AAAA记录，应该用qtype 28更高效。

[Journalist-HK]

所以这也算不上bug，如果你想匹配全部AAAA记录，应该用qtype 28更高效。

      - matches:
          - qtype 1
          - has_wanted_ans
          - resp_ip $cloudflare_ip
        exec: jump blackhole_cloudflare
      - matches:
          - qtype 28
          - has_wanted_ans
          - resp_ip $cloudflare_ipv6
        exec: jump blackhole_cloudflare_ipv6

这样写的话，能实现如果原来是 IPv4 only / IPv4 only / dual stack，替换后也是。

但是如何实现 prefer_ipv4？因为我希望只对命中规则的域名（在这里就是使用 CF CDN 的域名）实施 prefer_ipv4，其他的保留 IPv6。而 prefer_ipv4 只能放在 forward 之前，还没有请求上游前并不知道它是不是用了Cloudflare CDN。

[kkkgo]

“使用 CF CDN 的域名实施 prefer_ipv4"，那你直接把解析结果命中 CF V6 CDN 的请求reject掉不就好了……

1、A记录和AAAA记录是不同的qtype，他们的查询并不在同一个请求里面。

[Journalist-HK]

这样会影响只返回 AAAA 而没有 A 记录的域名，虽然这样的域名应该不多。正因为不是同一个请求，才不好实现。

[kkkgo]

纯v6 cf cdn的域名。不能说不多，只能说除非是故意的。