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 subject 내용 수정(PhoneNumber -> UserId) #119

Merged
merged 3 commits into from
Oct 21, 2023
Merged

[FIX]: jwt subject 내용 수정(PhoneNumber -> UserId) #119

merged 3 commits into from
Oct 21, 2023

Conversation

jihwan2da
Copy link
Member

@jihwan2da jihwan2da commented Oct 21, 2023 •
edited
Loading

Issue number

  • resolved #0

작업 사항

  • jwt subject 내용을 phoneNumber에서 userId로 수정하였습니다.
    • 토큰에 phoneNumber같은 민감한 개인 정보가 들어가는 것을 수정하였습니다.
    • 앞으로 서버가 여러개로 나누어지면서 API Gateway 부분에서 인증/인가 처리를 하게 될텐데 그럴 때 phoneNumber 보단 userId가 필요하여 수정하였습니다.

@jihwan2da jihwan2da added the fix label Oct 21, 2023
@jihwan2da jihwan2da self-assigned this Oct 21, 2023
@jihwan2da jihwan2da merged commit 0d5dcbb into dev Oct 21, 2023
1 check passed
@jihwan2da jihwan2da deleted the fix/jwt branch February 12, 2024 10:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@jihwan2da jihwan2da

Labels
fix
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@jihwan2da