From 8b4bd4330618cb87596843defd8391d0af6abfaa Mon Sep 17 00:00:00 2001 From: Jakob Borg Date: Thu, 28 Jul 2022 17:35:43 +0200 Subject: [PATCH] lib/api: Missing error handling in config handler (#8463) --- lib/api/confighandler.go | 8 ++++++++ 1 file changed, 8 insertions(+) diff --git a/lib/api/confighandler.go b/lib/api/confighandler.go index 8e06354a9..f26ba1c86 100644 --- a/lib/api/confighandler.go +++ b/lib/api/confighandler.go @@ -63,6 +63,10 @@ func (c *configMuxBuilder) registerFolders(path string) { c.HandlerFunc(http.MethodPut, path, func(w http.ResponseWriter, r *http.Request) { data, err := unmarshalToRawMessages(r.Body) + if err != nil { + http.Error(w, err.Error(), http.StatusBadRequest) + return + } folders := make([]config.FolderConfiguration, len(data)) defaultFolder := c.cfg.DefaultFolder() for i, bs := range data { @@ -94,6 +98,10 @@ func (c *configMuxBuilder) registerDevices(path string) { c.HandlerFunc(http.MethodPut, path, func(w http.ResponseWriter, r *http.Request) { data, err := unmarshalToRawMessages(r.Body) + if err != nil { + http.Error(w, err.Error(), http.StatusBadRequest) + return + } devices := make([]config.DeviceConfiguration, len(data)) defaultDevice := c.cfg.DefaultDevice() for i, bs := range data {