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

[BE] refactor: KakaoOAuth2Client UserInfo 반환 KakaoOpenIdClient 사용하도록 변경 (#994) #995

Merged
merged 2 commits into from
May 30, 2024

Conversation

seokjin8678
Copy link
Collaborator

📌 관련 이슈

✨ PR 세부 내용

이슈의 내용 그대로, KakaoOAuth2Client에서 UserInfo를 반환하는 것을 KakaoOpenIdClient를 사용하도록 개선했습니다.

기존 방법으로 로그인을 하려면 API 요청을 2번 보내야 하지만, 지금은 한 번만 보내면 로그인을 할 수 있습니다.

하지만 실 사용자는 KakaoOAuth2Client를 사용하는 것이 아니므로, 지금 누릴 수 있는 성능 향상은 없습니다. 😂

OAuth2Client 또한 삭제를 고려했는데, 로컬에서 웹으로 로그인하여 테스트가 필요할 수 있으므로(Postman 등) 삭제는 하지않고 유지하는게 맞을 것 같습니다.

그밖에 OAuth2Client에서 AccessToken과 UserInfo를 따로 반환하는 메서드를 하나로 합쳤습니다.

처음 구현할 때, 클라이언트에서 Authorization Code 또는 Access Token 둘 중 어느 것을 보낼 지 몰라 이렇게 나눠서 구현했었는데, 지금은 더 이상 그럴 일이 없으니 하나의 메서드를 사용하는게 혼란이 더 적을 것 같네요.

@seokjin8678 seokjin8678 added BE 백엔드에 관련된 작업 ⚙️ 리팩터링 리팩터링에 관련된 작업 labels May 27, 2024
@seokjin8678 seokjin8678 self-assigned this May 27, 2024
@github-actions github-actions bot requested review from BGuga, carsago and xxeol2 May 27, 2024 07:10
Copy link

Test Results

243 files  243 suites   30s ⏱️
793 tests 793 ✅ 0 💤 0 ❌
811 runs  811 ✅ 0 💤 0 ❌

Results for commit ca51042.

@seokjin8678 seokjin8678 changed the title refactor: KakaoOAuth2Client UserInfo 반환 KakaoOpenIdClient 사용하도록 변경 (#994) [BE] refactor: KakaoOAuth2Client UserInfo 반환 KakaoOpenIdClient 사용하도록 변경 (#994) May 27, 2024
Copy link
Member

@BGuga BGuga left a comment

Choose a reason for hiding this comment

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

수고하셨습니다!!

@seokjin8678 seokjin8678 merged commit 9109569 into dev May 30, 2024
7 checks passed
@seokjin8678 seokjin8678 deleted the feat/#994 branch May 30, 2024 00:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
BE 백엔드에 관련된 작업 ⚙️ 리팩터링 리팩터링에 관련된 작업
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[BE] 카카오 OAuth2 로그인 시 Userinfo를 OpenID 토큰을 통해 받도록 개선한다.
2 participants