[REFACTOR] 회원 및 인증 도메인 유즈케이스별 구조 개선#14
Conversation
There was a problem hiding this comment.
Pull Request Overview
This PR refactors the authentication and member-related modules by splitting responsibilities into distinct classes and packages, and by marking legacy code as deprecated. Key changes include package renaming for authentication classes, migration of RSA and verification code logic to the auth domain, and the introduction of refactored implementations for Kakao and JWT authentication.
Reviewed Changes
Copilot reviewed 53 out of 53 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
| src/main/java/com/example/spot/auth/presentation/controller/legacy/AuthController.java | Updated package/import paths and added deprecation to legacy controller. |
| src/main/java/com/example/spot/auth/infrastructure/kakao/KakaoOAuthClient.java | Applied package renaming and updated import paths for Kakao OAuth support. |
| src/main/java/com/example/spot/auth/domain/* | Updated various domain classes to the auth package. |
| src/main/java/com/example/spot/auth/application/refactor/* | Introduced refactored implementations for authentication services supporting Kakao and JWT. |
| src/main/java/com/example/spot/auth/application/legacy/* | Marked legacy authentication services as deprecated and updated package/imports accordingly. |
|
|
||
| // 응답에서 액세스 토큰을 파싱 | ||
| KaKaoOAuthToken.KaKaoOAuthTokenDTO oAuthToken = client.getAccessToken(accessTokenResponse); | ||
| System.out.println(oAuthToken.getAccess_token()); |
There was a problem hiding this comment.
Avoid using System.out.println in production code; please use a proper logging mechanism (e.g., log.debug or log.info) to maintain consistency with the rest of the code.
| System.out.println(oAuthToken.getAccess_token()); | |
| log.debug(oAuthToken.getAccess_token()); |
msk226
changed the title
dvlp-sy
left a comment
dvlp-sy
left a comment
There was a problem hiding this comment.
확인이 늦어서 죄송합니다! 고생하셨어요 👍🏻👍🏻
#️⃣ 연관된 이슈
🔎 작업 내용
@Deprecated애노테이션 추가 및/legacy패키지로 이동📷 스크린샷 (선택)
💬리뷰 요구사항 (선택)