[Feat] #39 - 로그 Validation 추가

MOONSHOT-Team · Jan 9, 2024 · 631beec · 631beec
1 parent a91005a
commit 631beec
Show file tree

Hide file tree

Showing 4 changed files with 5 additions and 5 deletions.
diff --git a/src/main/java/org/moonshot/server/domain/log/controller/LogController.java b/src/main/java/org/moonshot/server/domain/log/controller/LogController.java
@@ -1,5 +1,6 @@
 package org.moonshot.server.domain.log.controller;
 
+import jakarta.validation.Valid;
 import lombok.RequiredArgsConstructor;
 import org.moonshot.server.domain.log.dto.request.LogCreateRequestDto;
 import org.moonshot.server.domain.log.service.LogService;
@@ -21,7 +22,7 @@ public class LogController {
     private final LogService logService;
 
     @PostMapping
-    public ApiResponse<?> create(Principal principal, @RequestBody LogCreateRequestDto logCreateRequestDto) {
+    public ApiResponse<?> create(Principal principal, @RequestBody @Valid LogCreateRequestDto logCreateRequestDto) {
         logService.createRecordLog(JwtTokenProvider.getUserIdFromPrincipal(principal), logCreateRequestDto);
         return ApiResponse.success(SuccessType.POST_LOG_SUCCESS);
     }

diff --git a/src/main/java/org/moonshot/server/global/common/model/validator/LimitValueValidator.java b/src/main/java/org/moonshot/server/global/common/model/validator/LimitValueValidator.java
@@ -3,7 +3,7 @@
 import jakarta.validation.ConstraintValidator;
 import jakarta.validation.ConstraintValidatorContext;
 
-public class LimitValueValidator implements ConstraintValidator<ValidTargetNumber, Long> {
+public class LimitValueValidator implements ConstraintValidator<ValidLimitValue, Long> {
 
     private final long MAX_LIMIT = 99999999999L;
 
@@ -12,7 +12,7 @@ public boolean isValid(Long value, ConstraintValidatorContext context) {
         if (value == null) {
             return true;
         } else {
-            return value <= MAX_LIMIT;
+            return value > 0 && value <= MAX_LIMIT;
         }
     }
 

diff --git a/src/main/java/org/moonshot/server/global/common/model/validator/TargetNumberValidator.java b/src/main/java/org/moonshot/server/global/common/model/validator/TargetNumberValidator.java
@@ -2,7 +2,6 @@
 
 import jakarta.validation.ConstraintValidator;
 import jakarta.validation.ConstraintValidatorContext;
-import lombok.extern.slf4j.Slf4j;
 
 public class TargetNumberValidator implements ConstraintValidator<ValidTargetNumber, Long> {
 

diff --git a/src/main/java/org/moonshot/server/global/common/model/validator/ValidLimitValue.java b/src/main/java/org/moonshot/server/global/common/model/validator/ValidLimitValue.java
@@ -9,7 +9,7 @@
 
 @Target(ElementType.FIELD)
 @Retention(RetentionPolicy.RUNTIME)
-@Constraint(validatedBy = TargetNumberValidator.class)
+@Constraint(validatedBy = LimitValueValidator.class)
 public @interface ValidLimitValue {
 
     String message() default "수치는 99,999,999,999까지만 가능합니다.";