Fixing minio not closing read object (#4971)

Этот коммит содержится в:
Christopher Speller
2017-01-04 16:34:37 -05:00
коммит произвёл Harrison Healey
родитель 17968f0392
Коммит 3d8acd4ee1

Просмотреть файл

@@ -833,11 +833,12 @@ func ReadFile(path string) ([]byte, *model.AppError) {
return nil, model.NewLocAppError("ReadFile", "api.file.read_file.s3.app_error", nil, err.Error()) return nil, model.NewLocAppError("ReadFile", "api.file.read_file.s3.app_error", nil, err.Error())
} }
bucket := utils.Cfg.FileSettings.AmazonS3Bucket bucket := utils.Cfg.FileSettings.AmazonS3Bucket
reader, err := s3Clnt.GetObject(bucket, path) minioObject, err := s3Clnt.GetObject(bucket, path)
defer minioObject.Close()
if err != nil { if err != nil {
return nil, model.NewLocAppError("ReadFile", "api.file.read_file.s3.app_error", nil, err.Error()) return nil, model.NewLocAppError("ReadFile", "api.file.read_file.s3.app_error", nil, err.Error())
} }
if f, err := ioutil.ReadAll(reader); err != nil { if f, err := ioutil.ReadAll(minioObject); err != nil {
return nil, model.NewLocAppError("ReadFile", "api.file.read_file.s3.app_error", nil, err.Error()) return nil, model.NewLocAppError("ReadFile", "api.file.read_file.s3.app_error", nil, err.Error())
} else { } else {
return f, nil return f, nil