Skip to content

Conversation

@hughesgoon
Copy link
Collaborator

연관된 이슈를 적어주세요 📌

작업한 내용을 설명해주세요 ✔️

  • InMemoryCache 인터페이스에 Redis 구현체도 연결했습니다.
  • 토큰 리프레시 로직을 추가했습니다.
    • 리프레시 토큰 rotation 전략을 적용했습니다. 리프레시 토큰은 단 한번만 사용 가능합니다.

트러블 슈팅

리뷰어에게 하고 싶은 말을 적어주세요

  • auth 관련 코드는 먼 과거 컨벤션이 상당히 안맞는 영역이더군요.. 다 고치기엔 대작업이라 그냥 과거 방식에 맞춰 기능 이어붙였습니다.

확인하기

  • : 코드에 에러가 없는지 확인했나요?
  • : PR에 설명을 기재했나요?
  • : PR 태그를 붙였나요?
  • : 불필요한 로그나 System.out을 제거했나요?

@hughesgoon hughesgoon added ⚒️ enhancement ⚒️ New feature or request 🚀 feature 🚀 😄 L1 그렇게 급하지 않음 labels Jan 22, 2025
@hughesgoon hughesgoon requested review from ekgns33 and jeeheaG January 22, 2025 06:43
@hughesgoon hughesgoon self-assigned this Jan 22, 2025
@hughesgoon
Copy link
Collaborator Author

Q. 리프레시 토큰을 왜 저장해두는가?
jwt는 보안성을 위해 액세스는 짧게, 리프레시는 길게 ttl을 적용하는 것을 아실겁니다.
이때, 리프레시 토큰의 탈취가 발생했을때, 공격자가 이를 활용하지 못하도록 서버는 리프레시 토큰을 관리해야 합니다.
그런 사유로 일단 저장해두는 개념으로.. rotation 전략의 구현방법 정도로 생각해주세요.

Copy link
Collaborator

@ekgns33 ekgns33 left a comment

Choose a reason for hiding this comment

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

리프레쉬 로직 구현 감사합니다! 고생하셨어유

@hughesgoon hughesgoon merged commit 916efec into main Jan 22, 2025
@hughesgoon hughesgoon deleted the feat/auth-refresh branch January 22, 2025 09:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

⚒️ enhancement ⚒️ New feature or request 🚀 feature 🚀 😄 L1 그렇게 급하지 않음

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants