使用DOH方式查询DNS，并且DOH服务器经过代理，当有并发DNS查询请求时，第一个DNS查询请求返回结果后，其余DNS查询请求均会失败。 about v2ray-core

[Loyalsoldier]

试一试 VMess 看看会不会有一样的问题？

[badO1a5A90]

试一试 VMess 看看会不会有一样的问题？

忘记补充了，已经试过，一样的

[badO1a5A90]

试一试 VMess 看看会不会有一样的问题？

v2ray-core/app/dns/dohdns.go 的 282 行 resp, err := s.httpClient.Do(req.WithContext(ctx))

好像是因为一个DNS请求完成时，Dispatched connection关闭，导致了其他并发请求失败
NewDoHNameServer和NewDoHLNameServer的httpClient不同的http.Transport。

我简单把这行改为了

	tr := &http.Transport{
		IdleConnTimeout:   90 * time.Second,
		ForceAttemptHTTP2: true,
		DialContext: func(ctx context.Context, network, addr string) (net.Conn, error) {
			dest, err := net.ParseDestination(network + ":" + addr)
			if err != nil {
				return nil, err
			}
			conn, err := internet.DialSystem(ctx, dest, nil)
			if err != nil {
				return nil, err
			}
			return conn, nil
		},
	}

	hc := &http.Client{
		Timeout:   time.Second * 180,
		Transport: tr,
	}

	resp, err := hc.Do(req.WithContext(ctx))

这样问题是可以解决的。
但是这样的改法应该是不OK的。

[badO1a5A90]

试一试 VMess 看看会不会有一样的问题？

v2ray-core/app/dns/dohdns.go 的 282 行 resp, err := s.httpClient.Do(req.WithContext(ctx))

好像是因为一个DNS请求完成时，Dispatched connection关闭，导致了其他并发请求失败
NewDoHNameServer和NewDoHLNameServer的httpClient不同的http.Transport。

我简单把这行改为了

	tr := &http.Transport{
		IdleConnTimeout:   90 * time.Second,
		ForceAttemptHTTP2: true,
		DialContext: func(ctx context.Context, network, addr string) (net.Conn, error) {
			dest, err := net.ParseDestination(network + ":" + addr)
			if err != nil {
				return nil, err
			}
			conn, err := internet.DialSystem(ctx, dest, nil)
			if err != nil {
				return nil, err
			}
			return conn, nil
		},
	}

	hc := &http.Client{
		Timeout:   time.Second * 180,
		Transport: tr,
	}

	resp, err := hc.Do(req.WithContext(ctx))

这样问题是可以解决的。
但是这样的改法应该是不OK的。

我傻了，这样就是直连没过代理所以没问题了。。。

[badO1a5A90]

原因似乎就是因为一个DNS请求完成时，Dispatched connection关闭，导致了其他并发请求失败了？

重新改了一下
给DoHNameServer加了个dispatcher routing.Dispatcher

type DoHNameServer struct {
	dispatcher routing.Dispatcher
	sync.RWMutex
	ips        map[string]record
	pub        *pubsub.Service
	cleanup    *task.Periodic
	reqID      uint32
	clientIP   net.IP
	httpClient *http.Client
	dohURL     string
	name       string
}

func NewDoHNameServer 中增加了一行

s.dispatcher = dispatcher

282 行 resp, err := s.httpClient.Do(req.WithContext(ctx)) 改成了下方代码

即判断dispatcher是否为空，不为空则是DOH而不是DOHL
DOHL还是原来的处理方式
如果是DOH的话，每个request都是独立的httpClient，就不会因为一个请求完成而中断了connection导致其他的请求失败了。

	hc := s.httpClient

	if s.dispatcher != nil {
		tr := &http.Transport{
			MaxIdleConns:        30,
			IdleConnTimeout:     90 * time.Second,
			TLSHandshakeTimeout: 30 * time.Second,
			ForceAttemptHTTP2:   true,
			DialContext: func(ctx context.Context, network, addr string) (net.Conn, error) {
				dest, err := net.ParseDestination(network + ":" + addr)
				if err != nil {
					return nil, err
				}

				link, err := s.dispatcher.Dispatch(ctx, dest)
				if err != nil {
					return nil, err
				}
				return net.NewConnection(
					net.ConnectionInputMulti(link.Writer),
					net.ConnectionOutputMulti(link.Reader),
				), nil
			},
		}
		hc = &http.Client{
			Timeout:   time.Second * 180,
			Transport: tr,
		}
	}

	resp, err := hc.Do(req.WithContext(ctx))

这样能解决问题。
但是这样处理看起来似乎有点笨拙。。。

[Loyalsoldier]

ping @vcptr

[darhwa]

dnsCtx = session.ContextWithMuxPrefered(dnsCtx, true)

你把这行删掉试试还有没有问题？如果没有的话我就提交补丁

[badO1a5A90]

dnsCtx = session.ContextWithMuxPrefered(dnsCtx, true)

你把这行删掉试试还有没有问题？如果没有的话我就提交补丁

不行，依然是一堆io: read/write on closed pipe

[bbi8]

有客户端1使用远程doh，客户端2不使用远程dns
在客户端1dns解析失败的时候服务端可以看到日志127.0.0.1:60352 accepted tcp:dns.google:443
此时用客户端2下的浏览器访问https://dns.google/dns-query 也是无法打开的，但是访问其他网站没有毛病，服务端日志正常
再同时vps上运行curl https://dns.google/dns-query 也没有问题
感觉问题可能出在服务端而未必是客户端

[bbi8]

测试了段时间，可能跟Freedom的"domainStrategy": "UseIP"参数有关，vps上ipv6只是局域网没有公网出口，现在改成UseIPv4，有问题再说

[github-actions]

This issue is stale because it has been open 120 days with no activity. Remove stale label or comment or this will be closed in 5 days