-
-
Notifications
You must be signed in to change notification settings - Fork 1.6k
feat(drivers): add degoo driver #1097
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
Signed-off-by: CaspianGUAN <app@caspian.im>
Signed-off-by: CaspianGUAN <app@caspian.im>
Signed-off-by: CaspianGUAN <app@caspian.im>
Signed-off-by: CaspianGUAN <app@caspian.im>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> Signed-off-by: CaspianGUAN <app@caspian.im>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> Signed-off-by: CaspianGUAN <app@caspian.im>
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.
这是一个海外网盘(非中国大陆),建议使用英文进行注释和错误输出。
另外,是否能够使用 base.HttpClient 执行请求?
Signed-off-by: CaspianGUAN <app@caspian.im>
Signed-off-by: CaspianGUAN <app@caspian.im>
Signed-off-by: CaspianGUAN <app@caspian.im>
Signed-off-by: CaspianGUAN <app@caspian.im>
全部改成英语注释了 |
Signed-off-by: CaspianGUAN <app@caspian.im>
Signed-off-by: CaspianGUAN <app@caspian.im>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> Signed-off-by: CaspianGUAN <app@caspian.im>
Signed-off-by: CaspianGUAN <app@caspian.im>
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.
请问您测试过,能用吗?是否需要帮助?
需要帮助,你测试一下 |
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 introduces a new Degoo driver to support cloud storage operations with the Degoo cloud service. The implementation provides authentication, file management, and upload capabilities through Degoo's GraphQL API.
- Complete driver implementation with authentication flow including login and token management
- Full CRUD operations for files and directories (list, create, move, rename, delete)
- Upload functionality with checksum validation and S3-based file transfer
Reviewed Changes
Copilot reviewed 6 out of 6 changed files in this pull request and generated 4 comments.
Show a summary per file
| File | Description |
|---|---|
| drivers/degoo/util.go | Core utility functions for API communication, authentication, and file operations |
| drivers/degoo/upload.go | Upload functionality including checksum calculation and S3 multipart upload |
| drivers/degoo/types.go | Type definitions for API requests and responses |
| drivers/degoo/meta.go | Driver configuration and registration |
| drivers/degoo/driver.go | Main driver implementation with all CRUD operations |
| drivers/all.go | Registration of the new degoo driver in the driver registry |
Comments suppressed due to low confidence (2)
drivers/degoo/util.go:1
- Parse errors are silently ignored. If time parsing or integer conversion fails, the function will return zero values without indicating an error occurred, which could lead to incorrect timestamps being used.
package degoo
drivers/degoo/util.go:1
- Parse errors are silently ignored. If time parsing or integer conversion fails, the function will return zero values without indicating an error occurred, which could lead to incorrect timestamps being used.
package degoo
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
xrgzs
left a comment
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.
测试 列出、下载、复制、移动、删除、上传、新建文件夹无问题
登录一开始可以,后面会报错 429,增加了手动填写 Token、失效自动刷新的功能
我自己生成了arm64和amd64的版本,测试了,可以用了,什么时候可以合成上架呢? |
还有文档没弄 |
* Create driver.go Signed-off-by: CaspianGUAN <app@caspian.im> * Create util.go Signed-off-by: CaspianGUAN <app@caspian.im> * Create types.go Signed-off-by: CaspianGUAN <app@caspian.im> * Create meta.go Signed-off-by: CaspianGUAN <app@caspian.im> * Update drivers/degoo/driver.go Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> Signed-off-by: CaspianGUAN <app@caspian.im> * Update drivers/degoo/driver.go Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> Signed-off-by: CaspianGUAN <app@caspian.im> * Update driver.go Signed-off-by: CaspianGUAN <app@caspian.im> * Update meta.go Signed-off-by: CaspianGUAN <app@caspian.im> * Update types.go Signed-off-by: CaspianGUAN <app@caspian.im> * Update util.go Signed-off-by: CaspianGUAN <app@caspian.im> * Update driver.go Signed-off-by: CaspianGUAN <app@caspian.im> * Update util.go Signed-off-by: CaspianGUAN <app@caspian.im> * Update drivers/degoo/util.go Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> Signed-off-by: CaspianGUAN <app@caspian.im> * Update util.go Signed-off-by: CaspianGUAN <app@caspian.im> * refactor(degoo): add Degoo driver integration and update API handling * fix(degoo): apply suggestions --------- Signed-off-by: CaspianGUAN <app@caspian.im> Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> Co-authored-by: MadDogOwner <xiaoran@xrgzs.top>
add degoo driver