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/83-스프링 시큐리티를 전체 API에 적용하기 #84

Merged
merged 3 commits into from
Apr 5, 2024

Conversation

koojun99
Copy link
Contributor

@koojun99 koojun99 commented Apr 3, 2024

🚀 개요

🔍 변경사항

  • 서비스 전체 API에서 @RequestHeader를 이용한 JWT 토큰을 직접 검사하는 방식 삭제
  • @AuthMember를 이용해 현재 로그인 된 유저의 정보를 불러와 비즈니스 로직(서비스)에 사용

⏳ 작업 내용

  • 스프링 시큐리티 관련 코드들 이해하기
  • 전체 API에 스프링 시큐리티 인증을 적용(토큰으로부터 사용자 정보 불러오기가 필요한 API에 한해)

📝 논의사항

/reissue 동작 관련 서비스 코드를 개선하고자 하는데 Redis 붙일 때 같이 해도 괜찮을까요?

@koojun99 koojun99 added the ♻️ refactor Refactoring code label Apr 3, 2024
@koojun99 koojun99 self-assigned this Apr 3, 2024
@koojun99 koojun99 changed the title Refactor/83 Refactor/83-스프링 시큐리티를 전체 API에 적용하기 Apr 3, 2024
@CYY1007 CYY1007 merged commit 4023bf6 into develop Apr 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
♻️ refactor Refactoring code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

🔨[refactor] 스프링 시큐리티 대공사에 이은 비즈니스 로직에서의 인증절차 적용
2 participants