Facade 객체를 활용해 트랜잭션에서 외부 API 통신 분리하기

페스타고 팀에서는 Facade 객체를 활용해 트랜잭션에서 외부 API 통신을 분리했다.

 

문제 상황

Oauth2 기반 로그인 메서드에서, Oauth2 API 통신이 트랜잭션 범위 안에 속해있었다.

 

외부 API 통신은 비교적 시간이 오래 걸리는 메서드로, 이가 트랜잭션 범위 안에 포함되면 DB 커넥션을 가지고 있는 시간과 트랜잭션이 활성화된 시간이 불필요하게 길어진다. 이는 성능 저하와 직결되는 문제이다.

 

따라서 외부 API 통신 코드를 트랜잭션 범위에서 분리하는 것이 바람직하다.

 

Facade 패턴 적용

우리 팀은 파사드 패턴(Facade Pattern)을 적용하여 해당 문제를 해결하였다.

파사드 패턴이란, 서브 시스템을 감추는 상위 수준의 인터페이스를 제공함으로써 시스템의 복잡도를 낮추는 디자인 패턴이다.

 

아래와 같이 AuthService 상위에 AuthFacadeService 객체를 도입함으로써, 트랜잭션의 범위를 최소한으로 줄였다.

(외부 API 통신과 함께, accessToken 생성 로직도 트랜잭션 범위에서 제외하였다.)

 

 

아래는 파사드 객체를 도입한 코드이다.

@Service
public class AuthFacadeService {

	private final AuthService authService;
	private final OAuth2Clients oAuth2Clients;
	private final AuthProvider authProvider;

	/***/

	public LoginResponse login(LoginRequest request) {
		LoginMemberDto loginMember = authService.login(getUserInfo(request));
		String accessToken = getAccessToken(loginMember.memberId());
		return LoginResponse.of(accessToken, loginMember);
	}

	private String getAccessToken(Long memberId) {
		return authProvider.provide(new AuthPayload(memberId, Role.MEMBER));
	}

	private UserInfo getUserInfo(LoginRequest request) {
		OAuth2Client oAuth2Client = oAuth2Clients.getClient(request.socialType());
		return oAuth2Client.getUserInfo(request.accessToken());
	}

	public void deleteMember(Long memberId) {
		authService.deleteMember(memberId);
	}
}

@Service
@Transactional
public class AuthService {

	private final MemberRepository memberRepository;

	/***/

	public LoginMemberDto login(UserInfo userInfo) {
		return memberRepository.findBySocialIdAndSocialType(userInfo.socialId(), userInfo.socialType())
			.map(LoginMemberDto::isExists)
			.orElseGet(() -> {
				Member member = signUp(userInfo);
				return LoginMemberDto.isNew(member);
			});
	}

	private Member signUp(UserInfo userInfo) {
		return memberRepository.save(userInfo.toMember());
	}

	public void deleteMember(Long memberId) {
		Member member = memberRepository.findById(memberId)
			.orElseThrow(() -> new NotFoundException(ErrorCode.MEMBER_NOT_FOUND));
		memberRepository.delete(member);
	}
}

 

해당 PR 링크

https://github.com/woowacourse-teams/2023-festa-go/pulls?q=is%3Apr+is%3Aclosed