Skip to content

fix: specify algorithm - #10

Closed
Sehi55 wants to merge 1 commit into
devfrom
bug/9-specify-algorithm
Closed

Sehi55 wants to merge 1 commit into
devfrom
bug/9-specify-algorithm

Conversation

@Sehi55

@Sehi55 Sehi55 commented May 1, 2026

Copy link
Copy Markdown
Contributor

📝 작업 내용

이번 PR에서 작업한 내용을 설명해주세요.

  • 알고리즘 명시

🚀 주요 변경 사항

완료한 이슈 번호
Close #9
관련된 이슈 번호 (닫고 싶지 않은 경우)
Related to #

✅ 자체 체크리스트 (필수)

  • ./gradlew build 실행 결과 정상 (인증샷 첨부)
  • IntelliJ HTTP Client 테스트 완료 (인증샷 첨부)
  • 팀 내 컨벤션 준수 및 불필요한 로그, import 제거
  • 중요한 변경 사항이 팀에 공유되었는지

📸 테스트 인증샷

빌드 결과 및 IntelliJ HTTP Client 실행 화면을 여기에 첨부해 주세요.

💬 리뷰어 전달사항 (선택)

특별히 봐주었으면 하는 부분이나 논의가 필요한 점을 적어주세요.

  • 논의점


📎 참고 자료

관련 문서, 레퍼런스 링크 등이 있다면 여기에 첨부해주세요.

Summary by CodeRabbit

  • 보안 개선
    • JWT 복호화 알고리즘 명시적 구성을 통해 인증 처리의 안정성을 향상했습니다.

@Sehi55 Sehi55 self-assigned this May 1, 2026
@Sehi55 Sehi55 added the bug Something isn't working label May 1, 2026
@Sehi55 Sehi55 linked an issue May 1, 2026 that may be closed by this pull request
@coderabbitai

coderabbitai Bot commented May 1, 2026

Copy link
Copy Markdown

Caution

Review failed

Pull request was closed or merged during review

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: e940fff5-75e9-4c4d-a796-3463f35a9a40

📥 Commits

Reviewing files that changed from the base of the PR and between ff4632f and 3519b43.

📒 Files selected for processing (1)
  • src/main/java/com/michelet/gateway/infrastructure/config/SecurityConfig.java

📝 Walkthrough

Walkthrough

게이트웨이의 JWT 디코더 빈 생성 로직을 업데이트하여 HMAC 비밀키 사용 시 JWS MAC 알고리즘을 명시적으로 HS256으로 설정합니다. 이전에는 알고리즘 지정 없이 디코더를 생성했기 때문에 토큰 검증에 실패하는 문제를 해결합니다.

Changes

Cohort / File(s) Summary
JWT 디코더 알고리즘 명시화
src/main/java/com/michelet/gateway/infrastructure/config/SecurityConfig.java
NimbusReactiveJwtDecoder 생성 시 JWS MAC 알고리즘을 HS256으로 명시적으로 설정하여 토큰 검증 실패 문제 해결. 디코더 인스턴스화 로직 변경으로 외부 서비스에서 발급한 토큰 호환성 확보.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Poem

🐰 토큰 검증의 미스터리를 풀었네,
HS256 알고리즘, 이제 명시적으로 선언!
버그는 사라지고 신뢰 쌓이고,
간단하지만 강력한 이 수정,
게이트웨이가 활짝 열렸네! 🔐✨

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (4 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed 제목이 변경 사항의 핵심(JWT 디코더에 HS256 알고리즘 명시)을 명확하게 반영하고 있습니다.
Linked Issues check ✅ Passed PR이 이슈 #9의 요구사항을 충족합니다. JWT 디코더에 명시적으로 HS256 알고리즘을 설정하여 토큰 검증 문제를 해결했습니다.
Out of Scope Changes check ✅ Passed 모든 변경사항이 JWT 디코더의 알고리즘 명시라는 명확한 범위 내에 있으며, 이슈 #9의 요구사항과 관련이 있습니다.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ 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 bug/9-specify-algorithm

Review rate limit: 9/10 reviews remaining, refill in 6 minutes.

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

@Sehi55 Sehi55 closed this May 1, 2026
@Sehi55
Sehi55 deleted the bug/9-specify-algorithm branch May 1, 2026 01:07
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.

[BUG] 유효하지 않은 토큰 문제

1 participant