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

add ipv6 support #27

Open
wants to merge 7 commits into
base: master
Choose a base branch
from
Open

add ipv6 support #27

wants to merge 7 commits into from

Conversation

wits-fe
Copy link

@wits-fe wits-fe commented Apr 26, 2023

若浏览器不支持 dnsResolveEx 方法
会自动回退到使用 dnsResolve 方法

IE11,Win10系统代理,Chrome 测试通过

@iBug
Copy link
Owner

iBug commented Apr 28, 2023

如果 dnsResolveEx 没定义的话,JavaScript 的默认行为不是抛个异常吗?是否需要把 dnsResolveEx 放进一个 try catch 块里?

@wits-fe
Copy link
Author

wits-fe commented Apr 28, 2023

不知道啊,我也不懂 JavaScript
在 IE11 测试正常(这个应该是不支持 dnsResolveEx),Chrome 支持 dnsResolveEx

@wits-fe
Copy link
Author

wits-fe commented Apr 28, 2023

纠正下:
刚测试了下,IE 11 支持 dnsResolveEx
https://learn.microsoft.com/en-us/windows/win32/winhttp/differences-between-ipv6-aware-wpad-helper-functions-and-legacy-wpad-helper-functions
稍后修改下代码

Now checks if function "dnsResolveEx" exist.
@wits-fe
Copy link
Author

wits-fe commented Apr 29, 2023

另外在这里提醒一下:
开启 IPv6 可能打开网站卡顿,原因看这里(PMTU黑洞):
https://www.v2ex.com/t/800024
可以尝试将 mss 设置为1400或更低(我这里设置为1392)

@iBug
Copy link
Owner

iBug commented May 22, 2023

代码看起来没啥问题,我现在主要考虑的问题是这个项目我自己已经快三年没有使用了(我现在主要用 Clash),所以并不想继续维护了,而 merge 进来意味着还要处理后续的 issue 和 PR。

@wits-fe 考虑换个方案如何?我更新一下 README 链接到你 fork 的仓库,建议有 IPv6 需求的用户去看你的仓库。

@wits-fe
Copy link
Author

wits-fe commented May 23, 2023

好的。

Clash对于有分流需求确实是个好的方案

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.

2 participants