Unverified Commit e016244a authored by Hou Xiaoxuan's avatar Hou Xiaoxuan Committed by GitHub

fix: remove ACL when set URLPrefix (#2532)

parent d317b038
...@@ -64,13 +64,17 @@ func NewClient(ctx context.Context, config *Config) (*Client, error) { ...@@ -64,13 +64,17 @@ func NewClient(ctx context.Context, config *Config) (*Client, error) {
func (client *Client) UploadFile(ctx context.Context, filename string, fileType string, src io.Reader) (string, error) { func (client *Client) UploadFile(ctx context.Context, filename string, fileType string, src io.Reader) (string, error) {
uploader := manager.NewUploader(client.Client) uploader := manager.NewUploader(client.Client)
uploadOutput, err := uploader.Upload(ctx, &awss3.PutObjectInput{ // set ACL according to if user set prefix
obj := awss3.PutObjectInput{
Bucket: aws.String(client.Config.Bucket), Bucket: aws.String(client.Config.Bucket),
Key: aws.String(filename), Key: aws.String(filename),
Body: src, Body: src,
ContentType: aws.String(fileType), ContentType: aws.String(fileType),
ACL: types.ObjectCannedACL(*aws.String("public-read")), }
}) if client.Config.URLPrefix == "" {
obj.ACL = types.ObjectCannedACL(*aws.String("public-read"))
}
uploadOutput, err := uploader.Upload(ctx, &obj)
if err != nil { if err != nil {
return "", err return "", err
} }
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment