fix: security context에서 꺼내오도록 수정 (#305)

* fix: security context에서 꺼내오도록 수정

* fix: gemini 수정 사항 적용

---------

Co-authored-by: 나용준 <141994188+youngJun99@users.noreply.github.com>

Author: yongjun0511

src/main/java/ssu/eatssu/global/log/ControllerLogAspect.java

@@ -9,6 +9,8 @@
 import org.aspectj.lang.annotation.Aspect;
 import org.aspectj.lang.annotation.Pointcut;
 import org.aspectj.lang.reflect.MethodSignature;
+import org.springframework.security.core.Authentication;
+import org.springframework.security.core.context.SecurityContextHolder;
 import org.springframework.security.core.userdetails.UserDetails;
 import org.springframework.stereotype.Component;
 import org.springframework.web.context.request.RequestContextHolder;
@@ -51,15 +53,7 @@ public Object logApi(ProceedingJoinPoint joinPoint) throws Throwable {
         String[] paramNames = methodSignature.getParameterNames();
         Object[] args = joinPoint.getArgs();
 
-        // 요청자
-        String userId = IntStream.range(0, args.length)
-                .filter(i -> args[i] instanceof CustomUserDetails)
-                .mapToObj(i -> {
-                    CustomUserDetails user = (CustomUserDetails) args[i];
-                    return String.valueOf(user.getId());
-                })
-                .findFirst()
-                .orElse("anonymous");
+        String userId = getUserIdFromSecurityContext();
 
         String userIdLog = "userId=" + userId;
 
@@ -127,6 +121,15 @@ private String getCauseMessage(Throwable e) {
         return message != null ? message : e.getClass().getSimpleName();
     }
 
+    private String getUserIdFromSecurityContext() {
+        Authentication authentication = SecurityContextHolder.getContext().getAuthentication();
+        if (authentication != null && authentication.isAuthenticated() && authentication.getPrincipal() instanceof CustomUserDetails) {
+            CustomUserDetails userDetails = (CustomUserDetails) authentication.getPrincipal();
+            return String.valueOf(userDetails.getId());
+        }
+        return "anonymous";
+    }
+
     private Map<String, Object> toSafeMap(Object arg) {
         Map<String, Object> result = new HashMap<>();
         for (Field field : arg.getClass().getDeclaredFields()) {