From c49e191657aba4e0f06f64c311f3b4d07b59c6c6 Mon Sep 17 00:00:00 2001
From: Samuel Tariku <cybersamt@gmail.com>
Date: Fri, 12 Sep 2025 17:11:13 +0300
Subject: [PATCH] fix: change logger in veli handler

---
 internal/web_server/handlers/veli_games.go | 42 +++++++++++++++-------
 1 file changed, 30 insertions(+), 12 deletions(-)

diff --git a/internal/web_server/handlers/veli_games.go b/internal/web_server/handlers/veli_games.go
index b5f5f02..dbc57ea 100644
--- a/internal/web_server/handlers/veli_games.go
+++ b/internal/web_server/handlers/veli_games.go
@@ -4,12 +4,12 @@ import (
 	"context"
 	"errors"
 	"fmt"
-	"log"
 	"strings"
 
 	"github.com/SamuelTariku/FortuneBet-Backend/internal/domain"
 	"github.com/SamuelTariku/FortuneBet-Backend/internal/services/virtualGame/veli"
 	"github.com/gofiber/fiber/v2"
+	"go.uber.org/zap"
 )
 
 // GetProviders godoc
@@ -37,7 +37,10 @@ func (h *Handler) GetProviders(c *fiber.Ctx) error {
 	}
 	res, err := h.veliVirtualGameSvc.GetProviders(context.Background(), req)
 	if err != nil {
-		log.Println("GetProviders error:", err)
+		h.InternalServerErrorLogger().Error("Failed to [VeliGameHandler]GetProviders",
+			zap.Any("request", req),
+			zap.Error(err),
+		)
 		return c.Status(fiber.StatusBadGateway).JSON(domain.ErrorResponse{
 			Message: "Failed to retrieve providers",
 			Error:   err.Error(),
@@ -78,8 +81,10 @@ func (h *Handler) GetGamesByProvider(c *fiber.Ctx) error {
 
 	res, err := h.veliVirtualGameSvc.GetGames(context.Background(), req)
 	if err != nil {
-		log.Println("GetGames error:", err)
-
+		h.InternalServerErrorLogger().Error("Failed to [VeliGameHandler]GetGames",
+			zap.Any("request", req),
+			zap.Error(err),
+		)
 		// Handle provider disabled case specifically
 		if strings.Contains(err.Error(), "is disabled") {
 			return c.Status(fiber.StatusForbidden).JSON(domain.ErrorResponse{
@@ -89,7 +94,7 @@ func (h *Handler) GetGamesByProvider(c *fiber.Ctx) error {
 		}
 
 		// Fallback for other errors
-		return c.Status(fiber.StatusBadGateway).JSON(domain.ErrorResponse{
+		return c.Status(fiber.StatusInternalServerError).JSON(domain.ErrorResponse{
 			Message: "Failed to retrieve games",
 			Error:   err.Error(),
 		})
@@ -141,8 +146,10 @@ func (h *Handler) StartGame(c *fiber.Ctx) error {
 
 	res, err := h.veliVirtualGameSvc.StartGame(context.Background(), req)
 	if err != nil {
-		log.Println("StartGame error:", err)
-
+		h.InternalServerErrorLogger().Error("Failed to [VeliGameHandler]StartGame",
+			zap.Any("request", req),
+			zap.Error(err),
+		)
 		// Handle provider disabled case specifically
 		if strings.Contains(err.Error(), "is disabled") {
 			return c.Status(fiber.StatusForbidden).JSON(domain.ErrorResponse{
@@ -193,8 +200,10 @@ func (h *Handler) StartDemoGame(c *fiber.Ctx) error {
 
 	res, err := h.veliVirtualGameSvc.StartDemoGame(context.Background(), req)
 	if err != nil {
-		log.Println("StartDemoGame error:", err)
-
+		h.InternalServerErrorLogger().Error("Failed to [VeliGameHandler]StartDemoGame",
+			zap.Any("request", req),
+			zap.Error(err),
+		)
 		// Handle provider disabled case specifically
 		if strings.Contains(err.Error(), "is disabled") {
 			return c.Status(fiber.StatusForbidden).JSON(domain.ErrorResponse{
@@ -341,7 +350,10 @@ func (h *Handler) GetGamingActivity(c *fiber.Ctx) error {
 
 	resp, err := h.veliVirtualGameSvc.GetGamingActivity(c.Context(), req)
 	if err != nil {
-		log.Println("GetGamingActivity error:", err)
+		h.InternalServerErrorLogger().Error("Failed to [VeliGameHandler]GetGamingActivity",
+			zap.Any("request", req),
+			zap.Error(err),
+		)
 		return c.Status(fiber.StatusInternalServerError).JSON(domain.ErrorResponse{
 			Message: "Failed to retrieve gaming activity",
 			Error:   err.Error(),
@@ -378,7 +390,10 @@ func (h *Handler) GetHugeWins(c *fiber.Ctx) error {
 
 	resp, err := h.veliVirtualGameSvc.GetHugeWins(c.Context(), req)
 	if err != nil {
-		log.Println("GetHugeWins error:", err)
+		h.InternalServerErrorLogger().Error("Failed to [VeliGameHandler]GetHugeWins",
+			zap.Any("request", req),
+			zap.Error(err),
+		)
 		return c.Status(fiber.StatusInternalServerError).JSON(domain.ErrorResponse{
 			Message: "Failed to retrieve huge wins",
 			Error:   err.Error(),
@@ -416,7 +431,10 @@ func (h *Handler) GetCreditBalances(c *fiber.Ctx) error {
 
 	res, err := h.veliVirtualGameSvc.GetCreditBalances(c.Context(), brandID)
 	if err != nil {
-		log.Println("GetCreditBalances error:", err)
+		h.InternalServerErrorLogger().Error("Failed to [VeliGameHandler]GetCreditBalances",
+			zap.String("brandID", brandID),
+			zap.Error(err),
+		)
 		return c.Status(fiber.StatusBadGateway).JSON(domain.ErrorResponse{
 			Message: "Failed to fetch credit balances",
 			Error:   err.Error(),