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

阿里云盘下载优先使用cdn直链 #6645

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

qy527145
Copy link

新增功能:

  • 优先提取cdn直链,可提速下载阿里云盘文件

关联issue:#6644

Copy link

welcome bot commented Jun 22, 2024

Thanks for opening this pull request! Please check out our contributing guidelines.

@ershouqingge
Copy link

大佬有暂时可用的docker容器可用么

@qy527145
Copy link
Author

大佬有暂时可用的docker容器可用么

啥意思,没理解啊

@ershouqingge
Copy link

大佬有暂时可用的docker容器可用么

啥意思,没理解啊

pr一直没合并啊 docker没有新映像

@@ -106,17 +106,26 @@ func (d *AliDrive) Link(ctx context.Context, file model.Obj, args model.LinkArgs
"file_id": file.GetID(),
"expire_sec": 14400,
}
res, err, _ := d.request("https://api.alipan.com/v2/file/get_download_url", http.MethodPost, func(req *resty.Request) {
res, err, _ := d.request("https://bj29.api.aliyunpds.com/v2/file/get_download_url", http.MethodPost, func(req *resty.Request) {
Copy link
Member

@SeanHeuc SeanHeuc Jun 27, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

replacing the official url with a third-party one is never a good practice. Alipan official could change this url anytime, and then many user will have trouble use this driver.
please put this url as an option if this can provide acceleration benefit.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants