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.

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

Bug/343 #344

Merged
merged 1 commit into from
Jun 8, 2024
Merged

Bug/343 #344

merged 1 commit into from
Jun 8, 2024

Conversation

ohgus
Copy link
Contributor

@ohgus ohgus commented Jun 4, 2024

#️⃣연관된 이슈

close: #343

📝작업 내용

민우님이랑 테스트를 하던 중에 선물함 페이지에서 reissue요청이 끝나고 기존 요청이 전달되지 않는 에러를 발견했습니다.

  • api 요청에 body가 있는 경우에만 관련 로직이 동작하게 수정하고 에러가 없어진 것을 확인했습니다.

🙏리뷰 요구사항(선택)

  • 조건문이 좀 복잡한 느낌이 있는데 이건 리프레시 토큰을 쿠키로 수정하면 해결이 가능할거 같습니다!

@ohgus ohgus self-assigned this Jun 4, 2024
Copy link

vercel bot commented Jun 4, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
kakao-funding ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jun 4, 2024 11:35am

@ohgus ohgus changed the title [refactor]: api 요청에 Body가 있는 경우에만 관련된 로직이 수행되게 수정 (#343) Bug/343 Jun 4, 2024
Copy link
Contributor

Choose a reason for hiding this comment

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

요청 body가 있는 경우에만 reissue 요청을 하도록 수정하셨다는 말씀같은데, 제가 이해한 게 맞나요?

혹시 그렇다면 요청 body가 없지만 액세스토큰이 필요한 요청은 어떻게 되나용? (ex. 내 위시리스트 조회 /wishes/me)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

요청 바디가 있는 경우만 바디를 파싱해서 데이터를 채워주는거고 요청바디가 없는 경우에는 엑세스 토큰만 교체해서 다시 보내게 됩니다!

@ohgus ohgus merged commit f57f7ee into dev Jun 8, 2024
3 checks passed
@ohgus ohgus deleted the bug/343 branch June 8, 2024 04:22
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