Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

feat: Refresh token & Access token 적용하여 로그인 동작 변경 #231

Merged
merged 6 commits into from
Jul 8, 2024

Conversation

kimsuyeon0916
Copy link
Contributor

구현 내용/방법

  • 리프레쉬/액세스 토큰 발급 API 함수 추가 및 fetch 적용
  • 커스텀 Axios Instance 에 리프레쉬 토큰 적용 동작 추가

리뷰 필요

  • 액세스 토큰이 만료되어 API 요청에 문제가 생기는 경우, 리프레쉬 토큰으로 다시 액세스/리프레쉬 토큰을 재발급 받고 error response 이용해서 요청날렸던 config로 다시 API 재요청 날리는 로직을 추가하였습니다.
  • release-1.0 브랜치에 push 하면 preview 가 빌드되고, main 에 push 하면 production 이 빌드되도록 deployment.yml 파일 수정하였습니다.

제가 놓친 부분이 있다면 말씀 부탁드립니다!

close #111

@kimsuyeon0916 kimsuyeon0916 added Priority: Critical 우선순위: 긴급 Status: Review Needed 상태: 리뷰 대기중인 상태 Type: Improvement 유형: 기능 개선 labels Jul 3, 2024
@kimsuyeon0916 kimsuyeon0916 self-assigned this Jul 3, 2024
Copy link
Member

@prgmr99 prgmr99 left a comment

Choose a reason for hiding this comment

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

고생하셨습니다! 몇가지 질문 남겨드렸어요!! 👍

@kimsuyeon0916 kimsuyeon0916 merged commit f42110c into release-1.0 Jul 8, 2024
@kimsuyeon0916 kimsuyeon0916 deleted the feature/#111_apply_refresh_token branch November 6, 2024 04:22
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
Priority: Critical 우선순위: 긴급 Status: Review Needed 상태: 리뷰 대기중인 상태 Type: Improvement 유형: 기능 개선
Projects
None yet
Development

Successfully merging this pull request may close these issues.

feat: Refresh token & Access token 적용하여 로그인 동작 변경
2 participants