🔀 Feature/#101 각종 유효성 처리 최종

🔀 Feature/#101 각종 유효성 처리 최종

Author: cheshireHYUN

src/main/java/com/boggle_boggle/bbegok/controller/RedisController.java

@@ -6,6 +6,7 @@
 import com.boggle_boggle.bbegok.dto.request.RecentSearchRequest;
 import com.boggle_boggle.bbegok.dto.response.SearchLogListResponse;
 import com.boggle_boggle.bbegok.service.SearchLogService;
+import jakarta.validation.Valid;
 import lombok.RequiredArgsConstructor;
 import org.springframework.security.core.annotation.AuthenticationPrincipal;
 import org.springframework.security.core.userdetails.UserDetails;
@@ -29,15 +30,15 @@ public DataResponseDto<List<String>> getRecentSearchLogs(@AuthenticationPrincipa
     //최근검색어 저장
     @PostMapping
     public DataResponseDto<Void> saveRecentSearchLogs(@AuthenticationPrincipal UserDetails userDetails,
-                                                        @RequestBody RecentSearchRequest request) {
+                                                       @Valid @RequestBody RecentSearchRequest request) {
         SearchLogService.saveRecentSearchLogs(userDetails.getUsername(), request.getKeyword());
         return DataResponseDto.empty();
     }
 
     //최근검색어 삭제
     @DeleteMapping
     public DataResponseDto<Void> deleteRecentSearchLogs(@AuthenticationPrincipal UserDetails userDetails,
-                                                      @RequestBody DeleteRecentSearchRequest request) {
+                                                      @Valid @RequestBody DeleteRecentSearchRequest request) {
         SearchLogService.deleteRecentSearchLog(userDetails.getUsername(), request.getKeyword());
         return DataResponseDto.empty();
     }

src/main/java/com/boggle_boggle/bbegok/dto/request/DeleteRecentSearchRequest.java

@@ -1,10 +1,14 @@
 package com.boggle_boggle.bbegok.dto.request;
 
+import jakarta.validation.constraints.NotBlank;
+import jakarta.validation.constraints.NotNull;
 import lombok.Getter;
 import lombok.ToString;
 
 @Getter
 @ToString
 public class DeleteRecentSearchRequest {
+    @NotNull
+    @NotBlank
     private String keyword;
 }

src/main/java/com/boggle_boggle/bbegok/dto/request/LibraryRequest.java

@@ -1,13 +1,15 @@
 package com.boggle_boggle.bbegok.dto.request;
 
 import jakarta.validation.constraints.NotBlank;
+import jakarta.validation.constraints.NotNull;
 import jakarta.validation.constraints.Size;
 import lombok.Getter;
 import lombok.Setter;
 
 @Getter
 public class LibraryRequest {
     @NotBlank
+    @NotNull
     @Size(max = 15)
     private String libraryName;
 }

src/main/java/com/boggle_boggle/bbegok/dto/request/NewNoteRequest.java

@@ -24,12 +24,13 @@ public class NewNoteRequest {
 
     private JsonNullable<LocalDateTime> selectedDate = JsonNullable.undefined();
 
-    @Size(max = 30)
+    @Size(max = 50)
     private JsonNullable<String> title = JsonNullable.undefined();
 
-    @Size(max = 255)
+    @Size(max = 1024)
     private JsonNullable<String> content = JsonNullable.undefined();
 
+    @Size(min = 1, max = 99999)
     private JsonNullable<Integer> page = JsonNullable.undefined();
 
     private JsonNullable<PagesDto> pages = JsonNullable.undefined();

src/main/java/com/boggle_boggle/bbegok/dto/request/NewReadingRecordRequest.java

@@ -1,6 +1,7 @@
 package com.boggle_boggle.bbegok.dto.request;
 
 import com.boggle_boggle.bbegok.enums.ReadStatus;
+import jakarta.validation.constraints.NotBlank;
 import jakarta.validation.constraints.NotNull;
 import lombok.Getter;
 import lombok.NoArgsConstructor;
@@ -15,7 +16,7 @@
 @NoArgsConstructor
 @ToString
 public class NewReadingRecordRequest {
-    @NotNull
+    @NotNull @NotBlank
     private String isbn;
     @NotNull
     private ReadStatus readStatus;

src/main/java/com/boggle_boggle/bbegok/dto/request/NickNameRequest.java

@@ -8,6 +8,7 @@
 @Getter
 public class NickNameRequest {
     @NotBlank
-    @Size(max = 20)
+    @NotNull
+    @Size(max = 12)
     private String nickname;
 }

src/main/java/com/boggle_boggle/bbegok/dto/request/RecentSearchRequest.java

@@ -9,7 +9,7 @@
 @Getter
 @ToString
 public class RecentSearchRequest {
-    @NotBlank
-    @Size(min = 1, max = 255)
+    @NotNull
+    @Size(min = 1, max = 100)
     private String keyword;
 }

src/main/java/com/boggle_boggle/bbegok/entity/Note.java

@@ -29,16 +29,16 @@ public class Note {
     private ReadingRecord readingRecord;
 
     @ManyToOne(fetch = FetchType.LAZY)
-    @JoinColumn(name = "read_date_seq", nullable = true)
+    @JoinColumn(name = "read_date_seq")
     private ReadDate readDate;
 
-    @Column(name = "title", length = 30)
+    @Column(name = "title", length = 50)
     private String title;
 
-    @Column(name = "content", length = 255)
+    @Column(name = "content", length = 1024)
     private String content;
 
-    @Column(name = "page", nullable = true)
+    @Column(name = "page")
     private Integer page;
 
     @Embedded

src/main/java/com/boggle_boggle/bbegok/entity/Terms.java

@@ -19,7 +19,7 @@ public class Terms {
     @Column(name = "title", nullable = false)
     private String title;
 
-    @Column(name = "content", length = 1000, nullable = false)
+    @Column(name = "content", nullable = false)
     private String content;
 
     @Column(name = "version", nullable = false)

src/main/java/com/boggle_boggle/bbegok/exception/handler/ExceptionHandler.java

@@ -6,7 +6,9 @@
 import jakarta.validation.ConstraintViolationException;
 import org.springframework.http.HttpHeaders;
 import org.springframework.http.HttpStatus;
+import org.springframework.http.HttpStatusCode;
 import org.springframework.http.ResponseEntity;
+import org.springframework.web.bind.MethodArgumentNotValidException;
 import org.springframework.web.bind.annotation.RestController;
 import org.springframework.web.bind.annotation.RestControllerAdvice;
 import org.springframework.web.context.request.WebRequest;
@@ -17,13 +19,19 @@
 @RestControllerAdvice(annotations = {RestController.class})
 public class ExceptionHandler extends ResponseEntityExceptionHandler {
 
-    /** 유효성 검사 실패 처리
-     */
+    /** 유효성 검사 실패 처리 (RequestBody - @Valid) */
+    @Override
+    protected ResponseEntity<Object> handleMethodArgumentNotValid(
+            MethodArgumentNotValidException ex, HttpHeaders headers,
+            HttpStatusCode status, WebRequest request) {
+        return handleExceptionInternal(ex, Code.VALIDATION_ERROR, request);
+    }
+
+    /** 유효성 검사 실패 처리 (RequestParam, PathVariable - @Validated) */
     @org.springframework.web.bind.annotation.ExceptionHandler(ConstraintViolationException.class)
     public ResponseEntity<Object> validation(ConstraintViolationException e, WebRequest request) {
         return handleExceptionInternal(e, Code.VALIDATION_ERROR, request);
     }
-
     /** 사용자 정의 예외 실패 처리
      */
     @org.springframework.web.bind.annotation.ExceptionHandler