Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

refactor: 멤버의 로그아웃 로직 리팩토링 #297

Merged
merged 1 commit into from
Oct 29, 2023

Conversation

YongsHub
Copy link
Collaborator

Issues

구분

  • 버그 수정
  • 기능 추가
  • 코드 리팩터링
  • 문서 업데이트
  • 기타

로그아웃 로직을 리팩토링합니다.

  • 어떠한 이유로도 로그아웃 API 요청이 왔을때, 쿠키가 삭제되도록 리팩토링했습니다.
  • 현재 .ssafsound.com domain 설정으로 인해 test.ssafsound.com 및 ssafsound.com 으로 oauth login을 진행했을때 브라우저에 쿠키가 생성됩니다.
  • 로그아웃 로직에서 토큰 값에 대한 검증 후, memberId를 추출하여 memberToken을 삭제하는 로직이 존재했습니다. memberId가 테스트 서버 및 실 서버에서 체크됨에 따라 존재하지 않아 서버 에러가 발생하는 원인이 있었습니다.
  • MemberToken은 One To One 관계임으로 최초 생성 후, 업데이트만 진행될 수 있게 위에서 언급한 삭제 로직을 제거했습니다.

스크린샷

기타

@YongsHub YongsHub self-assigned this Oct 29, 2023
@YongsHub YongsHub merged commit 117178d into SSAF-SOUND:dev Oct 29, 2023
1 check failed
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.

4 participants