Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[REFACTOR] 리프레시 토큰 삭제 및 액세스 토큰 만료 시간 삭제 #54

Merged
merged 5 commits into from
Aug 10, 2024

Conversation

hcg0127
Copy link
Member

@hcg0127 hcg0127 commented Aug 10, 2024

💡 관련 이슈

#53

📢 작업 내용

  • 리프레시 토큰 관련 기능 삭제(토큰 발급 및 재발급, redis 저장 등)
  • 로그아웃 및 회원탈퇴 기능 수정

🗨️ 리뷰 요구사항(선택)

  • 회원가입하면 ACTIVE한 하루예산에 ClosingStatus가 ETC로 설정되는 오류 수정 필요
    • DB 테이블 다 drop하고 create하니까 정상 작동
  • 로그아웃한 액세스 토큰이 redis에 영구 저장되는 문제
    • 기존 로그인 방식으로 바꾸면 문제 없음 (런칭 시 바꾸기)

✅ 체크리스트

  • 코드가 정상적으로 컴파일되나요?
  • 이슈 내용을 전부 구현했나요?
  • 작업 기간 내에 개발을 완료했나요?
  • 리뷰어를 선택했나요?
  • 라벨을 지정했나요?

@hcg0127 hcg0127 added ♻️ : Refactor 코드 전체 리팩토링 🎨 Style 코드 포맷 변경, 세미콜론 누락 등 스타일과 관련된 코드 수정 (코드에 영향 X) 🔨 Fix 기능 수정 labels Aug 10, 2024
@hcg0127 hcg0127 self-assigned this Aug 10, 2024
Copy link
Member

@jimmy0524 jimmy0524 left a comment

Choose a reason for hiding this comment

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

수고하셨습니다😂
일단 임시로 변경하는 거니까 괜찮을 것 같습니다! 애초에 토큰 만료가 없다는 게 말이 안되기 떄문에..

@ohu-star
Copy link
Contributor

ohu-star commented Aug 10, 2024

수정하느라 수고하셨습니다😊😊

@hcg0127 hcg0127 merged commit 35b1caf into develop Aug 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🎨 Style 코드 포맷 변경, 세미콜론 누락 등 스타일과 관련된 코드 수정 (코드에 영향 X) 🔨 Fix 기능 수정 ♻️ : Refactor 코드 전체 리팩토링
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants