hyukjin-lee
commented
4 years ago 안녕하세요 UpperLeaf 님 해당 프로젝트는 추후에 private repo 로 변경될 예정이고 오픈소스 프로젝트가 아닌 학생들로 이뤄진 한 팀의 서비스 개발이 목적이기 때문에 외부인의 PR 을 merge 하진 못할 것 같습니다. 양해 부탁드립니다.
저희 프로젝트에 관심을 보여주신 것만으로도 감사한데 직접 PR 까지 날려주셔서 영광입니다.
PR 에 대해서 의견을 드리자면, UUID 를 인증코드로 사용하게 되면 사용자 입장에서 인증을 하기 위해 다뤄야하는 문자열의 갯수가 32개로 급증하게 되어 사용성에 문제를 끼칠 것 같아서 UUID 를 고려하지 않았습니다.
쿠폰의 인증코드나 회원 관련 인증코드에 대한 중복은 일반적으로 어느정도 수용 되어도 된다고 가정하고 서비스 기획을 하였는데 이 또한 맞는 정책일지 다시 한 번 점검해보도록 하겠습니다.
해당 PR 은 close 하겠습니다. 이걸 그대로 Issue 로 등록해주시면 저희의 고민과 해결과정을 앞으로 Issue 에 업데이트 되도록 해놓겠습니다.
감사합니다.
leaf-upper
안녕하세요! 스프링부트 프로젝트를 보고 지나가다가 PR한번 날려봅니다. 인증코드 생성방식에서 random한 8자리 문자열을 생성해서 메일을 인증하기 위한 코드로 사용하시는데 8자리 문자열을 랜덤하게 생성하는 경우 운이 안좋다면 사용자끼리 문자열이 겹칠 수 있습니다. UUID같은경우 운영 환경에서 거의 문자열이 겹치치 않도록 한다는 말을 들어본것 같아서 해당 코드를 UUID를 random하게 바꾸는 코드로 refactoring하고 develop 브랜치에 PR 날려봅니다.