refactor: 사용자 구조를 팀 레포지토리 기반 아키텍처에 맞게 정렬#12
Conversation
zxcv9203
left a comment
zxcv9203
left a comment
There was a problem hiding this comment.
기존 코드를 바로 머지하셔서 말씀을 못드렸는데 특정 기능을 개발하실 때는 어떤 기획적 요구사항이 있었고, 그걸 풀기 위해 어떤 점들을 고려하셨는지 PR 본문에 적어주셔야 리뷰어가 이해하고 리뷰를 진행할 수 있습니다.
리뷰어는 작성자가 어떤 요구사항을 어떻게 해석해서 풀어냈는지 알 수 없기 때문에, PR 본문만 보고도 충분히 맥락을 이해할 수 있도록 적어주시면 좋겠습니다.
| @DisplayName("application does not depend on entrypoint") | ||
| void application_should_not_depend_on_entrypoint() { |
There was a problem hiding this comment.
제가 작성했던 테스트 DisplayName을 영문으로 변경하셨는데 변경하신 이유가 있을까요?
별도의 이유가 없다면 한글로 작성하는게 이해하기 좋을 것 같습니다.
There was a problem hiding this comment.
별도 의도는 없었고 제 실수였습니다. 한글로 복원하겠습니다..!
| private final UserJpaRepository userJpaRepository; | ||
| private final SocialAccountJpaRepository socialAccountJpaRepository; |
There was a problem hiding this comment.
Repository 네이밍에 JPA라는 기술을 사용한다는걸 명시적으로 알려줄 필요는 없을 것 같아요
| private final UserJpaRepository userJpaRepository; | ||
| private final SocialAccountJpaRepository socialAccountJpaRepository; | ||
|
|
||
| public UserProfileResponse getProfile(Long userId) { |
There was a problem hiding this comment.
아키텍처 규칙상 Application Layer에서 바로 Response를 반환하면 안됩니다.
별도의 DTO로 변환 후 컨트롤러에서 Response로 변환해주세요.
| if (this.role == null) { | ||
| this.role = UserRole.USER; | ||
| } |
There was a problem hiding this comment.
onCreate, onUpdate에서 유저 권한을 할당하는 이유를 모르겠습니다.
할당하는 이유가 무엇일까요? 🤔
| new OAuth2Error("invalid_user_info"), | ||
| registrationId + " 사용자 식별자(providerUserId)를 가져오지 못했습니다." |
There was a problem hiding this comment.
이부분은 사용자에게 어떻게 핸들링되서 내려가게 되나요?
There was a problem hiding this comment.
현재 OAuth2 실패 핸들러는 별도 등록하지 않아 Spring Security 기본 failure flow(리다이렉트)로 처리되고 있습니다.
반면 성공은 OAuth2SuccessHandler에서 JSON을 직접 내려주고 있어 성공/실패 응답 방식이 혼재된 상태입니다.
응답 일관성을 위해 OAuth2 성공/실패를 한 방식으로 통일하는 게 맞다고 생각합니다.. 놓친부분 확인해 주셔서 감사합니다! 현재 브라우저 기반 소셜 로그인 흐름을 고려하면 리다이렉트 방식으로 맞추는 방향이 더 자연스럽다고 보고 있습니다.
리뷰 의견 주시면 성공 처리도 리다이렉트 기반으로 정리해 반영하겠습니다.
There was a problem hiding this comment.
넵 그리고 리다이렉트시에 어디로 보낼지는 프론트와 협의가 필요한 부분이니 이야기 나눠보시면 좋을 것 같습니다.
2 participants
✨ 작업 내용 한 줄 요약
🛠️ 작업 내용
📚 참고 자료 (선택)
👀 리뷰 포인트 (선택)