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: add lru cache required property #120

Merged
merged 2 commits into from
Feb 26, 2025
Merged

fix: add lru cache required property #120

merged 2 commits into from
Feb 26, 2025

Conversation

laggu
Copy link
Member

@laggu laggu commented Feb 25, 2025

lru cache 의 필수 속성을 설정합니다

현재는 임의로 1000으로 설정했습니다.
해당 값은 logout 된 token 을 저장하는 cache 로 1000개 이상 저장이 될 경우 기존 logout 처리한 token 을 다시 사용 가능해집니다
하지만 저희 유저가 1000명 이상이 되기 전까지는 문제가 없을거라 생각됩니다.
추후 실제 유저가 많아지면 해당 값 설정을 변경해야 할수 있습니다

Copy link

@laggu laggu merged commit ac0bae0 into develop Feb 26, 2025
2 checks passed
@laggu laggu deleted the FIENMEE-340 branch February 26, 2025 09:05
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