[REFACTOR] 유저 역할 접근 방식 리팩토링

[catomat0]

📌 PR 개요

• DB 접근 유저 role 조회 -> 토큰에서 직접 꺼내도록 리팩토링

---

🔗 관련 이슈

• Closes [REFACTOR] 유저 인증부 수정 #202

---

🛠 변경 사항

• JWT provider / filter 유저 role 조회 방식 변경을 위한 수정 (기존 폼로그인 잔재 삭제)

---

⚠️ 리뷰 시 참고 사항

---

✅ 체크리스트

• 로컬에서 정상 실행됨
• 로그 / 네이밍 정리
• main / develop 직접 커밋 아님

[coderabbitai]

Important

Review skipped

Auto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Path: .coderabbit.yml

Review profile: CHILL

Plan: Pro

Run ID: 434d72b9-7932-427b-abc3-1d867cc813d6

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

• 🔍 Trigger review

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch refactor/#202-remove-user-role-from-db

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[catomat0]

@CodeRabbit 폼 로그인에서 토큰으로 리팩토링 이후에 리팩토링 하지 못했던 클레임에서 유저 역할 꺼내기 이렇게 해결하면 되는거 맞지?

[coderabbitai]

Tip

For best results, initiate chat on the files or code changes.

🧩 Analysis chain

🏁 Script executed:

#!/bin/bash
# Get the PR diff to understand the changes
gh pr diff 211 --repo UMC-DearE/BackEnd

Length of output: 2982

---

🏁 Script executed:

#!/bin/bash
# Check the token generation logic to confirm "role" claim is being set
rg -n "role" src/main/java/com/deare/backend/global/auth/jwt/JwtProvider.java -C 5

Length of output: 721

---

@catomat0: 네, 맞습니다! 접근 방식이 올바르게 구현되어 있습니다. 코드를 확인해 보면:

1. 토큰 생성 시 — JwtProvider에서 .claim("role", user.getRole().name())으로 role을 토큰 클레임에 저장하고 있고,
2. 토큰 인증 시 — getRoleFromToken()으로 클레임에서 String으로 꺼내어 SimpleGrantedAuthority(role)에 바로 넣어주고 있습니다.

흐름이 일관되고, UserRepository 의존성 및 매 요청마다 발생하던 불필요한 DB 조회도 제거되어 성능 면에서도 개선된 좋은 리팩토링입니다. ✅

한 가지 기존부터 있었던 부분인데 참고로 말씀드리면, Spring Security의 hasRole()은 내부적으로 "ROLE_" 접두사를 자동으로 붙여서 비교합니다. 만약 토큰에 저장된 role 값이 "USER", "ADMIN" 같은 형태라면 hasRole() 대신 hasAuthority()를 사용하시거나, 저장 시 "ROLE_USER" 형태로 저장하셔야 합니다. 현재 코드가 어떻게 되어 있는지에 따라 확인해 보시면 좋을 것 같습니다.