cupping-note / cupping-note-server

0 stars 0 forks source link

feat: oauth login 1차 개발 완료 #14

Closed white-gyu closed 1 year ago

white-gyu commented 1 year ago

📄 Description

5 oauth flow 문서를 바탕으로 kakao를 통한 로그인 개발 완료하였습니다.

📌 구현 내용

✅ PR 포인트

현재는 email을 통해 jwt를 발행하는 로직이 UserService 내에 존재합니다. 해당 로직이 UserService 내에 존재하는 점이 옳은지 의문이 듭니다..!

현재 refresh token을 관리하는 테이블인 session 테이블 내 user_id를 unique로 하지 않아 한 명의 user가 여러 개의 refresh token을 가질 수 있어 중복 로그인이 가능합니다. 중복 로그인을 가능하게 하는 점이 괜찮은지 궁금합니다 👀

refresh token이 만료됐을 때 soft delete를 해야 하는지, 만료 상태로 유지해야 하는지 궁금합니다. 차이는 아래와 같습니다. 이 로직은 access token 갱신 API에서 수행하고 있습니다. 관련 커밋 : feat: session 만료된 경우 만료시키는 로직 구현

순서soft delete만료 상태 유지
1삭제기존 토큰 유지
2만료된 토큰으로 API 응답
3이후 같은 만료된 토큰으로 API 재요청
4없는 토큰으로 API 응답만료된 토큰으로 API 응답