-
Notifications
You must be signed in to change notification settings - Fork 8
feat: 애플 인증 구현 #184
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: 애플 인증 구현 #184
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
OAuthService를 추상 클래스로 정의하여 OAuth 처리의 공통 흐름을 잘 캡슐화하신 거 같습니다! 👍
특히 protected abstract 메서드를 통해 OAuth 제공자별 차이점이 명확하게 구분해놓으신 게 정말 좋네요! 이런 방식으로 구조화하면 새로운 OAuth 제공자를 추가할 때도 기존 코드를 수정하지 않고 확장할 수 있을 거 같아요. 고생하셨습니다!!
추가로 더 나은 구조에 대해 여쭤보셔서 제 개인적인 생각만 코멘트에 남겨놨습니다!
src/main/java/com/example/solidconnection/auth/controller/AuthController.java
Show resolved
Hide resolved
src/main/java/com/example/solidconnection/auth/dto/oauth/AppleUserInfoDto.java
Show resolved
Hide resolved
✍️ 추가 커밋 설명사실 클라이언트에서 id_token 를 받아오는게 아니라, 이 PR 이 머지되면 아래 두 PR 도 머지하겠습니다! |
|
선 구현 후 리팩터링 🏃♀️🏃♀️🏃♀️ |
관련 이슈
작업 내용
특이 사항
❗️api 에서 변경되는 부분은 다음과 같습니다❗️
/auth/apple이 POST 로 code 를 받게 했습니다.kakaoOauthToken이signUpToken으로 바뀌었습니다. (회의에서 논의된 내용)참고 : https://dev.to/varsilias/complete-guide-to-implementing-apple-oauth-20sign-in-with-apple-authentication-in-a-nodeexpress-application-4hf
리뷰 요구사항 (선택)
더 좋은 구조가 있을지 리뷰해주세요~