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

Fix: S3 bug with AliyundriveOpen when objectName contains /. #6660

Closed
wants to merge 8 commits into from

Conversation

akang943578
Copy link
Contributor

Fix: when S3 PutObject with objectName contains /, aliyundriveopen failed due to KeyNotFound, make dir to fix this.

Refer Issue: #6588

Tested with Seafile and AliyundriveOpen.

…iled due to KeyNotFound, make dir to fix this.
Copy link

welcome bot commented Jun 27, 2024

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

…eopen failed due to KeyNotFound, make dir to fix this."

This reverts commit eb24f45.
@akang943578 akang943578 marked this pull request as draft June 28, 2024 10:56
…es of create multi-level file"

This reverts commit 7ae7857.
…iyundriveopen failed due to KeyNotFound, make dir to fix this.""

This reverts commit 6f85663.
@pull-request-size pull-request-size bot added size/M and removed size/S labels Jun 30, 2024
@itsHenry35
Copy link
Contributor

@akang943578 gofakes3的库不要替换,还是用alist-org下的,go mod中更新版本即可

@itsHenry35
Copy link
Contributor

然后你这pr是不是得改个名😊又是修另一个问题了

@akang943578
Copy link
Contributor Author

然后你这pr是不是得改个名😊又是修另一个问题了

Ok, 现在commit太多,我重新提一个PR。
是2个问题:
1) S3上传支持多级目录文件
2) go.mod 更新gofakes3 v0.05,支持AWS Signature V2

@pull-request-size pull-request-size bot added size/S and removed size/M labels Jul 2, 2024
@akang943578
Copy link
Contributor Author

然后你这pr是不是得改个名😊又是修另一个问题了

Ok, 现在commit太多,我重新提一个PR。 是2个问题: 1) S3上传支持多级目录文件 2) go.mod 更新gofakes3 v0.05,支持AWS Signature V2

New PR: #6683

@itsHenry35 itsHenry35 closed this Jul 2, 2024
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.

2 participants