Merge pull request #22 from TeamAiary/fix/modify-userapi-exception

fix: 유효성 검증 마무리 및 interceptor 예외 로직 추가

Author: BaeJunH0

src/main/java/com/aiary/be/auth/application/AuthService.java

@@ -21,6 +21,10 @@ public class AuthService {
 
     // 회원가입: 신규 유저 등록
     public void save(SignupRequest request) {
+        if(userRepository.findUserByEmail(request.email()).isPresent()){
+            throw CustomException.from(UserErrorCode.DUPLICATE_EMAIL);
+        }
+
         User newUser = new User(
             request.email(),
             request.password(),

src/main/java/com/aiary/be/global/exception/errorCode/UserErrorCode.java

@@ -1,12 +1,15 @@
 package com.aiary.be.global.exception.errorCode;
 
 import lombok.RequiredArgsConstructor;
+import org.springframework.boot.autoconfigure.graphql.GraphQlProperties.Http;
 import org.springframework.http.HttpStatus;
 
 @RequiredArgsConstructor
 public enum UserErrorCode implements ErrorCode {
     NOT_FOUND(HttpStatus.NOT_FOUND, "U001", "존재하지 않는 유저 아이디입니다."),
-    INVALID_EMAIL_PASSWORD(HttpStatus.UNAUTHORIZED, "U002", "잘못된 이메일, 비밀번호 조합입니다.");
+    INVALID_EMAIL_PASSWORD(HttpStatus.UNAUTHORIZED, "U002", "잘못된 이메일, 비밀번호 조합입니다."),
+    REQUIRED_LOGIN(HttpStatus.UNAUTHORIZED, "U003", "로그인이 필요합니다."),
+    DUPLICATE_EMAIL(HttpStatus.BAD_REQUEST,  "U004", "이미 존재하는 이메일입니다.");
 
     private final HttpStatus httpStatus;
     private final String code;

src/main/java/com/aiary/be/global/interceptor/LoginCheckInterceptor.java

@@ -1,6 +1,8 @@
 package com.aiary.be.global.interceptor;
 
 
+import com.aiary.be.global.exception.CustomException;
+import com.aiary.be.global.exception.errorCode.UserErrorCode;
 import com.aiary.be.user.presentation.dto.UserResponse;
 import jakarta.servlet.http.HttpServletRequest;
 import jakarta.servlet.http.HttpServletResponse;
@@ -24,22 +26,10 @@ public boolean preHandle(
 
         if(session==null || session.getAttribute("loggedInUser")==null){
             log.warn("인증되지 않은 사용자, url: {}", request.getRequestURI());
-            response.setStatus(HttpServletResponse.SC_UNAUTHORIZED);
-            response.setContentType("application/json;charset=UTF-8");
-            response.getWriter().write("{\"message\": \"로그인이 필요합니다.\"}");
 
-            return false;
-        }
-
-        // 이후 다른 페이지 인가 구현 예정
-//        if(request.getRequestURI().startsWith("/api/admin") &&
-//        !session.getAttribute("userRole").equals("ADMIN")){
-//            log.warn("일반 사용자는 이용할 수 없습니다.");
-//            ...
-//        }
+            throw CustomException.from(UserErrorCode.REQUIRED_LOGIN);
 
-        // controller에서 자주 사용할 만한 것들은 @RequestAttribute()로 받아서 사용할 수 있도록 전달할 수 있다.
-//        request.setAttribute("userEmail", session.getAttribute("userEmail"));
+        }
         UserResponse userResponse = (UserResponse) session.getAttribute("loggedInUser");
         request.setAttribute("userId", userResponse.userId());
         return true;

src/main/java/com/aiary/be/user/presentation/UserApiController.java

@@ -4,6 +4,7 @@
 import com.aiary.be.user.application.UserService;
 import com.aiary.be.user.presentation.dto.UserRequest;
 import com.aiary.be.user.presentation.dto.UserResponse;
+import jakarta.validation.Valid;
 import lombok.RequiredArgsConstructor;
 import org.springframework.http.HttpStatus;
 import org.springframework.http.ResponseEntity;
@@ -27,7 +28,7 @@ public ResponseEntity<?> readMyProfile(
     @PatchMapping
     public ResponseEntity<?> updateMyProfile(
         @RequestAttribute("userId") Long userId,
-        @RequestBody UserRequest userRequest
+        @Valid @RequestBody UserRequest userRequest
     ) {
         userService.updateUser(userId, userRequest);
 

src/main/java/com/aiary/be/user/presentation/dto/UserRequest.java

@@ -1,11 +1,13 @@
 package com.aiary.be.user.presentation.dto;
 
 import com.aiary.be.user.domain.Gender;
+import jakarta.validation.constraints.Email;
 
 // user 정보 update에만 사용되는 dto
 // 나중에 login/signup request 등이랑 통합 예정
 // 얘는 전달하지 않는 필드는 update하지 않기 때문에 valid를 추가하지 않았음
 public record UserRequest(
+    @Email(message = "올바른 이메일 형식이 아닙니다.")
     String email,
     String password,
     String userName,