Skip to content

Feat/security#133

Closed
kwongio wants to merge 4 commits intodevelopfrom
feat/security
Closed

Feat/security#133
kwongio wants to merge 4 commits intodevelopfrom
feat/security

Conversation

@kwongio
Copy link
Copy Markdown
Member

@kwongio kwongio commented Apr 23, 2025

작업

JWT 로직 수정

기존의 경우 loadUserByUsername을 요청마다 DB 호출

변경: loadUserByUsername 호출을 하지 않음
jwt subject에 email대신 MemberId를 넣음
claim에는 member의 권한을 넣음

아래와 같은 스프링 시큐리티에서 기본으로 제공하는 어노테이션을 사용해 Member의 정보를 가져올 수 있음
image

현재는 member의 id, role 권한만 불러오도록 처리
image

나중에 어드민 추가되면 해야하는 것

  1. 현재 @LoginMember로 되어있는 로직을 @AuthenticationPrinciple로 변경
  2. Admin API는 시큐리티에서 권한 처리해야함 현재는 테스트만 해봄, 정상적으로 403에러로 forbidden처리됨
    image
  3. 403 에러는 커스텀해서 처리해줘야함
  4. 화이트리스트 권한 처리, 현재는 모든 API가 열려있음, 로그인 X, 로그인 O, 어드민 권한 3가지로 구분해야함
    image
  5. 리프레시 토큰 도입, 우선 유저 기능 먼저 개발하고, 시간 남을 때 어떻게 처리할 지 협의 후 제가 백엔드, 프론트 , 앱 설정하겠습니다

kwongio added 4 commits April 22, 2025 18:12
기존의 경우 loadUserByUsername을 요청마다 DB 호출

변경: loadUserByUsername 호출을 하지 않음
jwt subject에 email대신 MemberId를 넣음
claim에는 member의 권한을 넣음
@kwongio kwongio requested a review from LEEKYUNGIMI April 23, 2025 02:17
@kwongio kwongio closed this Apr 23, 2025
@kwongio kwongio deleted the feat/security branch April 23, 2025 02:36
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.

1 participant