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

fix : JWT Token의 현재 로그인한 사용자의 권한을 넣어준다. #57

Closed
kkangh00n opened this issue Jan 5, 2024 · 0 comments · Fixed by #80
Closed

fix : JWT Token의 현재 로그인한 사용자의 권한을 넣어준다. #57

kkangh00n opened this issue Jan 5, 2024 · 0 comments · Fixed by #80
Assignees
@kkangh00n
Labels
fix

Comments

@kkangh00n
Copy link
Collaborator

🚨 오류 내용

  • 현재 로그인 한 유저가 JWT에는 Email 정보만 담겨있다.
  • Owner의 Email을 들고, Email이 같은 User의 권한을 가질 우려가 있기에, JWT의 권한 정보도 담아준다.

✒️ 수정 사항

  • JWT Token 내의 권한정보 함께 담아줌
  • Filter에서 권한에 따라 Member 혹은 Owner 정보를 알맞게 담아줌
@kkangh00n kkangh00n added the fix label Jan 5, 2024
@kkangh00n kkangh00n self-assigned this Jan 5, 2024
@kkangh00n kkangh00n linked a pull request Jan 8, 2024 that will close this issue
feat : JWT에 권한을 함께 넣어준다. #80
Merged
kkangh00n added a commit that referenced this issue Jan 9, 2024
@kkangh00n
Merge pull request #80 from dev-hooon/feat/#57
0e120f4 
feat : JWT에 권한을 함께 넣어준다.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@kkangh00n kkangh00n

Labels
fix
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

feat : JWT에 권한을 함께 넣어준다. dev-hooon/catchtable
1 participant
@kkangh00n