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.

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

func of checking user's email #15

Open
wants to merge 14 commits into
base: master
Choose a base branch
from
Open

func of checking user's email #15

wants to merge 14 commits into from

Conversation

G1ltchy
Copy link
Contributor

@G1ltchy G1ltchy commented Jun 11, 2024

추가목록:

auth 라우터

  • /send/{userId}/
    • Demonstration
      • 인증 메일을 보내는 버튼
      • pending DB에 {userId}가 존재하면 {userId}의 authCode를 찾아 이메일로 전송함, HTTPS 200
      • pending DB에 {userId}가 없다면 HTTPS 500
      • 리빙포인트: 메일은 '아웃룩'에서 확인 필요
  • /check/{userId}/
    • Demonstration
      • 웹에서 인증코드를 입력하는 부분.
      • pending DB에 {userId}가 없다면 HTTPS 500
      • pending DB에 {userId}가 있지만 인증 코드가 틀렸다면 HTTPS 400
      • pending DB에 {userId}가 있고 인증 코드가 맞다면 pending DB의 해당 계정에서 authCode를 지운 다음, user DB에 추가후 pending DB에서 해당 데이터를 del, HTTPS 200

예상 Issue:

pending 쓰레기 값 범람
현재 pending에서 값을 지우는 방법은 계정을 완전히 인증하는 방법밖에 없음.


await database["user"].insert_one(findUser)

del findUser
Copy link
Contributor

Choose a reason for hiding this comment

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

데이터베이스 내에서 문서(데이터)를 지우기 위해서는 Collection.delete_one(filter) 비동기 메소드를 사용해야 합니다.

example

await database["pending"].delete_one({"userId": userId})


database = GetDatabase(config["DATABASE"]["URI"])

SMTP_SERVER = 'smtp-mail.outlook.com'
Copy link
Contributor

Choose a reason for hiding this comment

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

이건 퍼블리싱 할 때 수정하지 않아도 되는 메일인가요?
공개적으로 노출된 메일일 경우, 유저들에게 혼란을 줄 수 있는 메일을 생성할 가능성이 높습니다.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

이건 smtp서버여서 이메일 자체와는 상관 없습니다.

@G1ltchy
Copy link
Contributor Author

G1ltchy commented Jun 12, 2024

수정완료

@ZeroOneDeveloper
Copy link
Contributor

stmp 라이브러리를 썼을 때, 비동기 라우트 내에서 동기 함수를 호출했으므로, 관련한 오류는 발생하지 않나요?

@G1ltchy
Copy link
Contributor Author

G1ltchy commented Jun 12, 2024

나도 진짜 잘은 모르겠는데 await 붙이면 오류 나서 다 뗀거야.
그리고 로컬에서 swagger UI로 테스트 해봤는데 정상적으로 메일이 오긴 했어.

@G1ltchy
Copy link
Contributor Author

G1ltchy commented Jun 12, 2024

@ZeroOneDeveloper ZeroOneDeveloper changed the title G1ltchy func of checking user's email Jun 21, 2024
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.

2 participants