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

[FIX] 익명 유저 처리를 위한 수정 #172

Merged
merged 1 commit into from
Apr 29, 2024
Merged

[FIX] 익명 유저 처리를 위한 수정 #172

merged 1 commit into from
Apr 29, 2024

Conversation

mikekks
Copy link
Member

@mikekks mikekks commented Apr 28, 2024

👩‍💻 Contents

  • 저희 서비스는 모든 API 요청을 위해선 토큰이 필요한 줄 알았는데 그게 아니었습니다 !
  • 플그에서 요청하는 "플레이그라운드 마이페이지 내 모임 정보 조회 API" 한에서는 플그에서 토큰없이 요청을 하고 있더라고요!
  • 그래서 제가 며칠전에 구현한 로깅 인터셉터에서는 무조건 토큰을 가지고 요청하기에 principle이 항상 존재한다고 생각해서 구현을 했었는데요!
  • 인터셉터에서 익명유저 요청(토큰없이 요청)로 요청할 경우 userId 를 가지고 올 수 없기 떄문에 에러가 발생하는 것을 확인했습니다!
  • 그래서 해당 사항 수정해서 PR 올렸습니다!

📝 Review Note

📣 Related Issue

@mikekks mikekks added the 🐞 fix 기능 버그 label Apr 28, 2024
@mikekks mikekks requested a review from sgh002400 April 28, 2024 17:03
@mikekks mikekks self-assigned this Apr 28, 2024
Copy link

height bot commented Apr 28, 2024

Link Height tasks by mentioning a task ID in the pull request title or commit messages, or description and comments with the keyword link (e.g. "Link T-123").

💡Tip: You can also use "Close T-X" to automatically close a task when the pull request is merged.

@mikekks mikekks merged commit 19aaacc into develop Apr 29, 2024
@mikekks mikekks deleted the feat/#171 branch April 29, 2024 04:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🐞 fix 기능 버그 size/XS
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[FIX] 익명유저 처리
2 participants