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

fix: Token 객체 TTL 시간 단위 이슈 해결 #146

Merged
merged 2 commits into from
Dec 17, 2023
Merged

fix: Token 객체 TTL 시간 단위 이슈 해결 #146

merged 2 commits into from
Dec 17, 2023

Conversation

Ji-soo708
Copy link
Member

개요

  • 토큰 객체의 TTL은 MILLISECONDS로 설정되어 있음에도 불구하고, redisTemplate을 통해 레디스에 저장할 때 expiration 단위가 SECONDS로 설정되어 있음을 확인했습니다. 로컬에서 검토한 결과, 토큰 객체 자체는 MILLISECONDS로 ttl이 설정되어도 레디스에 저장될 때는 초 단위로 저장됨을 확인하여 이 부분을 수정했습니다.

작업사항

  • expiration 시간 단위를 밀리세컨드 단위로 수정

주의사항

  • 추가적으로 수정해야할 부분이 있는지 확인해주세요

@Ji-soo708 Ji-soo708 added the 🚨심각한 버그🚨 사용자의 주요 기능이 아예 동작하지 않을 우려가 있는 버그 label Dec 17, 2023
@Ji-soo708 Ji-soo708 self-assigned this Dec 17, 2023
kth990303
kth990303 approved these changes Dec 17, 2023
View reviewed changes
Copy link
Member

@kth990303 kth990303 left a comment

Choose a reason for hiding this comment

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

수고하셨어요~!

@Ji-soo708 Ji-soo708 merged commit 3e41d97 into develop Dec 17, 2023
2 checks passed
@Ji-soo708 Ji-soo708 deleted the MOCACONG-491-refreshtoken-expiration-issue branch December 17, 2023 21:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@kth990303 kth990303 kth990303 approved these changes

@jung-woo-kim jung-woo-kim Awaiting requested review from jung-woo-kim

Assignees

@Ji-soo708 Ji-soo708

Labels
🚨심각한 버그🚨 사용자의 주요 기능이 아예 동작하지 않을 우려가 있는 버그
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@Ji-soo708 @kth990303