Skip to content

Conversation

@DaveLee-b
Copy link
Contributor

📝요약(Summary)

이슈 번호 : #131

🔨변경 사항(Changes)

회원가입시 이메일을 검증하는 기능입니다

  1. certify 컨트롤러에서 코드 생성 및 검증 진행
  • pending,verified,locked,expired로 상태구분
  • 5회이상 틀릴시 제한 발생(일정시간이후 풀리도록 조치할예정 - 미개발)
  • 검증통과시 횟수 초기화
  • 기간만료시 에러발생(발급시간으로 부터 5분)
  1. 검증 통과시 회원가입 허용
  2. 검증 미통과시 회원가입 제한(코드 상태기반)
  3. (미개발)스케쥴러를 통해 사용이 끝난 코드 정리

😉리뷰 요구사항

미처 생각치 못한 허점이나 기능 로직상에 이상이 있다면 말씀해주시고 해당 기능을 개발을 했지만 테스트는 현재 시간상 그리고 혹시 모를 사고를 미연에 방지하기위해 로컬에서 진행했습니다. 최종발표때 해당 내용을 적용해서 시연영상을 찍을지,언급하고 넘어갈지 의견 주시면 감사하겠습니다.

…에 들어온 부분 수정, certify SecurityConfiguration에 추가,userservice에서 CertifyService를 사용해서 검증하는걸로 수정
Copy link
Contributor

@jong15325 jong15325 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

전체적으로 빠르게 확인했습니다 동석님이라 믿습니다!

Copy link
Contributor

@jeemin65-pixel jeemin65-pixel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

확인했습니다!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants