Skip to content

Commit

Permalink
Merge pull request #250 from n8225/fixS3MultiPartUpload
Browse files Browse the repository at this point in the history
Fix s3 100MB max upload issue.
  • Loading branch information
aspacca committed Aug 24, 2019
2 parents 11dffa2 + a2ab63c commit 6c54ddc
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 2 deletions.
1 change: 1 addition & 0 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -96,6 +96,7 @@ provider | which storage provider to use | (s3, gdrive or local) |
aws-access-key | aws access key | | AWS_ACCESS_KEY
aws-secret-key | aws access key | | AWS_SECRET_KEY
bucket | aws bucket | | BUCKET
s3-endpoint | Custom S3 endpoint. | |
s3-region | region of the s3 bucket | eu-west-1 | S3_REGION
s3-no-multipart | disables s3 multipart upload | false | |
s3-path-style | Forces path style URLs, required for Minio. | false | |
Expand Down
2 changes: 0 additions & 2 deletions server/storage.go
Original file line number Diff line number Diff line change
Expand Up @@ -243,9 +243,7 @@ func (s *S3Storage) Put(token string, filename string, reader io.Reader, content

// Create an uploader with the session and custom options
uploader := s3manager.NewUploader(s.session, func(u *s3manager.Uploader) {
u.PartSize = (1 << 20) * 5 // The minimum/default allowed part size is 5MB
u.Concurrency = concurrency // default is 5
u.MaxUploadParts = concurrency
u.LeavePartsOnError = false
})

Expand Down

0 comments on commit 6c54ddc

Please sign in to comment.