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]: 애플 로그인 구현 #28

Merged
merged 14 commits into from
Jan 31, 2024
Merged

[feat]: 애플 로그인 구현 #28

merged 14 commits into from
Jan 31, 2024

Conversation

wu-seong
Copy link
Contributor

PULL REQUEST

🎋 작업중인 브랜치

💡 작업동기

  • 애플 로그인 구현
  • 구현하다보니 카카오랑 겹치는 로직이 있어서 최대한 중복 없이 하려고 구조 리팩토링

🔑 주요 변경사항

  • 기존 액세스 토큰의 유효성으로 유저를 검증하던 부분을 idToken 유효성 검증으로 변경
  • 애플 공개키, 토큰 요청하기 위한 AppleAuthClient 생성
  • kakaoOauth2Client 공개키 요청 메서드 추가
  • Apple client-secret 생성 및 토큰 요청 기능 추가, 받아온 refreshToken 유저DB에 저장
  • 로그인 서비스 구조 카카오 구현체와 애플 구현체로 나누는 걸로 리팩토링
  • 토큰 검증을 TokenValidator 에서 수행하는 걸로 리팩토링
  • 프론트랑 얘기해서 토큰 검증 응답 수정

💡 관련 이슈

  • application.yml 파일 추가된 부분 노션에 기재했습니다
  • User DB 변경된 부분도 노션에 적어놨습니다

애플 로그인까지 구현은 했는데 애플 부분은 테스트를 할 수 없는 상태라서 빠르게 CI/CD를 구축해야 할 것 같습니다! 프론트 쪽이랑 얘기했을때도 서버 배포 기다리고 있다고 해서 오프라인으로 만날 때 온전히 테스트가 가능한 상태여야 효율적으로 작업할 수 있을 것 같아서 오늘 중으로 해보려 합니다

@wu-seong wu-seong added ✨ feat 새로운 기능 추가 ♻️ refactor 코드 리팩토링 labels Jan 28, 2024
@wu-seong wu-seong self-assigned this Jan 28, 2024
@wu-seong wu-seong linked an issue Jan 28, 2024 that may be closed by this pull request
10 tasks
Copy link
Contributor

@threeony threeony left a comment

Choose a reason for hiding this comment

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

고생하셨습니다!

Copy link
Contributor

@kimjm9841 kimjm9841 left a comment

Choose a reason for hiding this comment

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

수고하셨습니다~!

@wu-seong wu-seong merged commit b6f4c9c into dev Jan 31, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
♻️ refactor 코드 리팩토링 ✨ feat 새로운 기능 추가
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[feat]: 애플 로그인 구현
3 participants