From 38d3d2c4425a8ef9d4744ba49706032b2beec549 Mon Sep 17 00:00:00 2001 From: Thomas von Dein Date: Tue, 23 Dec 2025 14:30:37 +0100 Subject: [PATCH] fix liniting bugs --- rest/handlers.go | 13 +++++++++++-- rest/log.go | 2 -- rest/serve.go | 4 +--- 3 files changed, 12 insertions(+), 7 deletions(-) diff --git a/rest/handlers.go b/rest/handlers.go index 15361fc..aebb3f0 100644 --- a/rest/handlers.go +++ b/rest/handlers.go @@ -20,6 +20,7 @@ import ( //"github.com/alecthomas/repr" "encoding/json" + "log" "net/http" "codeberg.org/scip/anydb/app" @@ -62,12 +63,16 @@ func RestList(resp http.ResponseWriter, req *http.Request, conf *cfg.Config) { resp.Header().Set("Content-Type", "application/json") - json.NewEncoder(resp).Encode( + err = json.NewEncoder(resp).Encode( ListResponse{ Code: http.StatusOK, Success: true, Entries: entries, }) + + if err != nil { + log.Fatal(err) + } } func RestGet(resp http.ResponseWriter, req *http.Request, key string, conf *cfg.Config) { @@ -89,12 +94,16 @@ func RestGet(resp http.ResponseWriter, req *http.Request, key string, conf *cfg. resp.Header().Set("Content-Type", "application/json") - json.NewEncoder(resp).Encode( + err = json.NewEncoder(resp).Encode( SingleResponse{ Code: http.StatusOK, Success: true, Entry: entry, }) + + if err != nil { + log.Fatal(err) + } } func RestDelete(resp http.ResponseWriter, req *http.Request, key string, conf *cfg.Config) { diff --git a/rest/log.go b/rest/log.go index b0b37dc..577cc6a 100644 --- a/rest/log.go +++ b/rest/log.go @@ -45,8 +45,6 @@ func (rw *responseWriter) WriteHeader(code int) { rw.status = code rw.ResponseWriter.WriteHeader(code) rw.wroteHeader = true - - return } func (rw *responseWriter) Write(data []byte) (int, error) { diff --git a/rest/serve.go b/rest/serve.go index 78bb7eb..7070d3f 100644 --- a/rest/serve.go +++ b/rest/serve.go @@ -72,7 +72,7 @@ func Runserver(conf *cfg.Config, args []string) error { Wrapper to respond with proper json status, message and code, shall be prepared and called by the handlers directly. */ -func JsonStatus(resp http.ResponseWriter, code int, msg string) error { +func JsonStatus(resp http.ResponseWriter, code int, msg string) { success := code == http.StatusOK resp.Header().Set("Content-Type", "application/json") @@ -84,6 +84,4 @@ func JsonStatus(resp http.ResponseWriter, code int, msg string) error { Message: msg, Success: success, }) - - return nil }