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 #81 jwt role 추가 #81

Open
wants to merge 12 commits into
base: develop
Choose a base branch
from
Open

Conversation

hyxklee
Copy link
Member

@hyxklee hyxklee commented Nov 28, 2024

PR 내용

  • 스프링 시큐리티에서 권한검사를 하기 때문에 권한을 기존에는 유저 디비에서 조회한 사용자 정보를 바탕으로 검사를 진행했으나, 이가 비효율적이라 생각해 토큰에 역할을 담는 것으로 수정

PR 세부사항

  • 엑세스 토큰에 사용자 역할 담는 것으로 수정
  • 레디스 키를 이메일에서 userId로 수정. 이에 따라 리프레시 요청시 이메일을 주지 않아도 됨
  • 매번 요청시 디비에서 조회하지 않기 때문의 약간의 성능 향상 (평균 65ms -> 55ms 정도로 약 10ms의 성능 향상)
  • 해당 성능 향상은 모든 요청에 한하기 때문에 전반적인 응답속도 성능 향상을 만들 수 있음

관련 스크린샷

  • 수정전
image
  • 수정후
image

주의사항

  • 직접 요청 시간을 확인한 1차원 적인 임시 테스트이긴 하나 확실한 응답속도 향상이 있었음

체크 리스트

  • 리뷰어 설정
  • Assignee 설정
  • Label 설정
  • 제목 양식 맞췄나요? (ex. #0 Feat: 기능 추가)
  • 변경 사항에 대한 테스트

@hyxklee hyxklee self-assigned this Nov 28, 2024
@hyxklee hyxklee changed the title Refactor #80 jwt role 추가 Refactor #81 jwt role 추가 Nov 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant