LogalDeveloper/owncast
LogalDeveloper/owncast
0
Fork 0
Code

Fix linter warnings

Browse Source
This commit is contained in:
Gabe Kangas 2022-03-07 16:37:04 -08:00
parent a7687c861e
commit b5d60d9f40
No known key found for this signature in database
GPG Key ID: 9A56337728BC81EA
2 changed files with 8 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
config/config.go
View File

@ -13,6 +13,7 @@ var DatabaseFilePath = "data/owncast.db"
// LogDirectory is the path to various log files. // LogDirectory is the path to various log files.
var LogDirectory = "./data/logs" var LogDirectory = "./data/logs"
// TempDir is where we store temporary files.
var TempDir = "./data/tmp" var TempDir = "./data/tmp"
// EnableDebugFeatures will print additional data to help in debugging. // EnableDebugFeatures will print additional data to help in debugging.

14
utils/utils.go
View File

@ -58,7 +58,7 @@ func Copy(source, destination string) error {
return err return err
} }
return os.WriteFile(destination, input, 0600) return os.WriteFile(destination, input, 0o600)
} }
// Move moves the file at source to destination. // Move moves the file at source to destination.
@ -74,18 +74,18 @@ func Move(source, destination string) error {
// moveFallback moves a file using a copy followed by a delete, which works across file systems. // moveFallback moves a file using a copy followed by a delete, which works across file systems.
// source: https://gist.github.com/var23rav/23ae5d0d4d830aff886c3c970b8f6c6b // source: https://gist.github.com/var23rav/23ae5d0d4d830aff886c3c970b8f6c6b
func moveFallback(source, destination string) error { func moveFallback(source, destination string) error {
inputFile, err := os.Open(source) inputFile, err := os.Open(source) // nolint: gosec
if err != nil { if err != nil {
return fmt.Errorf("Couldn't open source file: %s", err) return fmt.Errorf("Couldn't open source file: %s", err)
} }
outputFile, err := os.Create(destination) outputFile, err := os.Create(destination) // nolint: gosec
if err != nil { if err != nil {
inputFile.Close() _ = inputFile.Close()
return fmt.Errorf("Couldn't open dest file: %s", err) return fmt.Errorf("Couldn't open dest file: %s", err)
} }
defer outputFile.Close() defer outputFile.Close()
_, err = io.Copy(outputFile, inputFile) _, err = io.Copy(outputFile, inputFile)
inputFile.Close() _ = inputFile.Close()
if err != nil { if err != nil {
return fmt.Errorf("Writing to output file failed: %s", err) return fmt.Errorf("Writing to output file failed: %s", err)
} }
@ -289,7 +289,7 @@ func VerifyFFMpegPath(path string) error {
mode := stat.Mode() mode := stat.Mode()
// source: https://stackoverflow.com/a/60128480 // source: https://stackoverflow.com/a/60128480
if mode&0111 == 0 { if mode&0o111 == 0 {
return errors.New("ffmpeg path is not executable") return errors.New("ffmpeg path is not executable")
} }
@ -302,7 +302,7 @@ func CleanupDirectory(path string) {
if err := os.RemoveAll(path); err != nil { if err := os.RemoveAll(path); err != nil {
log.Fatalln("Unable to remove directory. Please check the ownership and permissions", err) log.Fatalln("Unable to remove directory. Please check the ownership and permissions", err)
} }
if err := os.MkdirAll(path, 0750); err != nil { if err := os.MkdirAll(path, 0o750); err != nil {
log.Fatalln("Unable to create directory. Please check the ownership and permissions", err) log.Fatalln("Unable to create directory. Please check the ownership and permissions", err)
} }
} }