[refactor/#429] Auth 서비스의 UserRepository 직접 의존 경계 정리#438
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
❤️ 기능 설명
Auth / Security 코드가 User Account repository와 aggregate에 직접 의존하던 경계를 Auth 전용 useraccount seam으로 정리했습니다.
UserAuthAccountService,UserAuthProfile추가로 Auth/Security가 필요한 최소 인증 스냅샷만 조회하도록 변경AuthCommandService,KakaoLoginCommandService,JwtAuthenticationFilter,CustomOidcUserService의UserRepository직접 의존 제거UserReactivatedEvent발행 및 auth cache evict listener 추가로 재활성화 이후 stale principal 위험 완화UserAuthProfile -> UserPrincipal로 통일해 principal 매핑 drift 위험 축소id와 OIDCsub를 같은 service user ID socialId로 정규화하는KakaoSocialId추가UserAggregateReader로 변경해 외부 lookup seam과 구분Swagger 테스트 성공 결과 스크린샷 첨부
연결된 issue
close #429
✅ 체크리스트
🧪 테스트 결과