From ad1405bb12411dbc1948bf4053dcd493b1f8b0fa Mon Sep 17 00:00:00 2001
From: Nicola Zangrandi <wasp@wasp.ovh>
Date: Fri, 21 Feb 2025 08:09:53 +0100
Subject: [PATCH] refactor(go): clean up logging to remove debug statements

---
 main.go | 12 +++++-------
 1 file changed, 5 insertions(+), 7 deletions(-)

diff --git a/main.go b/main.go
index e06a0bb..fcc3285 100644
--- a/main.go
+++ b/main.go
@@ -66,9 +66,6 @@ func serveStaticFile(w http.ResponseWriter, r *http.Request, prefix string) erro
 	w.Header().Set("X-Content-Type-Options", "nosniff")
 	w.Header().Set("Content-Type", mimeType)
 
-	// Log the file being served and its MIME type
-	log.Printf("DEBUG: Serving %s with MIME type %s", cleanPath, mimeType)
-
 	w.Write(content)
 	return nil
 }
@@ -133,7 +130,6 @@ func main() {
 }
 
 func handleNotes(w http.ResponseWriter, r *http.Request) {
-	log.Printf("INFO: %s request to %s", r.Method, r.URL.Path)
 	switch r.Method {
 	case "GET":
 		rows, err := db.Query(`
@@ -177,6 +173,7 @@ func handleNotes(w http.ResponseWriter, r *http.Request) {
 			VALUES (?, ?, ?, ?, ?)
 			`, note.ID, note.Title, note.Content, note.CreatedAt, note.UpdatedAt)
 		if err != nil {
+			log.Printf("ERROR: Failed to create note: %v", err)
 			http.Error(w, err.Error(), http.StatusInternalServerError)
 			return
 		}
@@ -190,7 +187,6 @@ func handleNotes(w http.ResponseWriter, r *http.Request) {
 
 func handleNote(w http.ResponseWriter, r *http.Request) {
 	id := path.Base(r.URL.Path)
-	log.Printf("INFO: %s request to %s for note ID %s", r.Method, r.URL.Path, id)
 
 	switch r.Method {
 	case "GET":
@@ -201,11 +197,11 @@ func handleNote(w http.ResponseWriter, r *http.Request) {
       WHERE id = ?
       `, id).Scan(&note.ID, &note.Title, &note.Content, &note.CreatedAt, &note.UpdatedAt)
 		if err == sql.ErrNoRows {
-			log.Printf("INFO: Note not found with ID %s", id)
 			http.NotFound(w, r)
 			return
 		}
 		if err != nil {
+			log.Printf("ERROR: Failed to get note %s: %v", id, err)
 			http.Error(w, err.Error(), http.StatusInternalServerError)
 			return
 		}
@@ -214,6 +210,7 @@ func handleNote(w http.ResponseWriter, r *http.Request) {
 	case "PUT":
 		var note Note
 		if err := json.NewDecoder(r.Body).Decode(&note); err != nil {
+			log.Printf("ERROR: Failed to decode note update: %v", err)
 			http.Error(w, err.Error(), http.StatusBadRequest)
 			return
 		}
@@ -224,6 +221,7 @@ func handleNote(w http.ResponseWriter, r *http.Request) {
       WHERE id = ?
       `, note.Title, note.Content, note.UpdatedAt, id)
 		if err != nil {
+			log.Printf("ERROR: Failed to update note %s: %v", id, err)
 			http.Error(w, err.Error(), http.StatusInternalServerError)
 			return
 		}
@@ -231,6 +229,7 @@ func handleNote(w http.ResponseWriter, r *http.Request) {
 	case "DELETE":
 		_, err := db.Exec("DELETE FROM notes WHERE id = ?", id)
 		if err != nil {
+			log.Printf("ERROR: Failed to delete note %s: %v", id, err)
 			http.Error(w, err.Error(), http.StatusInternalServerError)
 			return
 		}
@@ -239,7 +238,6 @@ func handleNote(w http.ResponseWriter, r *http.Request) {
 }
 
 func handleFrontend(w http.ResponseWriter, r *http.Request) {
-	log.Printf("INFO: Serving frontend request for %s", r.URL.Path)
 	// Don't serve API routes
 	if path.Dir(r.URL.Path) == "/api" {
 		http.NotFound(w, r)