issues
search
sproutt
/
eussya-eussya-api
개발자 동기부여 서비스 플랫폼 '으쌰으쌰'
20
stars
1
forks
source link
[Back] 소셜 로그인 코드 리팩토링
#110
Closed
Bellroute
closed
3 years ago
Bellroute
commented
3 years ago
sns를 통해 회원가입 시도 및 로그인하는 기능을 가진 SocialService를 MemberService로 통합
OAuth2RequestService를 interface로 변경 후, Provider별로 구현체를 생성
hyukjin-lee
commented
3 years ago
git commit -m 말고 git commit 을 사용해서 작은 단위로 : 구현 내용
구체적인 설명
이렇게 구현한 이유 를 적어주면 히스토리 관리 측면에서 좋을 것 같습니다.
Bellroute
commented
3 years ago
소셜 로그인 로직 설계 수정
인터페이스를 통해 소셜서비스 별 클래스를 분리했음에도 여전히 컨트롤러에서 모든 구현체에 대해 종속적인 코드를 수정했습니다.
OAuth2RequestServiceFactory라는 객체가 요청에 대한 구현 클래스를 선택하도록 역할을 분리시킴으로써 의존성을 낮췄습니다.
Dto -> DTO
불필요한 OAuth2RequestServiceTest 제거
누락된 import 추가
hyukjin-lee
commented
3 years ago
좋네요 저는 승인합니다