fix: 배포 오류 해결 (custom fetcher의 토큰 인증 관련 코드 수정) #222
Merged
Conversation
This file contains hidden or 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
- 토큰 인증 api 엔드포인트 변경에 의한 custom fetcher 사이드 이펙트 해결 - 토큰 인증 api 타입 수정
|
💄 Storybook: https://674fa5b608628f124a188752-ozrydpqflk.chromatic.com/ |
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.
작업 개요
작업 상세
미들웨어에서 error가 났을 경우 edge 런타임에 페이지를 강제로 종료시키는 문제가 있어 토큰 인증이 실패하면 상태를 표현하는 지정된 객체를 반환하도록 custom fetcher에 로직을 추가해놓았었습니다. 백엔드 측 api 엔드포인트 변경 부분을 적용할 때 custom fetcher의 이 로직을 수정하지 못해 배포 단계에서 문제가 생겼던 것으로 보입니다.
개발 환경에서 온전히 페이지가 로딩되었던 이유는 msw를 이용한 모킹 api 이용 시 미들웨어가 실행되는 edge 런타임에 대한 대응을 현재 msw에서는 할 수 없기 때문에 mock-middleware를 작성하여 모킹된 api를 이용하고 있었기 때문이었습니다.
해당 부분 확인하여 수정하였으며 배포 사이트 정상 작동 확인했습니다.
코멘트
프로젝트 내의 다양한 값들에 대해 상수화 해야할 필요성이 크게 느껴졌던 이슈였습니다. 다음 미팅 때 안건으로 추가해두겠습니다.