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]:AuthGuard 로직 수정 #49

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

eomgerm
Copy link
Contributor

@eomgerm eomgerm commented Jan 15, 2024

📍 PR 타입 (하나 이상 선택)

  • 기능 추가
  • 버그 수정
  • 의존성, 환경 변수, 빌드 관련 코드 업데이트
  • 기타 사소한 수정

❗️ 관련 이슈 링크

Close #48

📌 개요

  • JwtStrategy에서 단순하게 payload만 반환하고 있음
  • payload를 받아 사용자를 찾는 로직 추가

🔁 변경 사항

📸 스크린샷

👀 기타 더 이야기해볼 점

✅ 체크 리스트

  • PR 템플릿에 맞추어 작성했어요.
  • 변경 내용에 대한 테스트를 진행했어요.
  • 프로그램이 정상적으로 동작해요.
  • PR에 적절한 라벨을 선택했어요.
  • 불필요한 코드는 삭제했어요.

@eomgerm eomgerm added the ♻️ Refactor 코드 리팩터링 label Jan 15, 2024
@eomgerm eomgerm self-assigned this Jan 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
♻️ Refactor 코드 리팩터링
Projects
None yet
Development

Successfully merging this pull request may close these issues.

♻️ [Refactor]: AuthGuard 로직 수정
1 participant