Conversation
|
This pull request is automatically being deployed by Amplify Hosting (learn more). |
| }, []); | ||
|
|
||
| useEffect(() => { | ||
| if (rememberEmail) { |
There was a problem hiding this comment.
이메일 기억하기 체크박스를 활성화하는 순간, 로그인 성공 여부와 관계없이 그때 입력한 이메일이 바로 쿠키에 저장됩니다.
그래서 로그인에 실패하거나 그냥 입력만 하고 새로고침을 해도 이메일이 그대로 남습니다..!
보통은 로그인을 성공했을 때 마지막으로 쓴 이메일을 기억해 주는 흐름이라 코멘트 남겨봅니다!
There was a problem hiding this comment.
넵 보안상 소희님께서 피드백 주신 흐름이 더 적절한 것 같아서 로직 수정하도록 하겠습니다!
Jsplix
left a comment
Jsplix
left a comment
There was a problem hiding this comment.
고생하셨습니다 ㅎㅎ 코멘트 확인 부탁드릴게요!
| const expires = new Date(); | ||
| expires.setDate(expires.getDate() + days); |
There was a problem hiding this comment.
P4: 이 코드는 유효기간 설정인거같은데, 다른 곳에서 쿠키의 유효기간에 대한 검증을 진행하는가요?
There was a problem hiding this comment.
다른 곳에서는 일반적으로 보안 강화를 위해서 넣는다고 하긴하는데 위에 소희님께서 남겨주신 피드백대로 로직 흐름을 수정한다면 서버 의존성도 없고 세션 쿠키로 동작하기 때문에 따로 유효기간 수동 설정이 필요없을 것 같아요!
제거하겠습니다~!
3 participants
Motivation
Problem Solving
쿠키에 저장(30일 유지)To Reviewer
Sign In페이지가 전반적으로 영문이라 영어로 작성해두었는데 추후 언어 통일 협의 필요할 것 같습니다!