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

#510 [feat] 인가 체제 개편 #515

Merged
merged 15 commits into from
Sep 25, 2024
Merged

#510 [feat] 인가 체제 개편 #515

merged 15 commits into from
Sep 25, 2024

Conversation

sohyundoh
Copy link
Member

✒️ 관련 이슈번호

Key Changes 🔑

제가 머지를 잘 못 눌렀습니다.. 죄송합니다 ㅠ 프론트 준비되면 바로 머지하겠습니다!

To Reviewers 📢

토큰이 <글모임 ID, 권한>을 Map으로 가지고 있을 수 있게 사전 작업을 진행했습니다. Test가 계속해서 통과하게 수정했습니다.
헤더에서 유저 정보를 빼오는 작업은 Bearer을 제외하고 빼오는 것이 포함되기 때문에 메서드명을 직관적으로 변경했습니다
기존 테스트 로직이 이상해서 테스트가 깨졌던 것 이라 수정했습니다!
Type 추론이 명확하게 필요해서 JwtTokenProvider에서 명확한 타입 제공하였습니다!
스레드 로컬의 메모리 누수가 발생하지 않게 afterCompletion 후 clear 해주었습니다.
@sohyundoh sohyundoh self-assigned this Sep 19, 2024
@sohyundoh sohyundoh merged commit b7027e4 into develop Sep 25, 2024
2 checks passed
@sohyundoh sohyundoh deleted the feat/#510 branch September 25, 2024 14:02
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

Successfully merging this pull request may close these issues.

[feat] 인가 로직 개편
1 participant