two-clock / backend

개발자를 위한 Github 계정 및 저장소 정보 공유 서비스
4 stars 0 forks source link

feat: Add Redis Integration and JWT Token Refresh Functionality #28

Closed openmpy closed 3 months ago

openmpy commented 3 months ago

관련 이슈

변경 사항

application.yml (Redis 관련 설정)

spring:
  data:
    redis:
      port: 6379
      host: localhost

쿠키 저장

스크린샷 2024-08-10 오후 9 02 07

레디스 저장 (새로운 Refresh Token 재발급)

스크린샷 2024-08-10 오후 9 03 01

체크 목록

리뷰 요청

@jjangsky 님~ 여름이 얼른 끝났으면 좋겠네요.. 🥵 로그인 관련해서 Refresh Token 발급과 재발급 기능을 추가했습니다. 확인 부탁드리겠습니다 ~

openmpy commented 3 months ago

@openmpy 님 지속되는 폭염에도 개발하시느라 고생이 많으십니다.

저도 회사에서 JWT 토큰을 사용하지만 Refresh Token에 대해서는 사용하지 않아서 무지한 부분이 많아 피드백이 부족할 수 있다는 점에 사전에 양해를 구하겠습니다.

Refresh Token과 관련하여 여쭙고 싶은 부분이 있습니다. /refresh로 따로 토큰을 발급 받는 부분을 Method로 분리하셨는데 로그인 과정에서 Access Token이 만료되었을 때 Refresh Token으로 재발급 받은 과정을 화면단을 고려하여 생각한 부분이 있으실까요?

@jjangsky 님 정확하십니다 ~

jjangsky commented 3 months ago

훌륭하십니다~ Merge 해도 문제가 없을 것 같습니다 !