CLAP-246 Refactor: 로그 서비스 생성

Author: parkjaehak

src/main/java/clap/server/adapter/outbound/persistense/ApiLogPersistenceAdapter.java

@@ -16,6 +16,7 @@
 import lombok.RequiredArgsConstructor;
 import org.springframework.data.domain.Page;
 import org.springframework.data.domain.Pageable;
+import org.springframework.transaction.annotation.Transactional;
 
 
 @PersistenceAdapter
@@ -28,6 +29,7 @@ public class ApiLogPersistenceAdapter implements CommandLogPort, LoadLogPort {
     private final ApiLogPersistenceMapper apiLogPersistenceMapper;
     private final MemberPersistenceMapper memberPersistenceMapper;
 
+    @Transactional
     @Override
     public void saveMemberLog(MemberLog memberLog) {
         apiLogRepository.save(apiLogPersistenceMapper.mapMemberLogToEntity(memberLog, memberPersistenceMapper.toEntity(memberLog.getMember())));

…service/log/CreateMemberLogsService.java …tion/port/inbound/domain/LogService.javasrc/main/java/clap/server/application/service/log/CreateMemberLogsService.java renamed to src/main/java/clap/server/application/port/inbound/domain/LogService.java src/main/java/clap/server/application/service/log/CreateMemberLogsService.java renamed to src/main/java/clap/server/application/port/inbound/domain/LogService.java

@@ -1,29 +1,27 @@
-package clap.server.application.service.log;
+package clap.server.application.port.inbound.domain;
 
-import clap.server.adapter.outbound.persistense.ApiLogPersistenceAdapter;
 import clap.server.adapter.outbound.persistense.entity.log.constant.LogStatus;
-import clap.server.application.port.inbound.domain.MemberService;
-import clap.server.application.port.inbound.log.CreateMemberLogsUsecase;
 import clap.server.application.port.outbound.log.CommandLogPort;
-import clap.server.common.annotation.architecture.ApplicationService;
+import clap.server.domain.model.log.AnonymousLog;
 import clap.server.domain.model.log.MemberLog;
 import clap.server.domain.model.member.Member;
 import jakarta.servlet.http.HttpServletRequest;
-import jakarta.servlet.http.HttpServletResponse;
 import lombok.RequiredArgsConstructor;
+import org.springframework.stereotype.Service;
 import org.springframework.transaction.annotation.Transactional;
 
-import java.time.LocalDateTime;
-
-@ApplicationService
+@Service
 @RequiredArgsConstructor
-public class CreateMemberLogsService implements CreateMemberLogsUsecase {
-
+@Transactional
+public class LogService {
     private final CommandLogPort commandLogPort;
     private final MemberService memberService;
 
-    @Override
-    @Transactional
+    public void createAnonymousLog(HttpServletRequest request,  int statusCode, String customCode, LogStatus logStatus, Object responseBody, String requestBody, String nickName) {
+        AnonymousLog anonymousLog = AnonymousLog.createAnonymousLog(request, statusCode,customCode, logStatus, responseBody, requestBody, nickName);
+        commandLogPort.saveAnonymousLog(anonymousLog);
+    }
+
     public void createMemberLog(HttpServletRequest request, int statusCode, String customCode,LogStatus logStatus, Object responseBody, String requestBody, Long userId) {
         Member member = memberService.findById(userId);
         MemberLog memberLog = MemberLog.createMemberLog(request, statusCode, customCode, logStatus,  responseBody, requestBody, member);

src/main/java/clap/server/application/port/inbound/log/CreateAnonymousLogsUsecase.java

@@ -3,8 +3,7 @@
 import clap.server.adapter.inbound.security.service.SecurityUserDetails;
 
 import clap.server.adapter.outbound.persistense.entity.log.constant.LogStatus;
-import clap.server.application.port.inbound.log.CreateAnonymousLogsUsecase;
-import clap.server.application.port.inbound.log.CreateMemberLogsUsecase;
+import clap.server.application.port.inbound.domain.LogService;
 import clap.server.common.annotation.log.LogType;
 import clap.server.exception.BaseException;
 import com.fasterxml.jackson.databind.JsonNode;
@@ -35,9 +34,8 @@
 @RequiredArgsConstructor
 public class LoggingAspect {
     private final ObjectMapper objectMapper;
-    private final CreateAnonymousLogsUsecase createAnonymousLogsUsecase;
-    private final CreateMemberLogsUsecase createMemberLogsUsecase;
     private final HandlerExceptionResolver handlerExceptionResolver;
+    private final LogService logService;
 
     @Pointcut("execution(* clap.server.adapter.inbound.web..*Controller.*(..))")
     public void controllerMethods() {
@@ -74,14 +72,14 @@ public Object logApiRequests(ProceedingJoinPoint joinPoint) throws Throwable {
 
             if (logStatus != null) {
                 if (LogStatus.LOGIN.equals(logStatus)) {
-                    createAnonymousLogsUsecase.createAnonymousLog(request, statusCode, customCode, logStatus, result, getRequestBody(request), getNicknameFromRequestBody(request));
+                    logService.createAnonymousLog(request, statusCode, customCode, logStatus, result, getRequestBody(request), getNicknameFromRequestBody(request));
                 } else {
                     if (!isUserAuthenticated()) {
                         log.error("로그인 시도 로그를 기록할 수 없음");
                     } else {
                         Object principal = SecurityContextHolder.getContext().getAuthentication().getPrincipal();
                         if (principal instanceof SecurityUserDetails userDetails) {
-                            createMemberLogsUsecase.createMemberLog(request, statusCode, customCode, logStatus, result, getRequestBody(request), userDetails.getUserId());
+                            logService.createMemberLog(request, statusCode, customCode, logStatus, result, getRequestBody(request), userDetails.getUserId());
                         }
                     }
                 }