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

bugs, bad treat #39

Open
zz5678 opened this issue Oct 23, 2020 · 3 comments
Open

bugs, bad treat #39

zz5678 opened this issue Oct 23, 2020 · 3 comments

Comments

@zz5678
Copy link

zz5678 commented Oct 23, 2020

--user-rule-from

@@||dl.google.com
@@||kh.google.com
@@||khm.google.com
@@||khm0.google.com
@@||khm1.google.com
@@||khm2.google.com
@@||khm3.google.com
@@||khmdb.google.com
@@||tools.google.com
@@||clientservices.googleapis.com
@@||fonts.googleapis.com
@@||khm.googleapis.com
@@||khm0.googleapis.com
@@||khm1.googleapis.com
@@||khm2.googleapis.com
@@||khm3.googleapis.com
@@||khmdb.googleapis.com
||storage.googleapis.com
||googleapis.cn
||googleapis.com

after genpac:

        [
            "google.com"
        ], 
        [
            "googleapis.cn", 
            "googleapis.com"
        ]
@zz5678
Copy link
Author

zz5678 commented May 23, 2021

genpac 生成 pac 的这个机制是不对的
对于三级域名和二级域名同在一个pac里,是各自生效的。
譬如:

@@||dl.google.com
||google.com

可以保证 dl.google.com 直连,而 google.com 走代理。
目前的处理机制一直是错的。

@zz5678
Copy link
Author

zz5678 commented May 23, 2021

我又去重复了一下很久之前提到的这个_本帖一楼_例子(去年)

genpac --gfwlist-url - --pac-proxy "SOCKS5 127.0.0.1:1080" -o proxy.pac --user-rule-from test.txt

genpac 非常诡异的把其中并不包含 google.com 的多个三级域名都给忽略了,而留下了二级域名 google.com,对 googleapis 无论cn还是com 也是这样处理的。

这样大幅度的缩水原始 gfwlist 会带来非常严重的问题。
作者这个项目还在 pip 里提供,更是很多用户会中招的。赶紧改个新版本吧

多谢!

@zz5678
Copy link
Author

zz5678 commented May 23, 2021

@petronny 从你的 gfwlist2pac 那里找过来的,看看,是否确定也存在这个bug?
如上2楼 #39 (comment)

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

No branches or pull requests

1 participant