Specify the behavior for writing empty file

[Xuanwo]

Code like the following could hang forever (tested on go-service-s3)

_, err = fs.s.Write(path,nil, 0)
if err != nil {
    fs.logger.Error("write", zap.String("path", path), zap.Error(err))
    return nil, nil, err
}

I have to write it like:

_, err = fs.s.Write(path, bytes.NewReader([]byte{}), 0)
if err != nil {
    fs.logger.Error("write", zap.String("path", path), zap.Error(err))
    return nil, nil, err
}

Maybe we can support user input nil Reader for empty files?

In order to add this support, we need to:

• Add a new GSP to make it clear
• Update definitions to reflect changes
• Update integrations tests to make sure all service passed

[Xuanwo]

ping @beyondstorage/go-storage-committer @beyondstorage/go-storage-maintainer for a look

[Xuanwo]

Moved to https://forum.beyondstorage.io/t/specify-the-behavior-for-writing-empty-file/204

BeyondStorage

Specify the behavior for writing empty file

Code like the following could hang forever (tested on go-service-s3) _, err = fs.s.Write(path,nil, 0) if err != nil { fs.logger.Error("write", zap.String("path", path), zap.Error(err)) return nil, nil, err } I have to write it like: _, err = fs.s.Write(path, bytes.NewReader([]byte{}), 0) if err != nil { fs.logger.Error("write", zap.String("path", path), zap.Error(err)) return nil, nil, err } Maybe we can support user input nil Reader for empty files? In order to add this sup...