From 0e150e9ded2fe995f949b59a0761ec074490be23 Mon Sep 17 00:00:00 2001
From: Constantin Simonis <constantin@simonis.lol>
Date: Wed, 21 May 2025 10:54:25 +0200
Subject: [PATCH] refactor(GitHubService): remove unnecessary logging
 statements

---
 .../szut/casino/security/GitHubService.java   | 19 ++-----------------
 1 file changed, 2 insertions(+), 17 deletions(-)

diff --git a/backend/src/main/java/de/szut/casino/security/GitHubService.java b/backend/src/main/java/de/szut/casino/security/GitHubService.java
index 9ef8274..a345825 100644
--- a/backend/src/main/java/de/szut/casino/security/GitHubService.java
+++ b/backend/src/main/java/de/szut/casino/security/GitHubService.java
@@ -25,8 +25,6 @@ import java.util.*;
 
 @Service
 public class GitHubService {
-    private static final Logger logger = LoggerFactory.getLogger(GitHubService.class);
-
     @Value("${spring.security.oauth2.client.registration.github.client-id}")
     private String clientId;
 
@@ -71,32 +69,27 @@ public class GitHubService {
             );
 
             Map<String, Object> responseBody = response.getBody();
-            logger.info("GitHub token response: {}", responseBody);
 
             // Check if there's an error in the response
             if (responseBody.containsKey("error")) {
                 String error = (String) responseBody.get("error");
                 String errorDescription = (String) responseBody.get("error_description");
-                logger.error("GitHub OAuth error: {} - {}", error, errorDescription);
+
                 throw new RuntimeException("GitHub OAuth error: " + errorDescription);
             }
 
             String accessToken = (String) responseBody.get("access_token");
             if (accessToken == null || accessToken.isEmpty()) {
-                logger.error("No access token received from GitHub");
+
                 throw new RuntimeException("Failed to receive access token from GitHub");
             }
 
-            logger.info("Received access token from GitHub");
-
             // Get user info
             HttpHeaders userInfoHeaders = new HttpHeaders();
             userInfoHeaders.set("Authorization", "Bearer " + accessToken);
 
             HttpEntity<String> userInfoRequestEntity = new HttpEntity<>(null, userInfoHeaders);
 
-            logger.info("Making request to GitHub API with token: {}", accessToken.substring(0, 5) + "...");
-
             ResponseEntity<Map> userResponse = restTemplate.exchange(
                     "https://api.github.com/user",
                     HttpMethod.GET,
@@ -105,7 +98,6 @@ public class GitHubService {
             );
 
             Map<String, Object> userAttributes = userResponse.getBody();
-            logger.info("Retrieved user info from GitHub: {}", userAttributes.get("login"));
 
             // Get user emails
             HttpHeaders emailsHeaders = new HttpHeaders();
@@ -137,8 +129,6 @@ public class GitHubService {
                 email = (String) emails.get(0).get("email");
             }
 
-            logger.info("Using email: {}", email);
-
             // Process user data
             String githubId = userAttributes.get("id").toString();
             String username = (String) userAttributes.get("login");
@@ -150,7 +140,6 @@ public class GitHubService {
             if (userOptional.isPresent()) {
                 // Update existing user
                 user = userOptional.get();
-                logger.info("Found existing user with providerId: {}", githubId);
             } else {
                 // Check if email exists
                 userOptional = userRepository.findByEmail(email);
@@ -159,7 +148,6 @@ public class GitHubService {
                     user = userOptional.get();
                     user.setProvider(AuthProvider.GITHUB);
                     user.setProviderId(githubId);
-                    logger.info("Updating existing user with email: {}", email);
                 } else {
                     // Create new user
                     user = new UserEntity();
@@ -170,7 +158,6 @@ public class GitHubService {
                     user.setEmailVerified(true);
 
                     user.setBalance(new BigDecimal("1000.00"));
-                    logger.info("Creating new user for: {}", username);
                 }
             }
 
@@ -183,12 +170,10 @@ public class GitHubService {
 
             // Generate JWT token
             String token = jwtUtils.generateToken(authentication);
-            logger.info("Generated JWT token");
 
             return new AuthResponseDto(token);
 
         } catch (Exception e) {
-            logger.error("Error processing GitHub code", e);
             throw new RuntimeException("Failed to process GitHub authentication", e);
         }
     }