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

[Fix/#307] 방장 로그인 403에러시 새로 반환된 토큰 저장 #308

Merged
merged 1 commit into from
Aug 13, 2024

Conversation

simeunseo
Copy link
Member

@simeunseo simeunseo commented Aug 13, 2024

🌀 해당 이슈 번호

🔹 어떤 것을 변경했나요?

  • 방장 로그인 403에러시 새로 반환된 토큰 저장

🔹 어떻게 구현했나요?

기존 코드

else if (err.response.status === 403) {
            console.log(err.response.data.message);
            setIsModalOpen(true);
          }

방장 로그인 api호출시, 방장이 가능시간을 입력하지 않은 상태라면 403에러와 함께 새로운 토큰을 발급받게됩니다.
그런데 기존 코드에서, 가능시간 입력하러 가는 모달만 뜨고 새로운 토큰을 저장하는 로직은 없습니다.

변경 코드

else if (err.response.status === 403) {
            localStorage.setItem('hostToken', err.response.data.data.accessToken);
            setIsModalOpen(true);
          }

따라서 localStorage에 토큰을 저장하는 로직을 추가하였습니다. (기존의 필요없는 console.log는 삭제)

🔹 PR 포인트를 알려주세요!

인증로직도 싹 점검.. 반드시 필요하겠습니다..^^

🔹 스크린샷을 남겨주세요!

2024-08-13.2.58.18.mov

@simeunseo simeunseo added the fix fix label Aug 13, 2024
@simeunseo simeunseo requested a review from ljh0608 August 13, 2024 05:58
@simeunseo simeunseo self-assigned this Aug 13, 2024
@ljh0608
Copy link
Member

ljh0608 commented Aug 13, 2024

빠른 hotfix 감사합니다 고생하셨습니다! 바로 merge하셔도 좋습니다~

@simeunseo simeunseo merged commit f8d4653 into develop Aug 13, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Fix] 방장 로그인 403에러시 새로 반환된 토큰 저장
2 participants