refactor: USER Role 삭제 및 관련 로직 제거#262
Merged
Merged
Conversation
USER Role에 대한 의존성을 제거하고 관련 코드 및 데이터베이스 마이그레이션 정리. 불필요한 역할 관리 로직을 간소화하여 구조 단순화.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
📝 요약(Summary)
USER Role에 대한 의존성을 제거하고 관련 코드 및 데이터베이스 마이그레이션 정리.
불필요한 역할 관리 로직을 간소화하여 구조 단순화.
💬 공유사항
이제 일반 사용자는 user_role 테이블에서 role 자체를 갖지 않습니다.
이전에는
@PreAuthorize("hasRole('USER')")를 썼지만 이제는@PreAuthroize("isAuthenticated()")를 쓰는 등 관련 의존성들도 제거해뒀구요사용자 로그인 시 나눠주는 액세스 토큰에도 roles 클레임은 아예 들어가지 않고,
현재로서는 user_role 테이블에서 들어가는 건 관리자 권한뿐이게 됩니다.
이제 Role에는 신고 관리, 공지 작성, 광고 운영 등등.. 세부적으로 그 관리를 할 수 있는 권한이 들어간다고 생각하면 돼요
✅ PR Checklist
PR이 다음 요구 사항을 충족하는지 확인하세요.