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

feat: add proxy support for download #4546

Merged
merged 1 commit into from
Jun 9, 2023
Merged

feat: add proxy support for download #4546

merged 1 commit into from
Jun 9, 2023

Conversation

hunshcn
Copy link
Contributor

@hunshcn hunshcn commented Jun 8, 2023

默认的 HTTPClient 没有配置 ProxyFromEnvironment,这导致在必须通过 proxy 访问互联网的环境无法使用。

而访问 api 使用的 resty 默认 transport 携带了 http.ProxyFromEnvironment,所以会发现目录可访问,文件无法下载。

@xhofe xhofe merged commit 47976af into AlistGo:main Jun 9, 2023
@welcome
Copy link

welcome bot commented Jun 9, 2023

Congrats on merging your first pull request! We here at behavior bot are proud of you!

@anwen-anyi
Copy link
Contributor

anwen-anyi commented Jun 16, 2023

这个pr 是不是 后面就不用在手动添加代理命令了?

例如我现在是在Windows本地搭建

1.启动一个命令窗口(使用的是Git bash),proxy使用的是v2,那就要输入export https_proxy=http://127.0.0.1:10809 http_proxy=http://127.0.0.1:10809 all_proxy=socks5://127.0.0.1:10809
2.命令输入好后 在输入 alist.exe server 启动命令

如果是clash 可以开tun模式 也可以使用 命令模式 端口改成 7890即可

是不是可以直接 省略上述命令 开启代理后它会自动检测到

@hunshcn
Copy link
Contributor Author

hunshcn commented Jun 17, 2023

这个pr 是不是 后面就不用在手动添加代理命令了?

例如我现在是在Windows本地搭建

1.启动一个命令窗口(使用的是Git bash),proxy使用的是v2,那就要输入export https_proxy=http://127.0.0.1:10809 http_proxy=http://127.0.0.1:10809 all_proxy=socks5://127.0.0.1:10809
2.命令输入好后 在输入 alist.exe server 启动命令
如果是clash 可以开tun模式 也可以使用 命令模式 端口改成 7890即可

是不是可以直接 省略上述命令 开启代理后它会自动检测到

不是,修复的是用你这个方法文件无法下载的问题。代理检测实际上就是检测环境变量,所以必然要设置的。

@anwen-anyi
Copy link
Contributor

那还不是得 像这样 #4363 (comment) 执行一下(也就是说和之前没区别)

@hunshcn
Copy link
Contributor Author

hunshcn commented Jun 18, 2023

那还不是得 像这样 #4363 (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

Successfully merging this pull request may close these issues.

3 participants