Skip to content

Commit

Permalink
Merge pull request #399 from dutchcoders/ISSUE-398
Browse files Browse the repository at this point in the history
s3 and storj storage: use nil when purgeDays is 0
  • Loading branch information
stefanbenten authored Aug 15, 2021
2 parents d16a6cc + 224f8dc commit 440ccf3
Showing 1 changed file with 12 additions and 2 deletions.
14 changes: 12 additions & 2 deletions server/storage.go
Original file line number Diff line number Diff line change
Expand Up @@ -273,11 +273,16 @@ func (s *S3Storage) Put(token string, filename string, reader io.Reader, content
u.LeavePartsOnError = false
})

var expire *time.Time
if s.purgeDays.Hours() > 0 {
expire = aws.Time(time.Now().Add(s.purgeDays))
}

_, err = uploader.Upload(&s3manager.UploadInput{
Bucket: aws.String(s.bucket),
Key: aws.String(key),
Body: reader,
Expires: aws.Time(time.Now().Add(s.purgeDays)),
Expires: expire,
})

return
Expand Down Expand Up @@ -713,7 +718,12 @@ func (s *StorjStorage) Put(token string, filename string, reader io.Reader, cont

ctx := context.TODO()

writer, err := s.project.UploadObject(ctx, s.bucket.Name, key, &uplink.UploadOptions{Expires: time.Now().Add(s.purgeDays)})
var uploadOptions *uplink.UploadOptions
if s.purgeDays.Hours() > 0 {
uploadOptions = &uplink.UploadOptions{Expires: time.Now().Add(s.purgeDays)}
}

writer, err := s.project.UploadObject(ctx, s.bucket.Name, key, uploadOptions)
if err != nil {
return err
}
Expand Down

0 comments on commit 440ccf3

Please sign in to comment.