oauth 회원가입 코드 리팩토링 - Githubissues

foodage / server

나만의 맛집 발자취, Foodage (2023.08.~)

1 stars 1 forks source link

oauth 회원가입 코드 리팩토링 #38

Closed B0X12 closed 2 months ago

B0X12 commented 2 months ago

작업 내용

[as-is]

OauthService에서 login 처리
MemberCommandService로붵 login 가능 여부 결과 전달받아 처리 (exception을 catch하는 방법으로 처리)
OauthService login() 메소드에서 반환된 loginResult를 OauthController에서 분기 처리 각 클래스 간 책임 분리가 명확하지 않고, 관계가 꼬여있었음

[to-be]

각 클래스 별 책임 분리
OauthService에서는 Oauth 사용자 정보만 조회하여 반환
그 외 login 로직은 MemberCommandService에서 처리
OauthController에서는 jwt 생성 및 redirect 주소만 반환

코멘트

@jjjaehoon 님, 위 작업 내용 develop으로 머지하기 위한 pr 요청드립니다. 정상 동작 여부 확인 후 머지 부탁드립니다. 감사합니다.