LAZY_RULES: Adjust the priority order of China domains to fix #144#197
Open
betaxab wants to merge 1 commit intoHackl0us:mainfrom
Open
LAZY_RULES: Adjust the priority order of China domains to fix #144#197betaxab wants to merge 1 commit intoHackl0us:mainfrom
betaxab wants to merge 1 commit intoHackl0us:mainfrom
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
所有规则都是按照从上到下的顺序读取规则的,当读取到某条规则对某个域名执行后,以下对这个域名所有操作都将无效。
所以,应将国内网站的域名排到最底下,这样才能避免底下某些必须走代理的规则失效,例如 #144 中提到的 line me。
经过多月测试,这样改动不会对国内网站打开速度产生任何影响,所以我认为可以安心合并。
或者,也可以将如下两条规则,独立生成一份 RULESETS 并引用:
DOMAIN-SUFFIX,cn,DIRECT
DOMAIN-KEYWORD,-cn,DIRECT
但是我自认为没有必要。
如果可以的话,也建议将上面两条规则置于 CN.list 等规则片段的最底部最好。