Skip to content

bug#42

Open
drgtdrgtgsd wants to merge 1 commit into
byJoey:mainfrom
drgtdrgtgsd:main
Open

bug#42
drgtdrgtgsd wants to merge 1 commit into
byJoey:mainfrom
drgtdrgtgsd:main

Conversation

@drgtdrgtgsd

@drgtdrgtgsd drgtdrgtgsd commented Mar 16, 2026

Copy link
Copy Markdown

发现问题
您好,感谢您开发这个好用的工具!我在 Windows 环境下使用时遇到了一些兼容性 Bug,目前已经自行在本地修复,特地反馈给您供参考w
仅限于Windows环境,mac和Linux我暂时没有测试

在 Windows PowerShell 中,默认的 curl 实际上是 Invoke-WebRequest 的别名,不支持脚本里的参数,导致二进制测速文件下载失败。
当出现下载失败或其他错误时,脚本会调用 input()。如果是自动化运行等非交互式环境,会导致进程永久卡住挂起。

建议的修复方案
我修改了 cloudflare_speedtest.py,主要改动如下:

  1. 替换 curl:将 subprocess 里的 curl 统一改为 curl.exe
  2. 增加 TTY 检查:将所有的 input() 包装在 if sys.stdin.isatty(): 中。
    现在windows环境下能进行测速
    系统:Windows11 25H2
    可能讲的不是很好,我第一次尝试这样,如有什么问题,请多指正谢谢

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.

1 participant