fantasy-fans-ko / lad-be

Backend for LAD (live auction draft) service
MIT License
4 stars 0 forks source link

[PR] Feature/kakao login #21

Closed juyohan closed 2 years ago

juyohan commented 2 years ago

Title

8 카카오 로그인 및 회원가입에 대한 PR 입니다.

Feature

Comment

kimdg1105 commented 2 years ago

고생 많으셨어요~ 코멘트 궁금한 점은 밑에 인라인으로 추가했고, 코드 보면서 리뷰 달겠습니다~

Comment

  • OAuth2 에 관한 정보들과 JWT의 secret key는 따로 application-oauth.yml & application-jwt.yml 로 관리하였습니다. (ignore 해두었습니다.) : yml 파일을 여러 개 두는 것보다 둘을 합치는 것도 좋을 것 같습니다.

  • Refresh 토큰에 대한 필요성이 아직 느껴지지 않아, Access 토큰만 생성한 뒤 관리하였습니다.

  • 예외처리는 최대한 세부적으로 나눠 처리하였습니다. : Refresh 토큰 기능 없이 엑세스 토큰의 시간을 충분히 늘려놓는 것이면 해당 예외는 우선 발생하지는 않는다는 전제인가요?

juyohan commented 2 years ago

고생 많으셨어요~ 코멘트 궁금한 점은 밑에 인라인으로 추가했고, 코드 보면서 리뷰 달겠습니다~ : yml 파일을 여러 개 두는 것보다 둘을 합치는 것도 좋을 것 같습니다.

조금 세분화하였는데 추후 수정사항이 있을 때, 하나로 합치겠습니다!

: Refresh 토큰 기능 없이 엑세스 토큰의 시간을 충분히 늘려놓는 것이면 해당 예외는 우선 발생하지는 않는다는 전제인가요?

기본 3시간으로 했습니다. 만약, 사용자가 로그인을 하고 3시간 동안 해당 페이지에 존재한다면, 예외가 발생이 됩니다!