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

[BE] 인증 관련 계층(Interceptor, ArgumentResolver) 책임 분리 #546

Open
1 task
yoondgu opened this issue Oct 3, 2023 · 0 comments
Open
1 task

[BE] 인증 관련 계층(Interceptor, ArgumentResolver) 책임 분리 #546

yoondgu opened this issue Oct 3, 2023 · 0 comments
Labels
BE 백엔드 관련 이슈 refactor 리팩토링 관련 이슈 우선순위 : 하

Comments

@yoondgu
Copy link
Collaborator

yoondgu commented Oct 3, 2023 •
edited
Loading

🛠️ 작업 대상

  • Interceptor
  • ArgumentResolver

두 클래스에서 DB 통신을 제거하고, 아래와 같이 책임을 분리한다.

  1. Interceptor에서는 AccessToken의 존재 여부만으로 LoginRequired를 확인한다.
  2. ArgumentResolver에서는 AccessToken을 MemberId로 변환만 해준다.
  3. Service에서 DB와 통신하여 MemberId의 유효성 여부를 확인한다.

Service 별 유효성 여부 로직 중복은 다른 방법들을 이용하여 개선한다.

관련 PR #539 참고

✅ 작업 내용

  • 작업 맡으신 분이 tasks 세분화해서 작성 부탁드립니다!

📎 참고 자료

No response

⏰ 추정 시간

작업 맡으신 분이 작성 부탁드립니다!
@yoondgu yoondgu added BE 백엔드 관련 이슈 우선순위 : 중 refactor 리팩토링 관련 이슈 labels Oct 3, 2023
@yoondgu yoondgu added this to the 최종 데모데이 milestone Oct 3, 2023
@yoondgu yoondgu removed this from the 최종 데모데이 milestone Oct 4, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
BE 백엔드 관련 이슈 refactor 리팩토링 관련 이슈 우선순위 : 하
Projects
2023-map-befine
Status: No status
Milestone
No milestone
Development

No branches or pull requests
1 participant
@yoondgu