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.

Sign up for GitHub

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

Jump to bottom

feat : JWT에 권한을 함께 넣어준다. #80

Merged
merged 15 commits into from
Jan 9, 2024
Merged

feat : JWT에 권한을 함께 넣어준다. #80

merged 15 commits into from
Jan 9, 2024

Conversation

kkangh00n
Copy link
Collaborator

⛏ 작업 상세 내용

  • 로그인 후 발급되는 JWT에 권한을 함께 넣어준다.
  • RefreshToken을 서버에 저장 시, 권한도 함께 저장해준다.
  • 로그인 후 요청 시, 권한을 통해 Member인지 Owner인지 구분한다.

📝 작업 요약

☑️ 중점적으로 리뷰 할 부분

  • MemberService의 로그인 로직
  • OwnerService의 로그인 로직
  • JwtAuthenticationFilter의 요청 시 JWT 검증 로직

@kkangh00n kkangh00n added the fix label Jan 8, 2024
@kkangh00n kkangh00n self-assigned this Jan 8, 2024
@github-actions GitHub Actions
Copy link

github-actions bot commented Jan 8, 2024 •
edited
Loading

Test Results

84 tests  ±0   84 ✅ ±0   22s ⏱️ ±0s
24 suites ±0    0 💤 ±0 
24 files   ±0    0 ❌ ±0 

Results for commit 3b6d442. ± Comparison against base commit b398382.

♻️ This comment has been updated with latest results.

@kkangh00n kkangh00n linked an issue Jan 8, 2024 that may be closed by this pull request
fix : JWT Token의 현재 로그인한 사용자의 권한을 넣어준다. #57
Closed
hyun2371
hyun2371 approved these changes Jan 8, 2024
View reviewed changes
Copy link
Member

@hyun2371 hyun2371 left a comment

Choose a reason for hiding this comment

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

권한 부여 로직 구현하느라 고생 많으셨습니다!

@kkangh00n kkangh00n merged commit 0e120f4 into dev Jan 9, 2024
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@hyun2371 hyun2371 hyun2371 approved these changes

@dlswns2480 dlswns2480 dlswns2480 approved these changes

Assignees

@kkangh00n kkangh00n

Labels
fix
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

fix : JWT Token의 현재 로그인한 사용자의 권한을 넣어준다.
3 participants
@kkangh00n @dlswns2480 @hyun2371