Skip to content

Conversation

@ChaeAg
Copy link
Member

@ChaeAg ChaeAg commented Feb 22, 2025

Related Issue

Key Changes

  • RefreshToken 엔티티에 @indexed 어노테이션 추가하여 userId로 RefreshToken을 조회할 수 있도록 함.

    • @indexed 사용 시 RefreshToken 생성/삭제 시 한 번의 추가 연산이 발생하나, 검토한 방법들 중 가장 성능 영향이 적음.
    • index로 생성된 값도 TTL이 적용되며, 본체 삭제 시 인덱스도 자동 삭제되어 관리가 용이할 것이라 판단함.
    • 인덱스 생성 예시
      image
  • 유저 탈퇴 로직에 유저의 모든 RefreshToken 제거 로직 추가

    • Redis 특성상 deleteAllByUserId가 지원되지 않아 findAllByUserId로 조회 후 deleteAll로 일괄 삭제하는 방식 적용.
  • 회원 탈퇴 API의 Request DTO 수정

    • 기존 사용하던 유저의 RefreshToken을 받아서 해당 토큰만 제거했지만 이제는 user의 모든 RefreshToken을 제거하기 때문에 받을 필요가 없어, 제거.

To Reviewers

References

Copy link
Member

@rinarina0429 rinarina0429 left a comment

Choose a reason for hiding this comment

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

인덱스 방법을 찾아낸 당신~ 멋져요~ ^.^

@ChaeAg ChaeAg merged commit 6026471 into dev Feb 25, 2025
2 checks passed
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.

[FIX] 회원 탈퇴 시 잔여 RefreshToken 삭제 로직 추가

4 participants