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

jwt 검증 로직을 추가합니다. #242

Merged
merged 8 commits into from
Feb 16, 2023
Merged

jwt 검증 로직을 추가합니다. #242

merged 8 commits into from
Feb 16, 2023

Conversation

hyejungg
Copy link
Member

What is this PR? 🔍

Key Changes 🔑

  1. 기존에 passport 를 사용해서 로그인/회원가입을 처리하는 로직을 삭제하고 새로 추가
    • jwt 까지 passport 껄로 사용하려 했으나 전혀 쓰지 않는 것을 확인하고, 더이상 유지해야 할 필요성이 사라짐
  2. ioredis 를 추가
    • redis 를 추가하여 토큰 갱신 로직 추가

@hyejungg hyejungg added feat 새 기능 추가 fix 잘못된 부분 수정 mod 코드 수정 add 코드 추가 labels Feb 12, 2023
@hyejungg hyejungg added this to the User 기능 구현 milestone Feb 12, 2023
@hyejungg hyejungg self-assigned this Feb 12, 2023
@hyejungg hyejungg merged commit ad23ff5 into dev Feb 16, 2023
@hyejungg hyejungg deleted the dev-jwt branch February 16, 2023 15:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
add 코드 추가 feat 새 기능 추가 fix 잘못된 부분 수정 mod 코드 수정
Projects
None yet
Development

Successfully merging this pull request may close these issues.

jwt 유효시간 설정 후 refresh 토큰 정책 추가
1 participant