-
-
Notifications
You must be signed in to change notification settings - Fork 1.2k
feat(123_open): add DirectLink
option
#1045
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
Conversation
Co-authored-by: Copilot <[email protected]> Signed-off-by: MadDogOwner <[email protected]>
UseDirectLink
optionDirectLink
option
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
This PR adds a UseDirectLink
option to the 123_open driver that allows users to return CDN direct links instead of regular download URLs, which will consume direct link traffic. The feature requires users to manually enable direct link space first.
Key changes:
- Added direct link configuration options with optional URL authentication
- Implemented direct link API endpoint and response handling
- Modified the Link method to support both direct link and traditional download modes
Reviewed Changes
Copilot reviewed 4 out of 4 changed files in this pull request and generated 3 comments.
File | Description |
---|---|
drivers/123_open/meta.go | Added configuration fields for direct link feature including private key and validation duration |
drivers/123_open/types.go | Added DirectLinkResp struct and changed UID type from int64 to uint64 |
drivers/123_open/util.go | Added direct link API endpoint, URL signing method, and getDirectLink function |
drivers/123_open/driver.go | Modified Link method to conditionally use direct links with authentication support |
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
You can also share your feedback on Copilot code review for a chance to win a $100 gift card. Take the survey.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.
123分享,可以像豆包一样一行一个目录吗。现在需要每个分享链接创建一个驱动 |
目前123open未实现分享,另外请到合适的地方反馈 |
万岁,测试版能用上了 |
增加 使用直链 开关,启用后,返回 CDN 直链,会消耗直链流量。需要先启用直链空间。
https://123yunpan.yuque.com/org-wiki-123yunpan-muaork/cr6ced/tdxfsmtemp4gu4o2
已知问题:
未实现直链URL鉴权,正在处理已实现Close: #1043
TODO: 增加文档OpenListTeam/OpenList-Docs#134