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

Issue 319 애플 로그인 구현 #55

Open
wants to merge 3 commits into
base: develop
Choose a base branch
from

Conversation

nara04040
Copy link
Contributor

📌 관련 이슈

  • closed: #319

✨ PR 세부 내용

  • 클라이언트에서 전송받은 JWT을 사용해 받은 토큰을 검증합니다.
    • 검증 시 JWK를 PEM으로 변환하여 검증 진행을 합니다.
  • jsonwebtoken 라이브러리를 사용한 토큰 검증을 합니다.
  • 현재 member테이블에 password, birthdate등 필드는 null값을 허용하지 않습니다.
    • 기본값을 주어 오류를 피했습니다. -> 온보딩 과정에서 업데이트를 해주어야합니다.

⌛ 소요 시간

@nara04040 nara04040 added the ✨ Feature 기능 개발 label Oct 2, 2024
@nara04040 nara04040 self-assigned this Oct 2, 2024
Copy link
Collaborator

@AllRightJunyoung AllRightJunyoung left a comment

Choose a reason for hiding this comment

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

Apple측 인증까지는 잘 작성이된거같은데 supabase에서 authentication의 Apple Provider가 연동되는 로직이 없는거같습니다

src/app/api/social/login/apple/route.ts Show resolved Hide resolved
console.log('Decoded Token:', decodedToken)

const { sub: appleUserId, email } = decodedToken as any

Copy link
Collaborator

Choose a reason for hiding this comment

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

저희가 사용하는 supabase의 authentication에 provider로 애플 소셜로그인이 등록되어야 하는 로직이 필요한거같네요

supabase authentication에 등록이 되고 인증이되면

supabase측에서 session을 제공해주는데 이를 클라이언트로 response내려줘야하는거로 알고있네요

@AllRightJunyoung
Copy link
Collaborator

추가로 말씀드리면
현재 로직은 Apple 측 소셜로그인 서버로부터 검증을 잘받았다는부분은 있지만 저희 supabase로부터 검증을 받았다는 부분이 빠진거같네요

Copy link
Contributor

@aong13 aong13 left a comment

Choose a reason for hiding this comment

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

전반적으로 스크럼/회의때 설명해주시면 좋을 것 같아요!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
✨ Feature 기능 개발
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants