Skip to content

Conversation

@plla2
Copy link
Member

@plla2 plla2 commented Jan 21, 2025

유형

  • 기능 구현
  • UI 구현
  • 리팩토링
  • 버그 해결
  • 문서 업데이트
  • 기타( )

작업 내용

  • api 요청마다 reissue 요청하는 버그 수정
  • refreshToken이 만료됐을 때의 에러처리

설명 (선택)

  • 여러개의 요청을 할 때 에러가 발생했을 경우 첫 originalRequest를 제외한 나머지 요청들은 reissue요청 후 토큰들을 재저장하기 전까지 queue에 보관 후 새로운 토큰저장이 완료됐을 때 새로운 토큰과 함께 queue에 저장된 기존 요청들을 순서대로 재요청하게끔 수정
  • refreshToken이 만료됐을 때 에러쓰로잉이 아닌 기존의 토큰들과 저장된 유저정보들을 제거한 후 로그인페이지로 리다이렉트하게끔 수정

리뷰 요구사항

  • 기존에 말씀하셨던 에러쓰로잉 전에 빨리 로그아웃하거나 직접 토큰삭제해야했던 불편함을 이제야 수정해씀다...
  • refreshToken 만료를 두번정도 테스트를 하긴했지만, 예기치 못한 에러가 있을수도 있습니다! 겪으시면 언제든 바로 말씀주세영!

@plla2 plla2 requested a review from grapefruit13 January 21, 2025 04:46
@plla2 plla2 self-assigned this Jan 21, 2025
Copy link
Contributor

@grapefruit13 grapefruit13 left a comment

Choose a reason for hiding this comment

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

넵 사용하다 문제 생기면 바로 말씀드릴게요
고생하셨습니다!

@plla2 plla2 merged commit 29a0a5a into dev Jan 21, 2025
1 check passed
@plla2 plla2 deleted the THKV-124 branch January 21, 2025 07:14
plla2 added a commit to plla2/NNplanner-FE that referenced this pull request Jan 21, 2025
Feat[THKV-124]: axios instance 수정
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.

3 participants