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 Implementation bug & Support AWS Signature V2 #6683

Merged
merged 2 commits into from
Jul 7, 2024

Conversation

akang943578
Copy link
Contributor

@akang943578 akang943578 commented Jul 2, 2024

This PR has 2 Commits:

  • Fix the bug Alist S3 Implementation not support multi-level objectName(like aaa/bbb.txt) when PutObject.
  • Upgrade gofakes3 to v0.0.5, which already supported AWS Signature V2.

Already tested by Seafile + Alist S3 + AliyunOpen backend storage.

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

(cherry picked from commit eb24f45)
@itsHenry35
Copy link
Member

@xhofe If you have time, pls merge this. Checked and seems fine.

@xhofe xhofe merged commit ca0d66b into alist-org:main Jul 7, 2024
3 checks passed
Copy link

welcome bot commented Jul 7, 2024

Congrats on merging your first pull request! We here at behavior bot are proud of you!

ForSourceCodeAnalysis pushed a commit to ForSourceCodeAnalysis/alist that referenced this pull request Aug 4, 2024
* Fix: when S3 PutObject with objectName contains /, aliyundriveopen failed due to KeyNotFound, make dir to fix this.

(cherry picked from commit eb24f45)

* Upgrade gofakes3 to v0.0.5, support AWS Signature V2

(cherry picked from commit 3218d7c)

---------

Co-authored-by: David Hao <akang943578@gmail.com>
Three-taile-dragon pushed a commit to loognsss/blist that referenced this pull request Sep 26, 2024
* Fix: when S3 PutObject with objectName contains /, aliyundriveopen failed due to KeyNotFound, make dir to fix this.

(cherry picked from commit eb24f45)

* Upgrade gofakes3 to v0.0.5, support AWS Signature V2

(cherry picked from commit 3218d7c)

---------

Co-authored-by: David Hao <akang943578@gmail.com>
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.

3 participants