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

✨ feat: OAuth2 소셜 로그인 구현 (#53) #97

Merged
merged 7 commits into from
May 30, 2024
Merged

Conversation

DDonghyeo
Copy link
Member

@DDonghyeo DDonghyeo commented May 30, 2024

☝️Issue Number

🔎 Key Changes

  • 카카오 로그인 구현
  • OAuth 용 회원가입 로직 추가
  • AuthType, AuthId 추가

etc

소셜 로그인시 password를 null로 처리했는데, 더 좋은 방법이 있을까요?

해당 소셜 서버에 토큰 인증 방식을 기용하는게 더 좋을까요?

@DDonghyeo DDonghyeo added the feat label May 30, 2024
@DDonghyeo DDonghyeo requested review from jinho7 and seheonnn May 30, 2024 10:22
@DDonghyeo DDonghyeo self-assigned this May 30, 2024
Copy link
Contributor

@seheonnn seheonnn left a comment

Choose a reason for hiding this comment

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

혹시나 해서 조금 찾아봤는데 password null 이어도 상관없을 거 같습니다~!
카카오의 토큰 인증 방식을 사용하면 검증이나 만료 확인 등등의 작업 때마다 요청을 보내야 하니까 (마음대로 쓰기 어려울 듯) 토큰은 우리 서비스 내에서 발급하는 게 더 좋아 보여요

@DDonghyeo
Copy link
Member Author

DDonghyeo commented May 30, 2024

혹시나 해서 조금 찾아봤는데 password null 이어도 상관없을 거 같습니다~! 카카오의 토큰 인증 방식을 사용하면 검증이나 만료 확인 등등의 작업 때마다 요청을 보내야 하니까 (마음대로 쓰기 어려울 듯) 토큰은 우리 서비스 내에서 발급하는 게 더 좋아 보여요

넵 서비스 발급 토큰으로 사용하도록 하겠습니다 ~~!

@DDonghyeo DDonghyeo merged commit 03cb20e into develop May 30, 2024
1 check failed
@DDonghyeo DDonghyeo deleted the feature/#53 branch June 27, 2024 04:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants