Fix: 토큰으로 유저 조회 GRPC

[dungbik]

Summary by CodeRabbit

릴리스 노트

• Refactor

  • 인증 시스템의 토큰 검증 로직을 개선했습니다.
  • 예외 처리 방식을 현대적인 패턴으로 리팩토링했습니다.

• Chores

  • 내부 로깅 기능을 강화했습니다.
  • 코드 가독성 및 유지보수성을 개선했습니다.

[coderabbitai]

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 4da21421-6d22-4921-8659-3dbe04029fd2

📥 Commits

Reviewing files that changed from the base of the PR and between 97095e7 and b15d7fa.

📒 Files selected for processing (4)

• src/main/java/flipnote/user/application/AuthService.java
• src/main/java/flipnote/user/application/UserService.java
• src/main/java/flipnote/user/interfaces/grpc/GrpcExceptionHandlerImpl.java
• src/main/java/flipnote/user/interfaces/grpc/GrpcUserQueryService.java

---

Walkthrough

토큰 검증 로직이 AuthService로 통합되어 UserService가 이를 위임하도록 변경되었습니다. GrpcExceptionHandlerImpl은 패턴 매칭 스위치 문으로 리팩토링되었고, GrpcUserQueryService는 @Service에서 @GrpcService로 재어노테이션되고 로깅이 추가되었습니다.

Changes

Cohort / File(s)	Summary
토큰 검증 로직 통합 src/main/java/flipnote/user/application/AuthService.java, src/main/java/flipnote/user/application/UserService.java	AuthService의 validateToken 메서드에서 findActiveUser 호출이 제거되었습니다. UserService의 findUserByToken이 이제 authService.validateToken을 위임하여 토큰 검증을 수행합니다. 사용자 활성 상태 확인이 AuthService에서 제거되고 UserService로 이동했습니다.
gRPC 예외 처리 리팩토링 src/main/java/flipnote/user/interfaces/grpc/GrpcExceptionHandlerImpl.java	instanceof 연쇄 검사에서 Java 패턴 매칭 스위치 문으로 리팩토링되었습니다. 로깅 및 반환 동작은 동일하게 유지되며, 들여쓰기와 import 순서만 조정되었습니다.
gRPC 서비스 어노테이션 및 로깅 개선 src/main/java/flipnote/user/interfaces/grpc/GrpcUserQueryService.java	@Service에서 @GrpcService로 재어노테이션되고 @Slf4j가 추가되었습니다. getUserByToken 메서드에 액세스 토큰을 기록하는 디버그 로그 문이 추가되었습니다.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Possibly related PRs

• FlipNote-User#2: AuthService.java 수정 — 두 PR 모두 validateToken의 검증 로직을 변경하므로 연관이 있습니다.
• FlipNote-User#7: GrpcUserQueryService 클래스 수정 — 둘 다 gRPC 서비스 계층의 사용자 조회 기능을 변경합니다.
• FlipNote-User#10: 토큰 기반 사용자 조회 흐름 수정 — JWT 검증 및 claims에서 사용자를 가져오는 로직이 양쪽 모두 수정됩니다.

Suggested reviewers

• stoneTiger0912

Poem

🐰 토큰 검증을 옮기고, 스위치로 예외를 잡아내네,
gRPC 서비스는 새 옷을 입었네,
로그가 소곤거리며 토큰을 추적하고,
리팩토링의 춤을 춘다, 아름답게! 🎭✨

✨ Finishing Touches

📝 Generate docstrings

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch fix/get-user-by-token

---

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.}