Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

UserState가 created일때만 로그인되게 수정 #37

Merged
merged 1 commit into from
Nov 14, 2022
Merged

UserState가 created일때만 로그인되게 수정 #37

merged 1 commit into from
Nov 14, 2022

Conversation

dolong2
Copy link
Member

@dolong2 dolong2 commented Nov 14, 2022

💡 개요

  • gauth로그인과 oauth로그인할때 유저 상태가 created일때만 되게끔 수정했습니다.

📃 작업내용

  • UserIsPendingException추가
  • 자채 로그인에서 state검사 로직 추가
  • oauth로직에서도 state검사 로직 추가

🔀 변경사항

if(user.state != UserState.CREATED)
    throw UserStatePendingException()

모든 로그인 로직에 요거 추가

@dolong2 dolong2 added 2️⃣Priority: Medium 우선순위 중 ✨ Feature 기능 개발 labels Nov 14, 2022
@dolong2 dolong2 self-assigned this Nov 14, 2022
@dolong2 dolong2 merged commit 58dbfa0 into main Nov 14, 2022
@dolong2 dolong2 deleted the feature/lock branch November 14, 2022 23:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@baekteun baekteun baekteun approved these changes

Assignees

@dolong2 dolong2

Labels
2️⃣Priority: Medium 우선순위 중 ✨ Feature 기능 개발
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@dolong2 @baekteun