Conversation
…ckend_ver.test into G1ltchy follow up branch
routes/auth/route.py
Outdated
|
|
||
| await database["user"].insert_one(findUser) | ||
|
|
||
| del findUser |
Contributor
There was a problem hiding this comment.
데이터베이스 내에서 문서(데이터)를 지우기 위해서는 Collection.delete_one(filter) 비동기 메소드를 사용해야 합니다.
example
await database["pending"].delete_one({"userId": userId})
routes/auth/route.py
Outdated
|
|
||
| database = GetDatabase(config["DATABASE"]["URI"]) | ||
|
|
||
| SMTP_SERVER = 'smtp-mail.outlook.com' |
Contributor
There was a problem hiding this comment.
이건 퍼블리싱 할 때 수정하지 않아도 되는 메일인가요?
공개적으로 노출된 메일일 경우, 유저들에게 혼란을 줄 수 있는 메일을 생성할 가능성이 높습니다.
Contributor
Author
There was a problem hiding this comment.
이건 smtp서버여서 이메일 자체와는 상관 없습니다.
Contributor
Author
|
수정완료 |
Contributor
|
stmp 라이브러리를 썼을 때, 비동기 라우트 내에서 동기 함수를 호출했으므로, 관련한 오류는 발생하지 않나요? |
Contributor
Author
|
나도 진짜 잘은 모르겠는데 await 붙이면 오류 나서 다 뗀거야. |
Contributor
Author
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
추가목록:
auth 라우터
예상 Issue:
pending 쓰레기 값 범람
현재 pending에서 값을 지우는 방법은 계정을 완전히 인증하는 방법밖에 없음.