Log server errors as well as returning to http.

For easier debugging.

Author: Jean Barkhuysen

data/postgres/pg_wal/000000010000000000000001

@@ -37,27 +37,34 @@ type module struct {
 }
 
 func (s *server) handleIndex(w http.ResponseWriter, r *http.Request) {
+	slog.Info("received index request wtih query params ?since=%q and ?limit=%q", r.URL.Query().Get("since"), r.URL.Query().Get("limit"))
 	var since time.Time
 	var err error
 	if sinceParam := r.URL.Query().Get("since"); sinceParam != "" {
 		since, err = time.Parse(time.RFC3339, sinceParam)
 		if err != nil {
-			http.Error(w, fmt.Sprintf("error converting 'since' param %s: %v", sinceParam, err), http.StatusBadRequest)
+			errStr := fmt.Sprintf("error converting 'since' param %s: %v", sinceParam, err)
+			slog.Error(errStr)
+			http.Error(w, errStr, http.StatusBadRequest)
 			return
 		}
 	}
 
 	limit := defaultNumberOfOutputs
 	if limitParam := r.URL.Query().Get("limit"); limitParam != "" {
 		if limit, err = strconv.ParseInt(limitParam, 10, 64); err != nil {
-			http.Error(w, fmt.Sprintf("error converting 'limit' param %s: %v", limitParam, err), http.StatusBadRequest)
+			errStr := fmt.Sprintf("error converting 'limit' param %s: %v", limitParam, err)
+			slog.Error(errStr)
+			http.Error(w, errStr, http.StatusBadRequest)
 			return
 		}
 	}
 
 	repoTags, err := s.idb.FetchRepoTags(r.Context(), since, limit)
 	if err != nil {
-		http.Error(w, fmt.Sprintf("error fetching repo tags: %v", err), http.StatusInternalServerError)
+		errStr := fmt.Sprintf("error fetching repo tags: %v", err)
+		slog.Error(errStr)
+		http.Error(w, errStr, http.StatusInternalServerError)
 		return
 	}
 
@@ -69,15 +76,19 @@ func (s *server) handleIndex(w http.ResponseWriter, r *http.Request) {
 			Timestamp: rt.Created.Format(time.RFC3339),
 		})
 		if err != nil {
-			http.Error(w, fmt.Sprintf("error marshalling response for %v: %v", rt, err), http.StatusInternalServerError)
+			errStr := fmt.Sprintf("error marshalling response for %v: %v", rt, err)
+			slog.Error(errStr)
+			http.Error(w, errStr, http.StatusInternalServerError)
 			return
 		}
 
 		lines = append(lines, string(out))
 	}
 
 	if _, err := fmt.Fprint(w, strings.Join(lines, "\n")); err != nil {
-		http.Error(w, fmt.Sprintf("error writing response: %v", err), http.StatusInternalServerError)
+		errStr := fmt.Sprintf("error writing response: %v", err)
+		slog.Error(errStr)
+		http.Error(w, errStr, http.StatusInternalServerError)
 		return
 	}
 }