💬 New comment on Xray-core#7089 XDNS finalmask: Fix resolver close deadlock and socket leaks
by @RPRX
> 等等，比如5个DOH，不同sni是不是就不能共用tls配置了
若要共用，TLS 配置还是得放外面啊，不是同一个问题，要实现你这个想法得把 addr 弄成 addrs 数组
有些人说不定还想共用 sockopt 呢，然后 type 1v5 就有点尴尬了，实际上这里还是并起来写最好，`udp://127.0.0.1:53` 这样
~~GKD，Go 1.27.2 出来后真得发 stable 了，要不你先开个 PR 改一下配置格式吧~~
Reply to this message to post a comment on GitHub.
💬 New comment on Xray-core#7089 XDNS finalmask: Fix resolver close deadlock and socket leaks
by @RPRX
我仔细想了一下，https://github.com/XTLS/Xray-core/pull/6718 重构后的配置结构有亿点问题
比如服务端，`name` 应该改为 names`，这样就可以多个 domain 共享同一套 `types 而不用多写一份 domains
~~其实之前的写法也没啥大问题，主要是一个 domain 要支持多个 type 的话看起来有点重复，就像现在这样拆成两项吧~~
客户端问题比较大，以前是支持哪个 domain 只走哪个 resolver 的，现在不行了
应该改为客户端仅需要 resolvers`，然后把 `names`、`types 等配置项移过来，以及上面并起来写的 `addrs`，这个没必要拆
~~先开个 PR 改一下吧，是的，`resolvers` 暂时是 domains 的超集，除非以后出了服务端专属配置项~~
Reply to this message to post a comment on GitHub.
💬 New comment on Xray-core#6871 Mux client: Fix a data race between Dispatch and SessionManager.Close
by @RPRX
@Fangliding 能合的话说一声
Reply to this message to post a comment on GitHub.
💬 New comment on Xray-core#7089 XDNS finalmask: Fix resolver close deadlock and socket leaks
by @HPPC114514
一个addr独享一份tls配置，
通过打tag的方式来标记“这些addrs”实际上是同一个reslover？
Reply to this message to post a comment on GitHub.
💬 New comment on Xray-core#7089 XDNS finalmask: Fix resolver close deadlock and socket leaks
by @HPPC114514
> 一个addr独享一份tls配置， 通过打tag的方式来标记“这些addrs”实际上是同一个reslover？
~甚至可以通过tag组来做分流…？~
Reply to this message to post a comment on GitHub.
💬 New comment on Xray-core#7089 XDNS finalmask: Fix resolver close deadlock and socket leaks
by @RPRX
~~虽然很符合 NEXT 架构的想法，但这么个小功能搞出 tag 这么复杂就暂时没必要了，绝大多数人也就写一两个 `resolvers`~~
但是至少能力要完备，比如“支持哪个 domain 只走哪个 resolver”这个不能做不到，~~若写两个 outbounds 就过于重复了~~
Reply to this message to post a comment on GitHub.