Skip to content

fix: 서버 로그 확인#153

Merged
jaehyeon4406 merged 1 commit intodevfrom
feat/135/detail
Feb 10, 2026
Merged

fix: 서버 로그 확인#153
jaehyeon4406 merged 1 commit intodevfrom
feat/135/detail

Conversation

@jaehyeon4406
Copy link
Copy Markdown
Collaborator

@jaehyeon4406 jaehyeon4406 commented Feb 10, 2026

Summary by CodeRabbit

  • Chores
    • Enhanced error logging for authentication failures to improve system observability and troubleshooting capabilities.

@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Feb 10, 2026

Caution

Review failed

The pull request is closed.

📝 Walkthrough

Walkthrough

Added error logging to JWT authentication filter's exception handler using a static log import. The modification logs exception messages and stack traces when JWT processing fails, prior to returning the existing 401 unauthorized response with JSON error body.

Changes

Cohort / File(s) Summary
JWT Authentication Logging
booklog/src/main/java/com/example/booklog/global/auth/security/JwtAuthFilter.java
Added static import for logging and exception logging statement in catch block to capture JWT processing errors with full stack trace.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

🐰 A log statement hops into view,
Capturing JWT errors fresh and true,
No flow changes, just visibility gained,
Stack traces now beautifully retained! 📝

✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch feat/135/detail

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.

@jaehyeon4406 jaehyeon4406 merged commit 2b36e49 into dev Feb 10, 2026
1 check was pending
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