Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[luci-app-ssr-plus]: add chinadns mode for mosdns #1314

Merged
merged 2 commits into from
Nov 8, 2023

Conversation

XiaoliChan
Copy link
Contributor

Update log:

@XiaoliChan XiaoliChan force-pushed the mosdns-improv branch 4 times, most recently from 883cc62 to 764c10b Compare November 4, 2023 09:01
@zxlhhyccc
Copy link
Contributor

为何不同时添加国外的域名来实现分流呢?

@XiaoliChan
Copy link
Contributor Author

XiaoliChan commented Nov 5, 2023

为何不同时添加国外的域名来实现分流呢?

希望你能再好好看看yaml文件以及代码逻辑

@zxlhhyccc
Copy link
Contributor

为何不同时添加国外的域名来实现分流呢?

希望你能再好好看看yaml文件以及代码逻辑

不好意思,我看错了,yaml目前看起来应该是分流了。
请问一下,实机测试的咋样?

@XiaoliChan
Copy link
Contributor Author

不好意思,我看错了,yaml目前看起来应该是分流了。 请问一下,实机测试的咋样?

你可以自己测一测,我感觉挺好

@zxlhhyccc
Copy link
Contributor

不好意思,我看错了,yaml目前看起来应该是分流了。 请问一下,实机测试的咋样?

你可以自己测一测,我感觉挺好

测试了一下,确实不错了,但是汉化几个地方代码未改过来:
image
image

@chobitsos
Copy link

已经完美跑两天,就是我有点疑惑不解,根据mosdns官方的域名匹配规则

域名匹配规则
域名规则有多个匹配方式:
以 domain: 开头,域匹配。e.g: domain:google.com 会匹配自身 google.com,以及其子域名 www.google.com, maps.l.google.com 等。
以 full: 开头,完整匹配。e.g: full:google.com 只会匹配自身。
以 keyword: 开头,关键字匹配。e.g: keyword:google.com 会匹配包含这个字段的域名,如 google.com.hk, www.google.com.hk。
以 regexp: 开头,正则匹配()。e.g: regexp:.+.google.com$。

但实际geosite_cn.txt和geosite_geolocation_not_cn.txt这里面放的都是纯域名,也没有domain开头full开头,这样确定能匹配到域名吗?

@XiaoliChan
Copy link
Contributor Author

但实际geosite_cn.txt和geosite_geolocation_not_cn.txt这里面放的都是纯域名,也没有domain开头full开头,这样确定能匹配到域名吗?

我不知道,随时可以改反正

@zxlhhyccc
Copy link
Contributor

zxlhhyccc commented Nov 5, 2023

已经完美跑两天,就是我有点疑惑不解,根据mosdns官方的域名匹配规则

域名匹配规则
域名规则有多个匹配方式:
以 domain: 开头,域匹配。e.g: domain:google.com 会匹配自身 google.com,以及其子域名 www.google.com, maps.l.google.com 等。
以 full: 开头,完整匹配。e.g: full:google.com 只会匹配自身。
以 keyword: 开头,关键字匹配。e.g: keyword:google.com 会匹配包含这个字段的域名,如 google.com.hk, www.google.com.hk。
以 regexp: 开头,正则匹配()。e.g: regexp:.+.google.com$。

但实际geosite_cn.txt和geosite_geolocation_not_cn.txt这里面放的都是纯域名,也没有domain开头full开头,这样确定能匹配到域名吗?

你可以用echo或者sed在前面加full之类的!多个的开头然后合并即可!

@AmberisMyShiba
Copy link
Contributor

请问 @chobitsos中提到的Google Play Store 谷歌商店无法下载的问题请问是如何解决的。修改了默认规则吗?

@XiaoliChan
Copy link
Contributor Author

XiaoliChan commented Nov 6, 2023

请问 @chobitsos中提到的Google Play Store 谷歌商店无法下载的问题请问是如何解决的。修改了默认规则吗?

geosite_cn和geosite_google_cn diff取差值

@chobitsos
Copy link

请问 @chobitsos中提到的Google Play Store 谷歌商店无法下载的问题请问是如何解决的。修改了默认规则吗?

geosite_cn和geosite google cn diff取差值

我把full:开头全部都加到了geosite_cn.txt,不出意外结果翻车
以taobao.com举例,我测试出这样的结果
full:taobao.com <--nslookup解析为163.181.35.232 无法打开网页错误为ERR_SSL_PROTOCOL_ERROR
taobao.com <--nslookup解析为121.11.2.101 正常打开网页
删除taobao.com <--nslookup解析为163.181.35.232 无法打开网页错误为ERR_SSL_PROTOCOL_ERROR
这是我对geosite_cn.txt里面的域名加full:的尝试

另外geosite_geolocation_not_cn.txt的域名里面加full:跟没加一样没什么变化
疑惑点就在这了geosite_cn.txt到底要不要加full:呢?

@chobitsos
Copy link

请问 @chobitsos中提到的Google Play Store 谷歌商店无法下载的问题请问是如何解决的。修改了默认规则吗?

对,我们默认规则全部都改了

@XiaoliChan
Copy link
Contributor Author

欸,,又不难改,我感觉加没加full都没差

@chobitsos
Copy link

chobitsos commented Nov 6, 2023

欸,,又不难改,我感觉加没加full都没差

可是在geosite_cn加了full:有些网站打不开啊,会被解析到海外IP,这不是geosite_cn吗,怎么整的跟not_cn一样效果了!!

@XiaoliChan
Copy link
Contributor Author

XiaoliChan commented Nov 6, 2023

可是在geosite_cn加了full:有些网站打不开啊,会被解析到海外IP,这不是geosite_cn吗,怎么整的跟not_cn一样效果了!!

不是,你为什么要动它,你自己都看到它说明写了,full 是指完全匹配域名,当你访问网站的时候,例如full:x.com,你访问的是www.x.com,那自然匹配不上啊....

再者,看图你就知道fullfull有没有差
image

我希望你们能不要再继续讨论这个问题,除非你们能够找到什么Bug

@XiaoliChan XiaoliChan force-pushed the mosdns-improv branch 2 times, most recently from d6594a2 to f66b33b Compare November 6, 2023 04:53
@XiaoliChan
Copy link
Contributor Author

@Beginner-Go 确定可以review了

@zxlhhyccc
Copy link
Contributor

@Beginner-Go 确定可以review了

大佬,请把改汉化对应的代码和添加需要汉化的代码处理一下,再请大屌合并!

@XiaoliChan
Copy link
Contributor Author

XiaoliChan commented Nov 6, 2023

大佬,请把改汉化对应的代码和添加需要汉化的代码处理一下,再请大屌合并!

改了啊,只不过我force push了而已

ipt2socks/Makefile Outdated Show resolved Hide resolved
@AmberisMyShiba
Copy link
Contributor

Hi all contributors! @XiaoliChan @Beginner-Go @smallprogram. I created a channel in Discord for discussing OpenWrt development. I sincerely extend an invitation to contributors who have been active in this repository recently. If interested, please join us by this link.

@XiaoliChan
Copy link
Contributor Author

v2dat 编译确定没问题吗? 上游没有 Releases

确定

@XiaoliChan
Copy link
Contributor Author

@Beginner-Go 好了

@zxlhhyccc
Copy link
Contributor

zxlhhyccc commented Nov 7, 2023

@Beginner-Go 好了

大佬,测试了一下,咋上游dns显示只有几个而已,而且全部是香港的dns?
image

用smartdns做上游再启动ssrp或者openclsh有很多上游dns,各国家的都有!
image

@XiaoliChan
Copy link
Contributor Author

XiaoliChan commented Nov 7, 2023

大佬,测试了一下,咋上游dns显示只有几个而已,而且全部是香港的dns?
用smartdns做上游再启动ssrp或者openclsh有很多上游dns,各国家的都有!

哥们,你是不是开了dns防泄露... 请你做多点功课先再来评论

@XiaoliChan
Copy link
Contributor Author

可以 medg 了吗?

可以了

@zxlhhyccc
Copy link
Contributor

哥们,你是不是开了dns防泄露... 请你做多点功课先再来评论

是开了防泄漏,这个是必须的啊,ssrp其他解析存在dns泄漏现象!

@zxlhhyccc
Copy link
Contributor

可以 medg 了吗?

大佬,你的习惯还没改,就是喜欢修改或合并哦!哈哈!

@XiaoliChan
Copy link
Contributor Author

XiaoliChan commented Nov 7, 2023

是开了防泄漏,这个是必须的啊,ssrp其他解析存在dns泄漏现象!

你要是真懂什么是防泄漏的话,你就不会问出那样的问题,再这么搞下去我pr也不交了

但是我还是要回答一下,我没有用过smartdns这个垃圾,因此我不知道,不过你可以阅读一下它的代码看看它的逻辑,在这个pr中,防泄漏是直接转发到国外dns(如果目标域名非大陆),而你这个问题和本pr根本无关,是mosdns和smartdns的问题

@zxlhhyccc
Copy link
Contributor

但是我还是要回答一下,我没有用过smartdns这个垃圾,因此我不知道,不过你可以阅读一下它的代码看看它的逻辑,在这个pr中,防泄漏是直接转发到国外dns(如果目标域名非大陆),而你这个问题和本pr根本无关,是mosdns和smartdns的问题

大佬,不好意思,我可能没说清楚让你误会。另外,有打算把dnsproxy也搞进去吗?

@XiaoliChan
Copy link
Contributor Author

XiaoliChan commented Nov 7, 2023

大佬,不好意思,我可能没说清楚让你误会。另外,有打算把dnsproxy也搞进去吗?

那个东西没必要啊。。。实在不想搞那么多了,先让这个PR合并了先吧

谈话终止

@zxlhhyccc
Copy link
Contributor

大佬,不好意思,我可能没说清楚让你误会。另外,有打算把dnsproxy也搞进去吗?

那个东西没必要啊。。。

dnsproxy解析上游也是很好用的,你试试就知道了,dnsproxy以前是用在ACC上做转发!

@zxlhhyccc
Copy link
Contributor

可以 medg 了吗?

请合并上去,基本上没啥问题了,用起来也不错!

@Beginner-Go Beginner-Go merged commit 2941ee4 into fw876:main Nov 8, 2023
0 of 7 checks passed
@XiaoliChan
Copy link
Contributor Author

Sweet,接下来就是要把yaml换成json,然后用jq替代yq

@XiaoliChan
Copy link
Contributor Author

XiaoliChan commented Nov 8, 2023

yq在openwrt的包中 很少作为依赖

是的,我明天改成用jq,因为刚刚看了,mosdns也支持json格式的配置文件

BTW,dnsproxy指的是哪个项目

@XiaoliChan
Copy link
Contributor Author

XiaoliChan commented Nov 8, 2023

dnsproxy 也是 go 写的项目

收到

Anyway, 有办法把我的邮箱从commit去除吗,我本地的时候用了rebase编辑了啊,不应该继续存在的

@XiaoliChan
Copy link
Contributor Author

有点麻烦了 得 git commit --amend 然后 push -f

你那边应该可以用rebase -i帮我去除的,麻烦了!

@XiaoliChan
Copy link
Contributor Author

XiaoliChan commented Nov 8, 2023

处理最新的先吧,顺便把你回复里面的link也删了

@XiaoliChan
Copy link
Contributor Author

我去看看怎么设置成noreply

Beginner-Go pushed a commit that referenced this pull request Nov 8, 2023
* [luci-app-ssr-plus]: add chinadns mode for mosdns

Signed-off-by: Xiaoli Chan

* [luci-app-ssr] more translate

Signed-off-by: Xiaoli Chan
@XiaoliChan
Copy link
Contributor Author

我来研究下怎么改历史的commit

Thanks, I don't want to get into j**l.

@XiaoliChan XiaoliChan deleted the mosdns-improv branch November 8, 2023 17:07
@XiaoliChan
Copy link
Contributor Author

XiaoliChan commented Nov 8, 2023

Done, now the git commit message won't disclose my private email address anymore.

BTW, would you like to join the discord group that @AmberisMyShiba created? @Beginner-Go

Beginner-Go pushed a commit that referenced this pull request Nov 8, 2023
* [luci-app-ssr-plus]: add chinadns mode for mosdns

Signed-off-by: Xiaoli Chan

* [luci-app-ssr] more translate

Signed-off-by: Xiaoli Chan
Beginner-Go pushed a commit that referenced this pull request Nov 8, 2023
* [luci-app-ssr-plus]: add chinadns mode for mosdns

Signed-off-by: Xiaoli Chan

* [luci-app-ssr] more translate

Signed-off-by: Xiaoli Chan
Beginner-Go pushed a commit that referenced this pull request Nov 8, 2023
* [luci-app-ssr-plus]: add chinadns mode for mosdns

Signed-off-by: Xiaoli Chan

* [luci-app-ssr] more translate

Signed-off-by: Xiaoli Chan
Beginner-Go pushed a commit that referenced this pull request Nov 8, 2023
* [luci-app-ssr-plus]: add chinadns mode for mosdns

Signed-off-by: Xiaoli Chan

* [luci-app-ssr] more translate

Signed-off-by: Xiaoli Chan
coolsnowwolf added a commit that referenced this pull request Nov 12, 2023
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.

5 participants