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

refactor: oauth 인증 클래스 리팩토링 #124

Merged
merged 4 commits into from
Sep 25, 2023

Conversation

hseong3243
Copy link
Collaborator

⛏ 작업 사항

  • 기존 OAuthRestClient 인터페이스를 클래스로 변경하였습니다.
    • OAuthRestTemplateClient 구현체의 코드를 그대로 사용합니다.
    • 기존에 RestTemplate을 직접 사용하던 것에서 RestTemplate을 사용하는 클래스인 ApiService를 사용하도록 변경하였습니다.
    • ApiService에 쿼리 스트링을 인자로 받을 수 있는 메서드를 추가하고 기존의 ApiService 코드를 이에 맞게 개선하였습니다.
  • 담당한 컨트롤러 메서드 중 파라미터에 @RequestBody를 사용하던 get요청을 @ModelAttribute로 수정하였습니다.

📝 작업 요약

  • OAuthRestClient 인터페이스 -> 클래스로 변경
  • ApiService 쿼리 스트링을 인자로 받는 메서드 추가 및 기존 로직 개선
  • OAuthRestClient 단위 테스트 추가

💡 관련 이슈

  • 없음

@github-actions
Copy link

Test Results

388 tests  +5   387 ✔️ +4   10s ⏱️ -3s
190 suites +2       1 💤 +1 
190 files   +2       0 ±0 

Results for commit 2d036ac. ± Comparison against base commit 0603396.

@hseong3243 hseong3243 merged commit 34bdd98 into develop Sep 25, 2023
3 checks passed
@hseong3243 hseong3243 deleted the feature/refactor-auth branch September 25, 2023 05:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants