Skip to content

Conversation

@yu-so-young2
Copy link
Member

📝 작업 내용

  • 신규 회원 가입시 웰컴 팝업을 띄우기 위하여, 신규 회원 여부(boolean)를 쿠키에 추가하였습니다.
    • 해당 쿠키는 로그인 성공시 유저 정보(닉네임, 이메일, 관리자여부)와 함께 쿠키에 담겨 클라이언트로 리다이렉트됩니다.
    • DEVDEVDEV_MEMBER_IS_NEW : true / false
      스크린샷 2025-07-09 오후 11 36 14

🔗 참고할만한 자료(선택)

💬 리뷰 요구사항(선택)

UserPrincipal 에 isNewMember 필드를 추가한 이유

  • 신규 회원 여부를 확인하여 회원가입을 진행하는 부분(OAuth2MemberService.register())과 로그인 성공 후 쿠키를 생성하여 응답에 담는 부분(OAuth2SuccessHandler.onAuthenticationSuccess())이 분리되어 있어, 신규회원 여부를 어떻게 핸들러 쪽으로 전달할지 고민했는데요!
    • 1)UserPrincipal 에 isNewMember 필드를 추가하여 전달
    • 2)MDC(스레드 로컬)에 저장한 후 꺼내서 사용
  • 이중 신규 회원 여부는 로그인 성공 후 핸들러에서 쿠키 및 응답을 생성할 때 꼭 필요한 정보이기 때문에 인증 객체인 UserPrincipal에 포함하는 것이 더욱 적합하다고 판단했습니다.
  • MDC는 tracd_id, ip 등 부가적인 요청 정보를 저장하는 용도가 적합하다고 알고있기도 하고, 신규 회원 관련 로직이 퍼지지 않고 UserPrincipal 한 곳에 모여있도록 하는 것이 더 명확하다고 생각해서 UserPrincipal에 추가하는 방식으로 구현했습니다~!!

@yu-so-young2 yu-so-young2 requested a review from ssosee July 9, 2025 14:50
@yu-so-young2 yu-so-young2 self-assigned this Jul 9, 2025
Copy link
Member

@ssosee ssosee left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

MDC 에 대해서 잘 몰랐는데, 소영님 덕분에 알게되었네요!
소영님이 고민하신대로 MDC 보다는 회원의 상태이기 때문에 UserPrinciple 에 역할과 챔임을 부여하는 것이 좋아보이네요!!! 👍

고생하셨습니다.

@ssosee ssosee merged commit 5f94899 into develop Jul 12, 2025
1 check passed
@yu-so-young2 yu-so-young2 deleted the DP-537 branch July 13, 2025 06:58
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.

3 participants