Skip to content

[ZIM-35] OAuth 콜백 리다이렉트 및 리뷰 피드백 반영#19

Merged
buddle031 merged 8 commits intomainfrom
chore/local-env-guide-clean
Apr 20, 2026
Merged

[ZIM-35] OAuth 콜백 리다이렉트 및 리뷰 피드백 반영#19
buddle031 merged 8 commits intomainfrom
chore/local-env-guide-clean

Conversation

@buddle031
Copy link
Copy Markdown
Collaborator

@buddle031 buddle031 commented Apr 18, 2026

Relates to: ZIM-36

✨ 작업 내용 한 줄 요약

  • OAuth 로그인 성공/실패 리다이렉트 흐름을 정리하고, 아키텍처 리뷰 피드백(DisplayName/Repository 네이밍/Role 할당 위치)을 반영했습니다.

🛠️ 작업 내용

  • OAuth 로그인 콜백 처리 개선

    • 성공/실패 시 callbackUrl 기반 리다이렉트 처리
    • 성공/실패 상태를 query param(code=LOGIN_SUCCESS | LOGIN_FAILED)로 전달
    • callbackUrl 캡처/정규화/허용 origin 검증 로직 반영
  • 마이페이지 계층 경계 정리

    • Application Layer 반환 타입을 DTO로 분리
    • Controller(Entrypoint)에서 Response 변환하도록 수정
  • 리뷰 피드백 반영

    • LayerDependencyTest DisplayName 한글 복원
    • Repository 네이밍에서 Jpa 제거 (UserRepository, SocialAccountRepository)
    • UserJpaEntity의 onCreate/onUpdate에서 role 기본 할당 제거
  • 검증

    • AuthControllerTest 실행 확인
    • LayerDependencyTest 실행 확인

📚 참고 자료 (선택)

  • 팀 PR 리뷰 코멘트 반영
  • OAuth 성공/실패 리다이렉트 동작 FE 협의 사항 반영

👀 리뷰 포인트 (선택)

  • callbackUrl이 이상한 주소로 들어와도, 우리가 허용한 프론트 주소로만 이동하도록 처리가 잘 되는지

@buddle031 buddle031 requested a review from mike7643 as a code owner April 18, 2026 13:26
@mike7643
Copy link
Copy Markdown
Collaborator

참고하셔서 Jira가 참조할 수 있게 바꾸셔야 할 것 같습니다
https://discord.com/channels/1461341292469551137/1461348159170023488/1493140758155235489

@buddle031 buddle031 changed the title Chore/local env guide clean [ZIM-35] OAuth 콜백 리다이렉트 및 리뷰 피드백 반영 Apr 18, 2026
@buddle031
Copy link
Copy Markdown
Collaborator Author

참고했어요 감사합니다!

Comment thread src/main/java/com/zimdugo/user/entrypoint/UserController.java Outdated
@mike7643
Copy link
Copy Markdown
Collaborator

PR 리뷰 코멘트 관련 수정이 혹시나 필요하실때 PR을 새로 만들지 않고 수정하고 싶으시다면, 로컬에서 같은 브랜치를 수정하고 push 하시면 같은 PR에 반영이 됩니다 ! 한 번 참고해보세요 ~

@buddle031
Copy link
Copy Markdown
Collaborator Author

#17 응답 규약 변경(ok -> of) 반영으로 CI 깨진 부분 후속 커밋에서 수정했고 현재 체크 통과 상태입니다. 기존 refactor 변경사항은 유지되어 있습니다.

Comment thread src/main/java/com/zimdugo/user/infrastructure/UserStoreAdapter.java Outdated
@buddle031
Copy link
Copy Markdown
Collaborator Author

이전 리뷰 반영으로 UserJpaEntity를 persistence.User로 변경했습니다.
이후 도메인 User와 이름이 겹치는 부분이 있어, 해당 로컬 변수는 var saved = ... 로 처리해 FQCN 문제도 함께 반영한 상태입니다.

@mike7643 mike7643 force-pushed the chore/local-env-guide-clean branch from 6f741f5 to cb82a71 Compare April 19, 2026 16:34
Comment thread src/main/java/com/zimdugo/user/infrastructure/SocialAccountStoreAdapter.java Outdated
@buddle031
Copy link
Copy Markdown
Collaborator Author

User/SocialAccount 엔티티 네이밍 정리(Jpa 제거) 및 StoreAdapter 예외 응답 규약 수정했습니다

@mike7643
Copy link
Copy Markdown
Collaborator

수고하셨습니다 :)
개발 편의를 위해서 시큐리티 통과되는 url 에 대해서 일단은 permitAll() 로 해주시면 로컬에서 테스트할때도 편할 것 같습니다!

@buddle031 buddle031 merged commit c2076ec into main Apr 20, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants