Skip to content

Conversation

@Meo597
Copy link
Contributor

@Meo597 Meo597 commented Nov 26, 2025

以前漏掉了 dotless
也没说明涉及到正则表达式时的大小写敏感问题

Remove duplicate entries and clarify domain matching rules.
Clarify domain matching rule formats and examples in routing documentation.
@Meo597 Meo597 requested a review from Fangliding November 26, 2025 19:41
@Meo597 Meo597 changed the title Domain Matcher: More Detailed Explanation Domain Matcher: More detailed explanation Nov 26, 2025
Add explanation for loading domains from a file.
@yuhan6665
Copy link
Member

Thanks!

@yuhan6665 yuhan6665 merged commit 1606b37 into XTLS:main Nov 30, 2025
2 checks passed
@Meo597 Meo597 deleted the Meo597-patch-2 branch December 1, 2025 00:58
@Meo597
Copy link
Contributor Author

Meo597 commented Dec 1, 2025

@Fangliding 我看到你删除重复项了

dns 的 domains 和 router 的一样
但 hosts 的纯字符串是 full:

@Fangliding
Copy link
Member

我看都是strmatcher 还特意问了一下

@Meo597
Copy link
Contributor Author

Meo597 commented Dec 1, 2025

@Fangliding
Copy link
Member

好吧 早点回我也点了

@Meo597
Copy link
Contributor Author

Meo597 commented Dec 1, 2025

@Fangliding
我不在群里

dnsserverobj 的 qs 不是简单的继承覆盖全局
hosts 能匹配多个记录把它们 ip 合并
750 和 767 被你 fp 没了

我 push 了你看下

@Fangliding
Copy link
Member

querystrategy就那样改吧 多出来那两个真没必要我故意弄掉的

@Meo597
Copy link
Contributor Author

Meo597 commented Dec 1, 2025

这两个 yuhan 决定接受了,我个人觉得有必要

#767 是之前说 DNS Leak 你比较抵触,默默给 revert 了,现在换了个中立的说法,而且它确实影响了 sniff

#750 当用户用 reality + mux 的时候,第一个连接肯定是 TiT 特征,没什么人会注意到这个,我觉得提醒下好

@Meo597
Copy link
Contributor Author

Meo597 commented Dec 1, 2025

都是你之前 close
yuhan 又捞起来的

反正我是无所谓,你决定

@Fangliding
Copy link
Member

我再想想

@Meo597
Copy link
Contributor Author

Meo597 commented Dec 1, 2025

obs 会 7x24h 高强度探测,reality + mux,每 concurrency 次就必定有一个 tit
如果探测目标是 https 的话

@Fangliding
Copy link
Member

说实话一堆人用的clash是怎么测的 clash都在放肆用在这里杞人忧天没啥用

@Meo597
Copy link
Contributor Author

Meo597 commented Dec 1, 2025

高端用户和小白拉开差距就看这里了

所以最终 #750 还是 revert 了

@Meo597 Meo597 removed the request for review from Fangliding December 1, 2025 10:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants