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.

Sign up for GitHub

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

Jump to bottom

[FIX] 토큰 재발급 로직 변경 #77

Merged
merged 5 commits into from
Nov 8, 2023
Merged

[FIX] 토큰 재발급 로직 변경 #77

merged 5 commits into from
Nov 8, 2023

Conversation

HI-JIN2
Copy link
Member

@HI-JIN2 HI-JIN2 commented Nov 7, 2023 •
edited
Loading

개요

AppInterceptor에서 해결하도록 변경

시나리오

  1. Interceptor를 통해, 토큰이 헤더에 들어가야하는 API를 호출할시에 반환 코드가 401이나 403일 경우를 검출, 200일시 패스~
  2. 반환 코드가 401이나 403이라면, 토큰 재발급 api를 호출(헤더에 액세스 토큰 자리에 리프레쉬 토큰을 꽂아서 호출 & 헤더 이름은 변동 X)
  3. 재발급 받은 토큰을 헤더에 꽂고 이전에 요청한 그 요청을 다시 실행

이렇게 하면 사용자는 눈치 못채지만 토큰은 교체되게 할 수 있습니다!

@HI-JIN2 HI-JIN2 self-assigned this Nov 7, 2023
@HI-JIN2 HI-JIN2 linked an issue Nov 7, 2023 that may be closed by this pull request
[FEAT] token reissue #26
Closed
1 task
CJiu01
CJiu01 approved these changes Nov 8, 2023
View reviewed changes
Copy link

@CJiu01 CJiu01 left a comment

Choose a reason for hiding this comment

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

수고하셨습니다~ 화이팅🔥🔥🔥🔥

CJiu01
CJiu01 approved these changes Nov 8, 2023
View reviewed changes
Copy link

@CJiu01 CJiu01 left a comment

Choose a reason for hiding this comment

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

수고!

CJiu01
CJiu01 approved these changes Nov 8, 2023
View reviewed changes
Copy link

@CJiu01 CJiu01 left a comment

Choose a reason for hiding this comment

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

수고🔥

Amepistheo
Amepistheo approved these changes Nov 8, 2023
View reviewed changes
Copy link
Member

@Amepistheo Amepistheo left a comment

Choose a reason for hiding this comment

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

오 신기하네여!! 고생했습니다!

@HI-JIN2 HI-JIN2 merged commit d79daa8 into main Nov 8, 2023
1 check passed
@HI-JIN2 HI-JIN2 deleted the 26-feat-token-reissue branch November 8, 2023 12:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Amepistheo Amepistheo Amepistheo approved these changes

@CJiu01 CJiu01 CJiu01 approved these changes

Assignees

@HI-JIN2 HI-JIN2

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[FEAT] token reissue
3 participants
@HI-JIN2 @CJiu01 @Amepistheo