Detached

internal/http-server/handlers/storage/ImportMethods.go

@@ -5,6 +5,7 @@ import (
 	"io"
 	"log"
 	"log/slog"
+	"mime/multipart"
 	"net/http"
 	"os"
 	"path/filepath"
@@ -34,7 +35,13 @@ func ImportUsersFromCSV(p *params.ParamsApiHandlers[ImportStorage]) error {
 	defer p.R.Body.Close()
 	key := "users"
 	file, _, err := p.R.FormFile(key)
-	defer file.Close()
+	defer func(file multipart.File) {
+		err := file.Close()
+		if err != nil {
+			p.Log.Error("Empty file", err.Error())
+			http.Error(p.W, fmt.Errorf("File is empty").Error(), http.StatusBadRequest)
+		}
+	}(file)
 	if err != nil {
 		p.Log.Error("Error Reading File", err.Error())
 		//http.Error(p.W, err.Error(), http.StatusBadRequest)
@@ -82,7 +89,13 @@ func ImportSegmentsFromCSV(p *params.ParamsApiHandlers[ImportStorage]) error {
 	defer p.R.Body.Close()
 	key := "segments"
 	file, _, err := p.R.FormFile(key)
-	defer file.Close()
+	defer func(file multipart.File) {
+		err := file.Close()
+		if err != nil {
+			p.Log.Error("Empty file", err.Error())
+			http.Error(p.W, fmt.Errorf("File is empty").Error(), http.StatusBadRequest)
+		}
+	}(file)
 	if err != nil {
 		p.Log.Error("Error Reading File", err.Error())
 		//http.Error(p.W, err.Error(), http.StatusBadRequest)
@@ -132,7 +145,13 @@ func ImportUserSegmentsFromCSV(p *params.ParamsApiHandlers[ImportStorage]) error
 	defer p.R.Body.Close()
 	key := "connections"
 	file, _, err := p.R.FormFile(key)
-	defer file.Close()
+	defer func(file multipart.File) {
+		err := file.Close()
+		if err != nil {
+			p.Log.Error("Empty file", err.Error())
+			http.Error(p.W, fmt.Errorf("File is empty").Error(), http.StatusBadRequest)
+		}
+	}(file)
 	if err != nil {
 		p.Log.Error("Error Reading File", err.Error())
 		//http.Error(p.W, err.Error(), http.StatusBadRequest)