-
Notifications
You must be signed in to change notification settings - Fork 56
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
Error uploading path with special characters #741
Comments
Hello, can you also paste the error response here? |
Hi, I can't reproduce this error. Here is my test case: func TestIssue741(t *testing.T) {
if os.Getenv("STORAGE_S3_INTEGRATION_TEST") != "on" {
t.Skipf("STORAGE_S3_INTEGRATION_TEST is not 'on', skipped")
}
store := setupTest(t)
content := []byte("Hello, World!")
r := bytes.NewReader(content)
_, err := store.Write("IMG@@@¥&_0960.jpg", r, int64(len(content)))
if err != nil {
t.Errorf("write: %v", err)
return
}
return
} :) go test -count=1 ./tests -v -run TestIssue741
=== RUN TestIssue741
utils_test.go:15: Setup test for s3
--- PASS: TestIssue741 (0.58s)
PASS
ok github.com/beyondstorage/go-service-s3/v2/tests 0.584s Test passed with |
BTW, I found the |
If the name is not set to /default,Causes the following error: |
0 write: Storager s3 {Name: /default, WorkDir: /}, [IMG@@@¥&_0960.JPG]: unexpected: SignatureDoesNotMatch: The request signature we calculated does not match the signature you provided. Check your key and signing method.status code: 403, request id: 16A1779B7B3E2DF9, host id: |
This is by design, we need to use
I believe it's caused by the incorrect bucket name. Let's solve it by |
yep, s3.withForcepathstyle is required |
@jiaming-he , Do you have time to change the description of
|
The text was updated successfully, but these errors were encountered: