Closed chadongmin closed 1 week ago
Resolves #{이슈-번호}
Email 컬럼의 유니크 제약조건을 해제하고, Email과 SocialType을 복합 유니크 제약조건으로 설정했습니다.
Email 컬럼의 유니크 제약조건을 해제하는 것 뿐 아니라, 복합 유니크 키를 설정해서 DB에서 확실하게 Email + SocialType의 조합으로 유일성을 만족시켜야 할 필요가 있다고 판단했습니다.
토큰 생성 로직에서 에서 SocialType을 추가하도록 로직을 변경했습니다.
토큰 파싱 로직에서도 SocialType을 추출하도록 수정했습니다.
토큰의 길이가 길어져 refresh_token의 컬럼 길이를 255에서 512로 수정해야 합니다.
현재 POSTMAN으로 같은 이메일에서 다른 소셜타입으로 로그인 후, 리뷰 등록 및 내가 작성한 리뷰 정상적으로 조회되는 것을 확인했고, 통합 테스트 작성중입니다.
r: 꼭 반영해 주세요. 적극적으로 고려해 주세요. (Request changes) c: 웬만하면 반영해 주세요. (Comment) a: 반영해도 좋고 넘어가도 좋습니다. 그냥 사소한 의견입니다. (Approve)
Issues 0 New issues 0 Accepted issues
Measures 0 Security Hotspots 0.0% Coverage on New Code 0.0% Duplication on New Code
See analysis details on SonarCloud
Resolves #{이슈-번호}
234
해결하려는 문제가 무엇인가요?
어떻게 해결했나요?
Email 컬럼의 유니크 제약조건을 해제하고, Email과 SocialType을 복합 유니크 제약조건으로 설정했습니다.
Email 컬럼의 유니크 제약조건을 해제하는 것 뿐 아니라, 복합 유니크 키를 설정해서 DB에서 확실하게 Email + SocialType의 조합으로 유일성을 만족시켜야 할 필요가 있다고 판단했습니다.
토큰 생성 로직에서 에서 SocialType을 추가하도록 로직을 변경했습니다.
토큰 파싱 로직에서도 SocialType을 추출하도록 수정했습니다.
토큰의 길이가 길어져 refresh_token의 컬럼 길이를 255에서 512로 수정해야 합니다.
현재 POSTMAN으로 같은 이메일에서 다른 소셜타입으로 로그인 후, 리뷰 등록 및 내가 작성한 리뷰 정상적으로 조회되는 것을 확인했고, 통합 테스트 작성중입니다.
RCA 룰
r: 꼭 반영해 주세요. 적극적으로 고려해 주세요. (Request changes)
c: 웬만하면 반영해 주세요. (Comment)
a: 반영해도 좋고 넘어가도 좋습니다. 그냥 사소한 의견입니다. (Approve)