Skip to content

Refactor/60 refresh token#61

Merged
garusitell merged 6 commits into
devfrom
refactor/60-RefreshToken
May 20, 2025
Merged

Refactor/60 refresh token#61
garusitell merged 6 commits into
devfrom
refactor/60-RefreshToken

Conversation

@garusitell

Copy link
Copy Markdown
Collaborator

#️⃣연관된 이슈 번호

📝작업 내용

  • JwtTokneizer에서 Transaction 어노테이션 추가 및 , 로그아웃 시 삭제하였습니다.

🧪 테스트 여부

  • 테스트 코드를 작성함
  • 테스트를 수행함

💬리뷰 요구사항

  • 리뷰어가 특별히 봐주었으면 하는 부분이 있다면 작성해주세요
  • ex) 메서드 XXX의 이름을 더 잘 짓고 싶은데 혹시 좋은 명칭이 있을까요?

logout에 JwtTokeinzer의 기능이 들어가있는 것 같아서 리팩토링 진행하였습니다.
deleteByTokenValue기능으로 로그아웃 및 refreshToken 요청 일때 삭제하도록 하였습니다.

@pbk2312 pbk2312 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

고생하셨습니다.

@johnhuh619 johnhuh619 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

확인했습니다!

@garusitell garusitell merged commit 873bf1c into dev May 20, 2025
@mipangg mipangg deleted the refactor/60-RefreshToken branch May 21, 2025 08:24
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.

4 participants