Use %s together with fmt.Errorf

xyproto · xyproto · commit eafdd364b6c9 · 2025-06-26T19:52:51.000+02:00
diff --git a/lua/upload/upload.go b/lua/upload/upload.go
@@ -164,7 +164,7 @@ func (ulf *UploadedFile) write(fullFilename string, fperm os.FileMode) error {
 	// Check if the file already exists
 	if _, err := os.Stat(fullFilename); err == nil { // exists
 		logrus.Error(fullFilename, " already exists")
-		return fmt.Errorf("File exists: " + fullFilename)
+		return fmt.Errorf("File exists: %s", fullFilename)
 	}
 	// Write the uploaded file
 	f, err := os.OpenFile(fullFilename, os.O_WRONLY|os.O_CREATE, fperm)

Original file line number	Diff line number	Diff line change
`@@ -164,7 +164,7 @@ func (ulf *UploadedFile) write(fullFilename string, fperm os.FileMode) error {`
`164`	`164`	`// Check if the file already exists`
`165`	`165`	`if _, err := os.Stat(fullFilename); err == nil { // exists`
`166`	`166`	`logrus.Error(fullFilename, " already exists")`
`167`		`- return fmt.Errorf("File exists: " + fullFilename)`
	`167`	`+ return fmt.Errorf("File exists: %s", fullFilename)`
`168`	`168`	`}`
`169`	`169`	`// Write the uploaded file`
`170`	`170`	`f, err := os.OpenFile(fullFilename, os.O_WRONLY\|os.O_CREATE, fperm)`