임시 로그인 #21
Merged
임시 로그인 #21
Conversation
This file contains 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
eunbeann
requested review from
qkrdmstlr3,
woo-jk and
Collection50
as code owners
qkrdmstlr3
reviewed
Aug 11, 2024
src/app/(sidebar)/login/page.tsx
Outdated
|
|
||
| try { | ||
| const response = await postLogin({ loginId, password }); | ||
| localStorage.setItem('accessToken', response.data.accessToken); |
woo-jk
reviewed
Aug 11, 2024
src/app/(sidebar)/login/page.tsx
Outdated
Comment on lines
14
to
21
| useEffect(() => { | ||
| const token = localStorage.getItem('accessToken'); | ||
| if (token) { | ||
| setIsLogin(true); | ||
| } else { | ||
| setIsLogin(false); | ||
| } | ||
| }, []); |
There was a problem hiding this comment.
단순 accessToken이 스토리지에 있다 === 로그인 상태다 가 아니지 않나요!?
해당 토큰의 만료 여부 확인 로직도 필요할 것 같습니다!
…to feat/tempLoogin
…to feat/tempLoogin
…to feat/tempLoogin
qkrdmstlr3
approved these changes
Aug 21, 2024
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.
1. 무슨 이유로 코드를 변경했나요?
임시 로그인 기능을 구현해두었습니다. 소셜 로그인 로직이 아직 나오지 않아서 서버 측에서 전달 받은 id/pw 값을 입력하면 accessToken이 로컬스토리지에 저장 됩니다. (id/pw 값은 스웨거 예시 값으로 확인하시면 됩니다.)
2. 어떤 위험이나 장애를 발견했나요?
임시 구현이므로, 다른 기능과의 호환성 문제가 발생할 가능성이 있습니다.
로컬 스토리지에 토큰을 저장하는 방식이 향후 브라우저 보안 정책 변화에 영향을 받을 수 있습니다.
토큰 만료 기간을 확인하고 리프레쉬 토큰으로 새 토큰을 발급 받는 api가 아직 없어서 해당 부분은 아직 구현 안되어 있습니다.
추후 인터셉트 내에서 구현할 예정입니다.
3. 관련 스크린샷을 첨부해주세요.
https://github.com/user-attachments/assets/f6182611-1eee-43ba-9c91-af09101f045d
가라 뷰 입니다..
4. 완료 사항
임시 로그인 기능 구현
Axios 인스턴스에 토큰 추가 로직 적용
사용자 로그인 성공 시 토큰을 로컬 스토리지에 저장하는 기능 추가
수동 테스트를 통해 기본적인 로그인 흐름 검증 완료
5. 추가 사항 / 코드 리뷰 받고 싶은 부분
임시 로그인 구현이기 때문에 이후 소셜 로그인에서도 살릴 부분들 위주로 확인해주시면 좋을 것 같습니다
임시 로그인 구현에서 보완이 필요한 부분이 있는지 리뷰를 받고 싶습니다.
현재 로직이 이후 계획된 정식 로그인 구현에 어떻게 영향을 줄 수 있는지에 대한 의견을 듣고 싶습니다.