Skip to content

[Fix] JWT 재발급 로직 구현 완료, 쿠키 반영 실패 이슈 있음#202

Closed
seolminkkang wants to merge 8 commits intomainfrom
feature/#84
Closed

[Fix] JWT 재발급 로직 구현 완료, 쿠키 반영 실패 이슈 있음#202
seolminkkang wants to merge 8 commits intomainfrom
feature/#84

Conversation

@seolminkkang
Copy link
Collaborator

개요

Resolves: #84

PR 유형

어떤 변경 사항이 있나요?

  • 새로운 기능 추가
  • 버그 수정
  • CSS 등 사용자 UI 디자인 변경
  • 코드에 영향을 주지 않는 변경사항(오타 수정, 탭 사이즈 변경, 변수명 변경)
  • 코드 리팩토링
  • 주석 추가 및 수정
  • 문서 수정
  • 테스트 추가, 테스트 리팩토링
  • 빌드 부분 혹은 패키지 매니저 수정
  • 파일 혹은 폴더명 수정
  • 파일 혹은 폴더 삭제

###작업 내용

  1. accessToken / refreshToken 인증 흐름 리팩토링
  • accessToken 만료 시 refreshToken 기반으로 재발급 후 쿠키에 재저장하는 흐름 구현
  • JwtAuthenticationFilter에서 accessToken이 만료되었을 경우:
    쿠키에서 refreshToken 추출
    Redis의 토큰 값과 일치하는지 검증
    accessToken을 새로 발급하여 쿠키에 저장
  1. 쿠키 기반 인증 흐름 개선
  • TokenHelper를 통해 accessToken / refreshToken 쿠키를 생성 및 재사용
  • 기존 헤더 기반 인증을 제거하고 쿠키를 활용한 인증 흐름으로 전환
  1. 만료 시간 application.properties 연동
  • J- wtProvider의 accessToken 만료 시간을 하드코딩에서 frontend.access-token-expiration-time 프로퍼티로 수정
  • 기존 상수 ACCESS_TOKEN_VALIDITY 제거
  1. JWT 필터 내 디버깅 로깅 추가
  • accessToken, refreshToken 쿠키 로그 출력 (logTokensFromCookies())
  • 인증 성공/실패에 대한 로그 및 상태 코드 반환 처리 강화

공유사항 to 리뷰어

완벽실패했습니다... 절대 머지하지마세요.

PR Checklist

PR이 다음 요구 사항을 충족하는지 확인하세요.

  • 커밋 메시지 컨벤션에 맞게 작성했습니다. Commit message convention 참고 (Ctrl + 클릭하세요.)
  • 변경 사항에 대한 테스트를 했습니다.(버그 수정/기능에 대한 테스트).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

로그인 시 Refresh Token Rotation을 구현할 수 있다.

2 participants