-
-
Notifications
You must be signed in to change notification settings - Fork 1.7k
Open
Labels
defectSuspected defect such as a bug or regressionSuspected defect such as a bug or regression
Description
Observed behavior
There are multiple places in server/filestore.go where the file .Sync() call is used:
This call returns an error, or nil if no error: https://pkg.go.dev/os#File.Sync
However, this is not checked in the code, thus if the .Sync() call were to fail, then it would not be detected in the code and it would carry on.
Expected behavior
The code should check the return value and handle errors appropriately.
Server and client version
n/a - in current main branch
Host environment
n/a - in current main branch
Steps to reproduce
Examine the source code and compare to the go documentation.
ligurio and stvgo
Metadata
Metadata
Assignees
Labels
defectSuspected defect such as a bug or regressionSuspected defect such as a bug or regression