Skip to content

fix: specify algorithm - #11

Merged
Sehi55 merged 1 commit into
devfrom
fix/9-specify-algorithm
May 1, 2026
Merged

Sehi55 merged 1 commit into
devfrom
fix/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
@coderabbitai

coderabbitai Bot commented May 1, 2026

Copy link
Copy Markdown
📝 Walkthrough

개요

JWT 디코더 빈이 HMAC 시크릿 키로부터 NimbusReactiveJwtDecoder를 구성할 때 JOSE MAC 알고리즘을 HS256으로 명시적으로 설정하도록 변경되었습니다. 이전의 기본 구성에 의존하던 방식을 대체합니다.

변경 사항

응집 단위 / 파일(들) 요약
JWT 디코더 보안 구성
src/main/java/com/michelet/gateway/infrastructure/config/SecurityConfig.java
JOSE MAC 알고리즘을 명시적으로 HS256으로 설정하여 JWT 디코더 초기화 로직을 강화했습니다. 기본값에 의존하지 않고 보안 알고리즘을 명확히 지정합니다.

코드 리뷰 난이도 평가

🎯 2 (단순함) | ⏱️ ~10분

🐰 보안의 길을 더욱 밝히려고,
알고리즘을 명시적으로 선언하네요.
HS256의 약속, 더욱 견고해지고,
기본값의 모호함을 떨쳐내니,
신뢰의 문은 더욱 단단해진답니다! 🔐

🚥 Pre-merge checks | ✅ 3 | ❌ 2

❌ Failed checks (1 warning, 1 inconclusive)

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.
Title check ❓ Inconclusive 제목이 변경사항의 주요 내용을 구체적으로 설명하지 못하고 있습니다. 'specify algorithm'은 너무 모호하여 어떤 알고리즘이 지정되었는지, 어느 부분에서 지정되었는지 명확하지 않습니다. 제목을 더 구체적으로 수정하세요. 예: 'fix: set HS256 algorithm in JWT decoder' 또는 'fix: explicitly configure HMAC algorithm for JWT decoding'
✅ Passed checks (3 passed)
Check name Status Explanation
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.

✏️ 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 fix/9-specify-algorithm

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

@jihxonx jihxonx left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

확인했습니다!

@qldo qldo left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

확인하였습니다

@Sehi55
Sehi55 merged commit 7ac91e1 into dev May 1, 2026
6 of 7 checks passed
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] 유효하지 않은 토큰 문제

3 participants