From 44959df6d8de284d897db4726eba583bb1bd2691 Mon Sep 17 00:00:00 2001
From: Alex Selimov <alex@alexselimov.com>
Date: Thu, 19 Mar 2026 15:07:53 -0400
Subject: [PATCH] Fix logging

---
 src/votes/handlers.rs | 27 +++++++++++++++++++++------
 1 file changed, 21 insertions(+), 6 deletions(-)

diff --git a/src/votes/handlers.rs b/src/votes/handlers.rs
index cd82ad7..3eeb713 100644
--- a/src/votes/handlers.rs
+++ b/src/votes/handlers.rs
@@ -6,6 +6,7 @@ use axum::{
     routing::{delete, get, post},
 };
 use axum_extra::extract::{CookieJar, cookie::Cookie};
+use tracing::{error, info};
 use uuid::Uuid;
 
 use crate::{
@@ -52,9 +53,12 @@ async fn upvote_handler(
     let (jar, voter_id) = get_or_init_voter_id(jar);
 
     match insert_new_vote(&slug, &voter_id, &state.db).await {
-        Ok(()) => (StatusCode::OK, jar, "Successfully upvoted"),
+        Ok(()) => {
+            info!(slug = %slug, voter_id = %voter_id, "upvoted successfully");
+            (StatusCode::OK, jar, "Successfully upvoted")
+        }
         Err(err) => {
-            println!("{err}");
+            error!(error = %err, slug = %slug, voter_id = %voter_id, "failed to insert vote");
             (StatusCode::INTERNAL_SERVER_ERROR, jar, "Failed to upvote")
         }
     }
@@ -68,9 +72,12 @@ async fn delete_vote_handler(
     let (jar, voter_id) = get_or_init_voter_id(jar);
 
     match delete_vote(&slug, &voter_id, &state.db).await {
-        Ok(()) => (StatusCode::OK, jar, "Successfully deleted vote"),
+        Ok(()) => {
+            info!(slug = %slug, voter_id = %voter_id, "vote deleted successfully");
+            (StatusCode::OK, jar, "Successfully deleted vote")
+        }
         Err(err) => {
-            println!("{err}");
+            error!(error = %err, slug = %slug, voter_id = %voter_id, "failed to delete vote");
             (
                 StatusCode::INTERNAL_SERVER_ERROR,
                 jar,
@@ -87,9 +94,17 @@ async fn get_votes_and_voted_handler(
 ) -> impl IntoResponse {
     let (jar, voter_id) = get_or_init_voter_id(jar);
     match get_votes_and_voted(&slug, &voter_id, &state.db).await {
-        Ok(votes_and_voted) => (StatusCode::OK, jar, Json(votes_and_voted)).into_response(),
+        Ok(votes_and_voted) => {
+            info!(slug = %slug, voter_id = %voter_id, "fetched vote count and voter status successfully");
+            (StatusCode::OK, jar, Json(votes_and_voted)).into_response()
+        }
         Err(err) => {
-            println!("{err}");
+            error!(
+                error = %err,
+                slug = %slug,
+                voter_id = %voter_id,
+                "failed to fetch vote count and voter status"
+            );
             (StatusCode::INTERNAL_SERVER_ERROR, jar, "Failed to upvote").into_response()
         }
     }