CLAP-246 Refactor: 작업 목록 조회 valid 검증 설정

Author: parkjaehak

src/main/java/clap/server/adapter/inbound/web/task/FindTaskController.java

@@ -4,11 +4,14 @@
 import clap.server.adapter.inbound.web.dto.common.PageResponse;
 import clap.server.adapter.inbound.web.dto.task.request.FilterTaskListRequest;
 import clap.server.adapter.inbound.web.dto.task.response.*;
+import clap.server.adapter.outbound.persistense.entity.log.constant.LogStatus;
 import clap.server.application.port.inbound.task.FindTaskDetailsUsecase;
 import clap.server.application.port.inbound.task.FindTaskListUsecase;
 import clap.server.common.annotation.architecture.WebAdapter;
+import clap.server.common.annotation.log.LogType;
 import io.swagger.v3.oas.annotations.Operation;
 import io.swagger.v3.oas.annotations.tags.Tag;
+import jakarta.validation.Valid;
 import lombok.RequiredArgsConstructor;
 
 import org.springframework.data.domain.PageRequest;
@@ -28,12 +31,13 @@ public class FindTaskController {
     private final FindTaskListUsecase taskListUsecase;
 
     @Operation(summary = "사용자 요청 작업 목록 조회")
+    @LogType(LogStatus.TASK_VIEWED)
     @Secured({"ROLE_USER", "ROLE_MANAGER"})
     @GetMapping("/requests")
     public ResponseEntity<PageResponse<FilterRequestedTasksResponse>> findTasksRequestedByUser(
             @RequestParam(defaultValue = "0") int page,
             @RequestParam(defaultValue = "20") int pageSize,
-            @ModelAttribute FilterTaskListRequest filterTaskListRequest,
+            @ModelAttribute @Valid FilterTaskListRequest filterTaskListRequest,
             @AuthenticationPrincipal SecurityUserDetails userInfo){
         Pageable pageable = PageRequest.of(page, pageSize);
         return ResponseEntity.ok(taskListUsecase.findTasksRequestedByUser(userInfo.getUserId(), pageable, filterTaskListRequest));
@@ -54,7 +58,7 @@ public ResponseEntity<FindTaskDetailsResponse> findRequestedTaskDetails(
     public ResponseEntity<PageResponse<FilterAssignedTaskListResponse>> findTasksAssignedByManager(
             @RequestParam(defaultValue = "0") int page,
             @RequestParam(defaultValue = "20") int pageSize,
-            @ModelAttribute FilterTaskListRequest filterTaskListRequest,
+            @ModelAttribute @Valid FilterTaskListRequest filterTaskListRequest,
             @AuthenticationPrincipal SecurityUserDetails userInfo){
         Pageable pageable = PageRequest.of(page, pageSize);
         return ResponseEntity.ok(taskListUsecase.findTasksAssignedByManager(userInfo.getUserId(), pageable, filterTaskListRequest));
@@ -66,7 +70,7 @@ public ResponseEntity<PageResponse<FilterAssignedTaskListResponse>> findTasksAss
     public ResponseEntity<PageResponse<FilterPendingApprovalResponse>> findPendingApprovalTasks(
             @RequestParam(defaultValue = "0") int page,
             @RequestParam(defaultValue = "20") int pageSize,
-            @ModelAttribute FilterTaskListRequest filterTaskListRequest,
+            @ModelAttribute @Valid FilterTaskListRequest filterTaskListRequest,
             @AuthenticationPrincipal SecurityUserDetails userInfo){
         Pageable pageable = PageRequest.of(page, pageSize);
         return ResponseEntity.ok(taskListUsecase.findPendingApprovalTasks(userInfo.getUserId(), pageable, filterTaskListRequest));
@@ -78,7 +82,7 @@ public ResponseEntity<PageResponse<FilterPendingApprovalResponse>> findPendingAp
     public ResponseEntity<PageResponse<FilterAllTasksResponse>> findAllTasks(
             @RequestParam(defaultValue = "0") int page,
             @RequestParam(defaultValue = "20") int pageSize,
-            @ModelAttribute FilterTaskListRequest filterTaskListRequest,
+            @ModelAttribute @Valid FilterTaskListRequest filterTaskListRequest,
             @AuthenticationPrincipal SecurityUserDetails userInfo){
         Pageable pageable = PageRequest.of(page, pageSize);
         return ResponseEntity.ok(taskListUsecase.findAllTasks(userInfo.getUserId(), pageable, filterTaskListRequest));

src/main/java/clap/server/config/aop/LoggingAspect.java

@@ -12,6 +12,7 @@
 import jakarta.servlet.http.HttpServletResponse;
 import lombok.RequiredArgsConstructor;
 import lombok.extern.slf4j.Slf4j;
+import org.apache.http.HttpStatus;
 import org.aspectj.lang.ProceedingJoinPoint;
 import org.aspectj.lang.annotation.Around;
 import org.aspectj.lang.annotation.Aspect;
@@ -56,15 +57,12 @@ public Object logApiRequests(ProceedingJoinPoint joinPoint) throws Throwable {
             throw ex;
         } finally {
             LogStatus logStatus = getLogType((MethodSignature) joinPoint.getSignature());
-            int statusCode;
+            int statusCode = HttpStatus.SC_INTERNAL_SERVER_ERROR;
             String customCode = null;
             if (capturedException != null) {
                 if (capturedException instanceof BaseException e) {
                     statusCode = e.getCode().getHttpStatus().value();
                     customCode = e.getCode().getCustomCode();
-                } else {
-                    ModelAndView modelAndView = handlerExceptionResolver.resolveException(request, response, null, capturedException);
-                    statusCode = modelAndView.getStatus().value();
                 }
             } else {
                 statusCode = response.getStatus();