[BE] issue372: Github Token 받아오는 과정을 트랜잭션에서 제외

[tco0427]

요약

Github Token 받아오는 과정을 트랜잭션에서 제외

세부사항

• 프로그램의 코드가 DB 커넥션을 가지고 있는 범위를 최소화
• 트랜잭션이 활성화 되어 있는 프로그램의 범위를 최소화

RealMySQL8.0 책의 5.1.2 절을 보면 트랜잭션을 꼭 필요한 최소의 코드에만 적용하는 것이 좋다라는 문장으로 해당 이슈와 관련된 내용을 이야기한다.
현재 우리의 로그인 과정, 즉, AuthService 의 createToken 을 보면 전체 로직이 트랜잭션으로 묶이게 된다.
스프링에서 AOP 를 통해서 제공하는 @Transactional 어노테이션의 경우 메소드 시작과 동시에 트랜잭션을 시작하고, 해당 메소드가 끝날 때 트랜잭션이 끝나면서 commit 을 하게 된다. 하지만 우리에게 정말 트랜잭션이 필요한 부분은 다음 코드 부분이다.
따라서 외부 네트워크와 통신하는 작업을 하는 oAuthClient 를 우리 트랜잭션에서 제거하여 불필요한 커넥션을 가지고 있는 범위와 트랜잭션 활성화 범위를 제거한다.

        final MemberResponse memberResponse = memberService.saveOrUpdate(githubProfileResponse.toMember());
        final Long memberId = memberResponse.getId();

        final Optional<Token> token = tokenRepository.findByMemberId(memberId);
        final TokensResponse tokenResponse = tokenProvider.createToken(memberId);

        if (token.isPresent()) {
            token.get().updateRefreshToken(tokenResponse.getRefreshToken());
            return tokenResponse;
        }

        tokenRepository.save(new Token(memberId, tokenResponse.getRefreshToken()));

        return tokenResponse;

변경 이후

참고 사이트 : 서비스로직과 트랜잭션 분리

close #372

[jaejae-yoo]

github 프로필 정보를 받아 오는 부분을 분리하셨군요 👍

[2022-moamoa]

Analysis Details

0 Issues

• 0 Bugs
• 0 Vulnerabilities
• 0 Code Smells

Coverage and Duplications

• No coverage information (71.40% Estimated after merge)
• No duplication information (0.40% Estimated after merge)

Project ID: woowacourse-teams_2022-moamoa_AYKvd_z4VbW_bWBvgn13

View in SonarQube