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.
🖥️ Preview
close #145
✏️ 한 일
❗️ 발생한 이슈 (해결 방안)
기존에 로그인이 실패해도 token이 undefined로 들어가고 메인 페이지로 접속이 가능한 문제가 있었습니다. 원인은 fetch에 있었는데, fetch는 오류 응답을 자동으로 예외로 처리하지 않습니다. fetch API는 HTTP 오류 상태 코드(예: 400, 404, 500)를 네트워크 오류로 취급하지 않고, 성공적인 응답으로 간주해서 error를 throw하지 않습니다.
따라서 일반적으로 발생하는 오류를 try/catch하는 코드 뿐만 아니라 status code를 확인하는 코드가 필요했습니다.
status를 확인하기 위해서 respones.ok(200-299 사이의 status를 정상적인 응답으로 간주)를 사용했습니다.
위와 같이 response.ok가 아닌 경우 error를 throw하게 해서 try/catch 문에 걸리도록 했습니다.
❓ 논의가 필요한 사항