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.

Sign up for GitHub

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

Jump to bottom

[Bug] #87 - RefreshToken 갱신 시 Redis 저장 토큰과 비교 후 새로운 Token 저장 #88

Merged
merged 2 commits into from
Mar 13, 2025
Merged

[Bug] #87 - RefreshToken 갱신 시 Redis 저장 토큰과 비교 후 새로운 Token 저장 #88

merged 2 commits into from
Mar 13, 2025

Conversation

haeun1107
Copy link
Member

Related Issue 🍀


Key Changes 🔑

  • RefreshToken을 Request Body가 아닌 Request Header에서 받도록 변경
  • Redis에서 userId를 기반으로 저장된 RefreshToken을 조회 후, 요청된 RefreshToken과 비교하여 일치할 경우에만 갱신
  • 기존에는 기존 RefreshToken을 덮어쓰는 방식이었지만, 토큰을 완전히 삭제한 후 새 토큰을 저장하는 방식으로 변경

To Reviewers 🙏🏻

@haeun1107 haeun1107 added 🐛 fix 버그 수정 🐸 하은 작업 하은 작업 labels Mar 13, 2025
@haeun1107 haeun1107 self-assigned this Mar 13, 2025
@haeun1107 haeun1107 linked an issue Mar 13, 2025 that may be closed by this pull request
🐛 [Bug] - RefreshToken 갱신 시 Redis 저장 토큰과 비교하지 않는 문제 #87
Closed
2 tasks
@haeun1107 haeun1107 merged commit c477079 into main Mar 13, 2025
@haeun1107 haeun1107 deleted the bug/#87 branch March 13, 2025 15:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@haeun1107 haeun1107

Labels
🐛 fix 버그 수정 🐸 하은 작업 하은 작업
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

🐛 [Bug] - RefreshToken 갱신 시 Redis 저장 토큰과 비교하지 않는 문제
1 participant
@haeun1107