Closed seokjin8678 closed 3 months ago
이슈의 내용 그대로, KakaoOAuth2Client에서 UserInfo를 반환하는 것을 KakaoOpenIdClient를 사용하도록 개선했습니다.
KakaoOAuth2Client
UserInfo
KakaoOpenIdClient
기존 방법으로 로그인을 하려면 API 요청을 2번 보내야 하지만, 지금은 한 번만 보내면 로그인을 할 수 있습니다.
하지만 실 사용자는 KakaoOAuth2Client를 사용하는 것이 아니므로, 지금 누릴 수 있는 성능 향상은 없습니다. 😂
OAuth2Client 또한 삭제를 고려했는데, 로컬에서 웹으로 로그인하여 테스트가 필요할 수 있으므로(Postman 등) 삭제는 하지않고 유지하는게 맞을 것 같습니다.
OAuth2Client
그밖에 OAuth2Client에서 AccessToken과 UserInfo를 따로 반환하는 메서드를 하나로 합쳤습니다.
처음 구현할 때, 클라이언트에서 Authorization Code 또는 Access Token 둘 중 어느 것을 보낼 지 몰라 이렇게 나눠서 구현했었는데, 지금은 더 이상 그럴 일이 없으니 하나의 메서드를 사용하는게 혼란이 더 적을 것 같네요.
243 files 243 suites 30s :stopwatch: 793 tests 793 :white_check_mark: 0 :zzz: 0 :x: 811 runs 811 :white_check_mark: 0 :zzz: 0 :x:
Results for commit ca51042f.
📌 관련 이슈
✨ PR 세부 내용
이슈의 내용 그대로,
KakaoOAuth2Client
에서UserInfo
를 반환하는 것을KakaoOpenIdClient
를 사용하도록 개선했습니다.기존 방법으로 로그인을 하려면 API 요청을 2번 보내야 하지만, 지금은 한 번만 보내면 로그인을 할 수 있습니다.
하지만 실 사용자는
KakaoOAuth2Client
를 사용하는 것이 아니므로, 지금 누릴 수 있는 성능 향상은 없습니다. 😂OAuth2Client
또한 삭제를 고려했는데, 로컬에서 웹으로 로그인하여 테스트가 필요할 수 있으므로(Postman 등) 삭제는 하지않고 유지하는게 맞을 것 같습니다.그밖에
OAuth2Client
에서 AccessToken과 UserInfo를 따로 반환하는 메서드를 하나로 합쳤습니다.처음 구현할 때, 클라이언트에서 Authorization Code 또는 Access Token 둘 중 어느 것을 보낼 지 몰라 이렇게 나눠서 구현했었는데, 지금은 더 이상 그럴 일이 없으니 하나의 메서드를 사용하는게 혼란이 더 적을 것 같네요.