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

[dyh] -feat: add MultipartUpload method #51

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

Conversation

daiyuanhao
Copy link

补充了分片上传的接口

Copy link
Author

Choose a reason for hiding this comment

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

这边我不确定有没有影响到其他,逻辑可能还有问题,自测是通了。InitiateMultipartUpload这个url特殊,它是...?uploads,你的这个签名会跟oss那边的对不上

Choose a reason for hiding this comment

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

这边我不确定有没有影响到其他,逻辑可能还有问题,自测是通了。InitiateMultipartUpload这个url特殊,它是...?uploads,你的这个签名会跟oss那边的对不上

签名对不上的问题如何解决?

Choose a reason for hiding this comment

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

通了是指能上传成功了还是?

Copy link
Author

Choose a reason for hiding this comment

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

能上传成功

@huhx huhx self-assigned this Jan 18, 2024
@daiyuanhao
Copy link
Author

@huhx 有计划加到版本里嘛

@daiyuanhao
Copy link
Author

downloadObject方法的fileKey如果带上?x-oss-process=image/resize,l_200按我提交的改动会有问题。
后面发现原来直接在auth.sign的fileKey带上参数就能通...

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