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] Reduce Code Duplication with Custom Argument Resolver for MemberID Extraction #577

Closed
5 of 11 tasks
miiiinju1 opened this issue Nov 4, 2023 · 1 comment
Closed
5 of 11 tasks
Assignees

Comments

@miiiinju1
Copy link
Member

miiiinju1 commented Nov 4, 2023

이슈 개요

HandlerMethodArgumentResolver를 활용하여 기존 JWT에 포함된 memberId를 가져오는 과정의 중복 코드인 SecurityUtils.getCurrentMemberId 제거

상세 내용

현재 서비스의 여러 컨트롤러 메서드에서 JWT 토큰에서 memberId를 추출하는 동일한 로직이 반복적으로 사용되고 있다. 이를 HandlerMethodArgumentResolver 인터페이스를 구현함으로써 통합하고자 한다. 이를 통해 컨트롤러 메서드에서 직접 토큰을 파싱하지 않고, 메서드 인자로 memberId를 직접 받을 수 있게 해 코드의 중복을 줄이고, 가독성을 높일 수 있다.

이슈 유형 (아래에 해당하는 경우 [x]로 체크해주세요)

  • 버그 리포트 (Bug Report)
  • 기능 요청 (Feature Request)
  • 문서 개선 제안 (Documentation Improvement)
  • 성능 개선 요청 (Performance Improvement)
  • 코드 개선 제안 (Code Refinement)
  • 기타 (Others) - 설명해주세요:

예상 결과 (버그 리포트의 경우)

  • 각 컨트롤러에서 memberId를 가져오기 위한 중복 코드가 제거된다.
  • 컨트롤러의 메서드 인자를 통해 memberId를 직접 주입받을 수 있게 된다.

추가 컨텍스트

HandlerMethodArgumentResolver의 구현을 통해 @CurrentMember와 같은 커스텀 어노테이션을 만들어 JWT 인증 로직의 중복을 해소할 예정이다.

체크리스트

  • 이슈의 유형을 선택했습니다.
  • 이슈의 상세 내용을 충분히 기술했습니다.
  • 문제 재현을 위한 단계를 기술했습니다 (해당하는 경우).
  • 기능의 필요성과 상세 내용을 기술했습니다 (해당하는 경우).
  • 추가 컨텍스트를 제공했습니다 (해당하는 경우).
@aj4941
Copy link
Member

aj4941 commented Nov 4, 2023

memberId를 가져올 때 항상 작성해야 하던 부분에 개선이 필요했는데 좋습니다!!

@miiiinju1 miiiinju1 self-assigned this Nov 12, 2023
miiiinju1 added a commit that referenced this issue Dec 8, 2023
♻️ Refactor. MemberID Extraction
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants