[chore] 유저 인증 관련 로직을 Auth 폴더로 분리 #83
Conversation
[chore] Token 관련 로직을 Auth 폴더로 분리
Walkthrough인증 관련 엔드포인트가 기존 Changes
Sequence Diagram(s)sequenceDiagram
participant Client
participant AuthController
participant AuthService
participant KakaoLoginService
participant SecurityContext
Client->>AuthController: POST /api/v1/auth/login/basic (email, pw)
AuthController->>AuthService: login(request)
AuthService-->>AuthController: TokenResponseDto
AuthController-->>Client: CommonResponse<TokenResponseDto>
Client->>AuthController: POST /api/v1/auth/login/kakao (kakaoToken)
AuthController->>KakaoLoginService: kakaoLogin(request)
KakaoLoginService-->>AuthController: KakaoLoginResponseDto
AuthController-->>Client: CommonResponse<KakaoLoginResponseDto>
Client->>AuthController: POST /api/v1/auth/reissue (refreshToken)
AuthController->>AuthService: reissue(refreshToken)
AuthService-->>AuthController: TokenResponseDto
AuthController-->>Client: CommonResponse<TokenResponseDto>
Client->>AuthController: POST /api/v1/auth/logout (Authorization: Bearer)
AuthController->>SecurityContext: getUserIdFromContext()
AuthController->>AuthService: logout(userId)
AuthService-->>AuthController: Success message
AuthController-->>Client: CommonResponse<String>
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~20 minutes Possibly related PRs
Poem
Note ⚡️ Unit Test Generation is now available in beta!Learn more here, or try it out under "Finishing Touches" below. 📜 Recent review detailsConfiguration used: CodeRabbit UI 📒 Files selected for processing (3)
✅ Files skipped from review due to trivial changes (2)
🚧 Files skipped from review as they are similar to previous changes (1)
✨ Finishing Touches
🧪 Generate unit tests
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
SupportNeed help? Create a ticket on our support page for assistance with any issues or questions. Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 3
🔭 Outside diff range comments (3)
src/main/java/com/wayble/server/auth/service/KakaoLoginService.java (2)
32-42: 카카오에서 이메일을 반환하지 않는 경우 NPE 위험카카오 계정에서 이메일 제공에 동의하지 않은 사용자는
kakaoUserInfo.getKakaoAccount().getEmail()이null이 될 수 있습니다.
null 체크 없이는 이후findByEmailAndLoginType호출에서NullPointerException이 발생합니다.
방어 코드를 추가하거나, 이메일이 없을 때 다른 식별자를 사용하도록 로직을 보완해야 합니다.
62-70: 리프레시 토큰을 DB에 저장하지 않아 재발급 API가 동작하지 않음
AuthService.reissue()는RefreshTokenRepository를 통해 DB 에 저장된 토큰을 조회합니다.
그러나 카카오 로그인 흐름에서는 발급된refreshToken을 저장하지 않아 재발급 및 로그아웃 로직이 실패합니다.
refreshTokenRepository.save(...)로 저장하는 처리를 추가해 주세요.String refreshToken = jwtProvider.generateRefreshToken(user.getId()); Long expiry = jwtProvider.getTokenExpiry(refreshToken); +refreshTokenRepository.deleteByUserId(user.getId()); +refreshTokenRepository.save( + RefreshToken.builder() + .userId(user.getId()) + .token(refreshToken) + .expiry(expiry) + .build());src/main/java/com/wayble/server/common/config/SecurityConfig.java (1)
37-41: 이전 인증 엔드포인트 경로를 제거해주세요.인증 관련 엔드포인트가
/api/v1/auth/**로 이동했으므로, 더 이상 사용하지 않는 경로들을 제거해야 합니다./api/v1/users/signup은 여전히 UserController에서 처리하므로 유지해야 합니다.다음과 같이 수정해주세요:
.requestMatchers( "/api/v1/users/signup", - "/api/v1/users/login", - "/api/v1/users/reissue", - "/api/v1/users/logout", "/api/v1/auth/**",
♻️ Duplicate comments (2)
src/main/java/com/wayble/server/auth/repository/RefreshTokenRepository.java (1)
3-3:enttiy패키지명 오타 재사용앞서 지적한 패키지 오타가 여기서도 반복됩니다. 동일하게
entity로 수정해야 합니다.
이전 코멘트와 중복되어 추가 설명은 생략합니다.src/main/java/com/wayble/server/auth/service/AuthService.java (1)
7-8: 패키지 오타로 인한 임포트 오류
com.wayble.server.auth.enttiy.RefreshToken→com.wayble.server.auth.entity.RefreshToken로 수정 필요.
앞서 동일 사안에 대해 코멘트가 있으므로 중복 표기로 처리합니다.
🧹 Nitpick comments (1)
src/main/java/com/wayble/server/user/controller/UserController.java (1)
12-12: 사용하지 않는 import를 제거하세요.
AuthService는 이 컨트롤러에서 사용되지 않으므로 import를 제거해야 합니다.-import com.wayble.server.auth.service.AuthService;
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (9)
src/main/java/com/wayble/server/auth/controller/AuthController.java(1 hunks)src/main/java/com/wayble/server/auth/dto/TokenResponseDto.java(1 hunks)src/main/java/com/wayble/server/auth/enttiy/RefreshToken.java(1 hunks)src/main/java/com/wayble/server/auth/repository/RefreshTokenRepository.java(1 hunks)src/main/java/com/wayble/server/auth/service/AuthService.java(1 hunks)src/main/java/com/wayble/server/auth/service/KakaoLoginService.java(1 hunks)src/main/java/com/wayble/server/common/config/SecurityConfig.java(1 hunks)src/main/java/com/wayble/server/user/controller/KakaoLoginController.java(0 hunks)src/main/java/com/wayble/server/user/controller/UserController.java(2 hunks)
💤 Files with no reviewable changes (1)
- src/main/java/com/wayble/server/user/controller/KakaoLoginController.java
🧰 Additional context used
🧠 Learnings (4)
src/main/java/com/wayble/server/auth/enttiy/RefreshToken.java (1)
Learnt from: seung-in-Yoo
PR: #37
File: src/main/java/com/wayble/server/user/entity/User.java:61-62
Timestamp: 2025-07-13T15:08:59.318Z
Learning: Java에서 같은 패키지 내의 클래스들은 import 구문 없이 서로를 참조할 수 있다. com.wayble.server.user.entity 패키지 내의 클래스들은 명시적인 import 없이 사용 가능하다.
src/main/java/com/wayble/server/auth/service/AuthService.java (1)
Learnt from: seung-in-Yoo
PR: #37
File: src/main/java/com/wayble/server/user/entity/User.java:61-62
Timestamp: 2025-07-13T15:08:59.318Z
Learning: Java에서 같은 패키지 내의 클래스들은 import 구문 없이 서로를 참조할 수 있다. com.wayble.server.user.entity 패키지 내의 클래스들은 명시적인 import 없이 사용 가능하다.
src/main/java/com/wayble/server/auth/repository/RefreshTokenRepository.java (1)
Learnt from: seung-in-Yoo
PR: #37
File: src/main/java/com/wayble/server/user/entity/User.java:61-62
Timestamp: 2025-07-13T15:08:59.318Z
Learning: Java에서 같은 패키지 내의 클래스들은 import 구문 없이 서로를 참조할 수 있다. com.wayble.server.user.entity 패키지 내의 클래스들은 명시적인 import 없이 사용 가능하다.
src/main/java/com/wayble/server/auth/service/KakaoLoginService.java (1)
Learnt from: seung-in-Yoo
PR: #37
File: src/main/java/com/wayble/server/user/entity/User.java:61-62
Timestamp: 2025-07-13T15:08:59.318Z
Learning: Java에서 같은 패키지 내의 클래스들은 import 구문 없이 서로를 참조할 수 있다. com.wayble.server.user.entity 패키지 내의 클래스들은 명시적인 import 없이 사용 가능하다.
🧬 Code Graph Analysis (1)
src/main/java/com/wayble/server/auth/controller/AuthController.java (2)
src/main/java/com/wayble/server/common/config/security/jwt/JwtAuthenticationFilter.java (1)
RequiredArgsConstructor(13-43)src/main/java/com/wayble/server/user/controller/UserController.java (1)
RestController(25-94)
🔇 Additional comments (1)
src/main/java/com/wayble/server/auth/dto/TokenResponseDto.java (1)
1-3: 변경 사항 이상 없음패키지 이동만 이루어졌으며 레코드 정의는 간결하고 문제 없습니다.
| import com.wayble.server.user.dto.KakaoLoginRequestDto; | ||
| import com.wayble.server.user.dto.KakaoLoginResponseDto; | ||
| import com.wayble.server.user.dto.UserLoginRequestDto; | ||
| import com.wayble.server.auth.dto.TokenResponseDto; | ||
| import com.wayble.server.user.exception.UserErrorCase; |
There was a problem hiding this comment.
🛠️ Refactor suggestion
인증 관련 DTO와 에러 케이스를 auth 패키지로 이동하세요.
인증 로직을 Auth 폴더로 분리하는 PR 목적에 맞게, 인증 관련 DTO들(KakaoLoginRequestDto, KakaoLoginResponseDto, UserLoginRequestDto)과 UserErrorCase도 auth 패키지로 이동해야 합니다. 현재 user 패키지에서 import하고 있어 패키지 간 의존성이 깔끔하게 분리되지 않았습니다.
다음 클래스들을 auth 패키지로 이동하시겠습니까?
com.wayble.server.user.dto.KakaoLoginRequestDto→com.wayble.server.auth.dto.KakaoLoginRequestDtocom.wayble.server.user.dto.KakaoLoginResponseDto→com.wayble.server.auth.dto.KakaoLoginResponseDtocom.wayble.server.user.dto.UserLoginRequestDto→com.wayble.server.auth.dto.UserLoginRequestDtocom.wayble.server.user.exception.UserErrorCase→com.wayble.server.auth.exception.AuthErrorCase
이 작업을 위한 이슈를 생성해드릴까요?
🤖 Prompt for AI Agents
In src/main/java/com/wayble/server/auth/controller/AuthController.java around
lines 7 to 11, the imports for authentication-related DTOs and error cases are
still from the user package, causing unclear package dependencies. Move the
classes KakaoLoginRequestDto, KakaoLoginResponseDto, UserLoginRequestDto from
com.wayble.server.user.dto to com.wayble.server.auth.dto, and move UserErrorCase
from com.wayble.server.user.exception to
com.wayble.server.auth.exception.AuthErrorCase. Then update the import
statements in AuthController.java to reflect these new package locations.
| if (accessToken == null || !accessToken.startsWith("Bearer ")) { | ||
| throw new ApplicationException(UserErrorCase.INVALID_CREDENTIALS); | ||
| } | ||
| String token = accessToken.replace("Bearer ", ""); | ||
| if (token.isEmpty()) { | ||
| throw new ApplicationException(UserErrorCase.INVALID_CREDENTIALS); | ||
| } | ||
| Long userId = (Long) SecurityContextHolder.getContext().getAuthentication().getPrincipal(); | ||
| authService.logout(userId); |
There was a problem hiding this comment.
🛠️ Refactor suggestion
불필요한 토큰 검증 로직을 제거하고 NPE 방지 로직을 추가하세요.
현재 구현에는 두 가지 문제가 있습니다:
JwtAuthenticationFilter에서 이미 토큰 검증을 수행하므로 중복 검증입니다.- 인증이 실패한 경우
SecurityContextHolder.getContext().getAuthentication()이 null일 수 있어 NPE가 발생할 수 있습니다.
다음과 같이 수정하세요:
- public CommonResponse<String> logout(
- @Parameter(
- description = "사용자 인증용 accessToken (Bearer {token} 형태)",
- required = true,
- example = "Bearer eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9..."
- )
- @RequestHeader("Authorization") String accessToken
- ) {
- if (accessToken == null || !accessToken.startsWith("Bearer ")) {
- throw new ApplicationException(UserErrorCase.INVALID_CREDENTIALS);
- }
- String token = accessToken.replace("Bearer ", "");
- if (token.isEmpty()) {
- throw new ApplicationException(UserErrorCase.INVALID_CREDENTIALS);
- }
- Long userId = (Long) SecurityContextHolder.getContext().getAuthentication().getPrincipal();
+ public CommonResponse<String> logout() {
+ Authentication authentication = SecurityContextHolder.getContext().getAuthentication();
+ if (authentication == null || !(authentication.getPrincipal() instanceof Long userId)) {
+ throw new ApplicationException(UserErrorCase.UNAUTHORIZED);
+ }
authService.logout(userId);
return CommonResponse.success("로그아웃에 성공하였습니다.");
}Committable suggestion skipped: line range outside the PR's diff.
🤖 Prompt for AI Agents
In src/main/java/com/wayble/server/auth/controller/AuthController.java around
lines 100 to 108, remove the redundant token validation logic since
JwtAuthenticationFilter already handles it. Instead, add a null check for
SecurityContextHolder.getContext().getAuthentication() before accessing its
principal to prevent NullPointerException. If authentication is null, handle it
appropriately, such as throwing an ApplicationException with
INVALID_CREDENTIALS.
| @@ -1,4 +1,4 @@ | |||
| package com.wayble.server.user.entity; | |||
| package com.wayble.server.auth.enttiy; | |||
There was a problem hiding this comment.
패키지명 오타로 인한 컴파일 실패 가능성
enttiy → entity 로 수정하지 않으면 동일 패키지를 import 하는 모든 클래스가 컴파일 오류가 발생합니다.
또한 실제 디렉터리 구조도 함께 수정되어야 합니다.
-package com.wayble.server.auth.enttiy;
+package com.wayble.server.auth.entity;📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
| package com.wayble.server.auth.enttiy; | |
| package com.wayble.server.auth.entity; |
🤖 Prompt for AI Agents
In src/main/java/com/wayble/server/auth/enttiy/RefreshToken.java at line 1, the
package name is misspelled as "enttiy" instead of "entity". Correct the package
declaration to "com.wayble.server.auth.entity" and rename the directory from
"enttiy" to "entity" to match the corrected package name, ensuring all imports
and references are updated accordingly to prevent compilation errors.
seung-in-Yoo
left a comment
seung-in-Yoo
left a comment
There was a problem hiding this comment.
RefreshToken 파일 들어있는 폴더 이름이 enttiy로 되어있는 곳만 수정하면 될것 같네요 수고하셨습니다!
| @@ -1,4 +1,4 @@ | |||
| package com.wayble.server.user.entity; | |||
| package com.wayble.server.auth.enttiy; | |||
There was a problem hiding this comment.
요기 코드래빗 말처럼 entity로 수정해야 하지 않을까요??
| package com.wayble.server.auth.repository; | ||
|
|
||
| import com.wayble.server.user.entity.RefreshToken; | ||
| import com.wayble.server.auth.enttiy.RefreshToken; |
✔️ 연관 이슈
📝 작업 내용
유저 인증 관련 로직을 Auth 폴더로 분리
스크린샷 (선택)
Summary by CodeRabbit
신규 기능
버그 수정
리팩터링
보안