[BE] refactor: OpenIdUserInfoProvider 삭제 및 OpenIdClient에 비즈니스 로직 구현 (#950)

seokjin8678 commented 6 months ago

📌 관련 이슈

이슈 내용 처럼 OpenIdUserInfoProvider 인터페이스를 삭제하고, 해당 인터페이스를 구현하고 있는 KakaoOpenIdUserInfoProvider에 있는 비즈니스 로직을 KakaoOpenIdClient로 이동시켰습니다.

특별히 위임을 하면서 까지 얻을 수 있는 장점이 크게 보이진 않더라구요. 😂

추후 AppleOpenIdClient가 새로 생긴다면 뭔가 있을 수 있겠지만, OpenID Connect가 표준 방식이라, 리팩터링한 방법처럼 OpenIdClient에 비즈니스 로직이 모두 담길 것 같네요.

github-actions[bot] commented 6 months ago

228 files 228 suites 28s :stopwatch: 761 tests 761 :white_check_mark: 0 :zzz: 0 :x: 780 runs 780 :white_check_mark: 0 :zzz: 0 :x:

Results for commit 83917190.

:recycle: This comment has been updated with latest results.

seokjin8678 commented 5 months ago

리뷰 기간이 지났기에 머지 처리 하도록 하겠습니다!