Skip to content

Fix: 토큰으로 유저 조회 GRPC#14

Merged
dungbik merged 1 commit intomainfrom
fix/get-user-by-token
Apr 5, 2026
Merged

Fix: 토큰으로 유저 조회 GRPC#14
dungbik merged 1 commit intomainfrom
fix/get-user-by-token

Conversation

@dungbik
Copy link
Copy Markdown
Contributor

@dungbik dungbik commented Apr 5, 2026

Summary by CodeRabbit

  • 버그 수정

    • 사용자 인증 토큰 처리 개선
  • 테스트

    • 디버그 로깅 설정 추가
    • OAuth2 설정 확장

@dungbik dungbik requested a review from stoneTiger0912 April 5, 2026 06:17
@dungbik dungbik self-assigned this Apr 5, 2026
@dungbik dungbik added the bug Something isn't working label Apr 5, 2026
@dungbik dungbik merged commit f8883d9 into main Apr 5, 2026
2 of 3 checks passed
@dungbik dungbik deleted the fix/get-user-by-token branch April 5, 2026 06:17
@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Apr 5, 2026

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: db8ec6a5-c14c-4e05-b1a3-9e55991cb3fc

📥 Commits

Reviewing files that changed from the base of the PR and between 1a21f6d and 3d47b8b.

📒 Files selected for processing (2)
  • src/main/java/flipnote/user/interfaces/grpc/GrpcUserQueryService.java
  • src/test/resources/application.yml

Walkthrough

gRPC 사용자 조회 서비스에서 접근 토큰을 트리밍하여 공백을 제거하고, 테스트 리소스 구성에 디버그 로깅 및 OAuth2 기본 URL 설정을 추가했습니다.

Changes

Cohort / File(s) Summary
gRPC 서비스 토큰 정규화
src/main/java/flipnote/user/interfaces/grpc/GrpcUserQueryService.java
getUserByToken 메서드에서 accessTokentrim()으로 정규화하여 선행/후행 공백을 제거한 후 전달합니다.
테스트 구성 업데이트
src/test/resources/application.yml
flipnote.user 패키지에 대한 디버그 로깅 활성화와 OAuth2 기본 URL 설정(http://localhost:8081)을 추가합니다.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related PRs

Suggested reviewers

  • stoneTiger0912

Poem

🐰 토큰을 깔끔하게 다듬고,
로그는 밝게 빛나며,
OAuth2는 주소를 찾아가네.
작은 손질로 완성된 변화,
깨끗함과 명확함이 함께하는 날! ✨

✨ 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

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant