improve error logs

SENERGY-Platform · Jan 22, 2025 · 2912714 · 2912714
1 parent 16a52a1
commit 2912714
Show file tree

Hide file tree

Showing 3 changed files with 15 additions and 2 deletions.
diff --git a/pkg/api/budgets.go b/pkg/api/budgets.go
@@ -40,6 +40,7 @@ func BudgetEndpoints(router *httprouter.Router, _ configuration.Config, control
 		}
 		limitInt, err := strconv.Atoi(limit)
 		if err != nil {
+			log.Println("ERROR: " + err.Error())
 			http.Error(writer, err.Error(), http.StatusBadRequest)
 			return
 		}
@@ -49,12 +50,14 @@ func BudgetEndpoints(router *httprouter.Router, _ configuration.Config, control
 		}
 		offsetInt, err := strconv.Atoi(offset)
 		if err != nil {
+			log.Println("ERROR: " + err.Error())
 			http.Error(writer, err.Error(), http.StatusBadRequest)
 			return
 		}
 
 		budgets, err := control.GetBudgets(limitInt, offsetInt, []string{}, "", "")
 		if err != nil {
+			log.Println("ERROR: " + err.Error())
 			http.Error(writer, err.Error(), http.StatusInternalServerError)
 			return
 		}
@@ -83,8 +86,10 @@ func BudgetEndpoints(router *httprouter.Router, _ configuration.Config, control
 		err := control.DeleteBudget(request.URL.Query().Get("budget_identifier"), request.URL.Query().Get("user_id"), request.URL.Query().Get("role"))
 		if err != nil {
 			if errors.Is(err, models.ErrorNotFound) {
+				log.Println("ERROR: " + err.Error())
 				http.Error(writer, err.Error(), http.StatusNotFound)
 			} else {
+				log.Println("ERROR: " + err.Error())
 				http.Error(writer, err.Error(), http.StatusInternalServerError)
 			}
 			return

diff --git a/pkg/api/flow-engine.go b/pkg/api/flow-engine.go
@@ -17,11 +17,13 @@
 package api
 
 import (
+	"log"
+	"net/http"
+
 	"github.com/SENERGY-Platform/budget/pkg/api/util"
 	"github.com/SENERGY-Platform/budget/pkg/configuration"
 	"github.com/SENERGY-Platform/budget/pkg/controller"
 	"github.com/julienschmidt/httprouter"
-	"net/http"
 )
 
 func init() {
@@ -32,10 +34,12 @@ func CheckFlowEngineEndpoints(router *httprouter.Router, c configuration.Config,
 	router.POST("/check/analytics/flow-engine", func(writer http.ResponseWriter, request *http.Request, params httprouter.Params) {
 		parsed, err := util.ParseRequest(request.Body, c.Debug)
 		if err != nil {
+			log.Println("ERROR: " + err.Error())
 			http.Error(writer, err.Error(), http.StatusBadRequest)
 		}
 		code, err := control.CheckFlowEngine(parsed)
 		if err != nil {
+			log.Println("ERROR: " + err.Error())
 			http.Error(writer, err.Error(), code)
 		}
 	})

diff --git a/pkg/api/import-deploy.go b/pkg/api/import-deploy.go
@@ -17,11 +17,13 @@
 package api
 
 import (
+	"log"
+	"net/http"
+
 	"github.com/SENERGY-Platform/budget/pkg/api/util"
 	"github.com/SENERGY-Platform/budget/pkg/configuration"
 	"github.com/SENERGY-Platform/budget/pkg/controller"
 	"github.com/julienschmidt/httprouter"
-	"net/http"
 )
 
 func init() {
@@ -32,11 +34,13 @@ func CheckImportDeployEndpoints(router *httprouter.Router, c configuration.Confi
 	router.POST("/check/import/deploy", func(writer http.ResponseWriter, request *http.Request, params httprouter.Params) {
 		parsed, err := util.ParseRequest(request.Body, c.Debug)
 		if err != nil {
+			log.Println("ERROR: " + err.Error())
 			http.Error(writer, err.Error(), http.StatusBadRequest)
 		}
 
 		code, err := control.CheckImportDeploy(parsed)
 		if err != nil {
+			log.Println("ERROR: " + err.Error())
 			http.Error(writer, err.Error(), code)
 		}
 	})