Skip to content

Commit 440ccf3

Browse files
authored
Merge pull request #399 from dutchcoders/ISSUE-398
s3 and storj storage: use nil when purgeDays is 0
2 parents d16a6cc + 224f8dc commit 440ccf3

File tree

1 file changed

+12
-2
lines changed

1 file changed

+12
-2
lines changed

server/storage.go

Lines changed: 12 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -273,11 +273,16 @@ func (s *S3Storage) Put(token string, filename string, reader io.Reader, content
273273
u.LeavePartsOnError = false
274274
})
275275

276+
var expire *time.Time
277+
if s.purgeDays.Hours() > 0 {
278+
expire = aws.Time(time.Now().Add(s.purgeDays))
279+
}
280+
276281
_, err = uploader.Upload(&s3manager.UploadInput{
277282
Bucket: aws.String(s.bucket),
278283
Key: aws.String(key),
279284
Body: reader,
280-
Expires: aws.Time(time.Now().Add(s.purgeDays)),
285+
Expires: expire,
281286
})
282287

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

714719
ctx := context.TODO()
715720

716-
writer, err := s.project.UploadObject(ctx, s.bucket.Name, key, &uplink.UploadOptions{Expires: time.Now().Add(s.purgeDays)})
721+
var uploadOptions *uplink.UploadOptions
722+
if s.purgeDays.Hours() > 0 {
723+
uploadOptions = &uplink.UploadOptions{Expires: time.Now().Add(s.purgeDays)}
724+
}
725+
726+
writer, err := s.project.UploadObject(ctx, s.bucket.Name, key, uploadOptions)
717727
if err != nil {
718728
return err
719729
}

0 commit comments

Comments
 (0)