Release에서 회원가입이 중복으로 일어나는 Issue

[60jong]

Todoary 릴리즈 후, 회원가입이 중복으로 일어나는 경우가 많이 보입니다. (현재는 일일이 대조 후 중복된 계정 중 미사용 계정은 모두 삭제했습니다.)

그 원인으로, 회원가입 버튼이 중복으로 눌려서 여러 차례 Request가 보내진 것으로 보입니다.

똑같은 상황을 테스트 해보았습니다.

동시성 테스트 by JUnit5

테스트 코드

• develop-rds 환경에서 테스트했습니다.

• 10개의 쓰레드를 통해 동시에 Apple OAuth User를 저장해보았습니다.

  @Transactional
  @SpringBootTest
  public class ConcurrencyTest {
  @Autowired
  MemberService memberService;
  
  @Autowired
  EntityManager em;
  
  @Test
  public void 회원가입_동시성_테스트() throws Exception {
      String name = "concurrencyName";
      String email = "concurrencyEmail";
      String providerId = "concurrencyProviderId";
      ExecutorService executorService = Executors.newFixedThreadPool(10);
      for (int i = 0; i < 10; i++) {
          executorService.execute(() -> memberService.joinOauthMember(
                  createOauthMemberParam(name, email, ProviderAccount.appleFrom(providerId)))
          );
      }
  
      executorService.shutdown();
      executorService.awaitTermination(10, TimeUnit.SECONDS);
  
      List<Member> members = em.createQuery("select m from Member m where m.name = :name", Member.class)
              .setParameter("name", name)
              .getResultList();
  
      members.stream()
              .map(member -> member.getId())
              .forEach(System.out::println);
      Assertions.assertThat(members.size()).isGreaterThan(1);
  }
  
  OauthMemberJoinParam createOauthMemberParam(String name, String email, ProviderAccount providerAccount) {
      return new OauthMemberJoinParam(name, email, providerAccount, "ROLE_USER", true);
  }
  }

문제 상황 확인

• console에 member id 출력 결과

  

• 실제 DB 상황

  

왜 10명이 아니라 9명이 생겼는지는 모르겠지만... 중복 계정이 생기는 것을 확인했습니다. (중복 request가 보내진 것이 맞아 보입니다.)

문제 해결

어떻게 해결하면 좋을 지 의견 부탁드립니다. @Todoary/todoary_serverdeveloper (일단 아래의 mysql 커맨드로 해결은 했습니다.)

ALTER TABLE member ADD UNIQUE unique_name (email, provider);

결과 확인

• console에 member id 출력

  

• 실제 DB 상황

  

결론

일단 DB에서 (member의 name & provider) set을 unique하게 함으로써 해결했지만, 실제 서비스에서 중복 요청이 또 들어오게 되면 예외 메시지가 앱에서 뜰 것이기에 중복 요청이 못 가도록 앱 버튼에 처리가 따로 필요해보입니다. @Todoary/todoary_iosdeveloper @yooyeri

[melonturtle]

아 버튼 중복으로 눌리는거 때문에 프론트에 버튼 클릭 후에 일시적으로 요청 안가게 막는 것 추가해달라고 해서 추가됐었는데... 회원가입 버튼엔 안됐던건지 모르겠네요 @Todoary/todoary_iosdeveloper

unique 해놓는건 좋은 것 같습니다!