Skip to content

[FIX] jwt 토큰 인증 추가#6

Merged
taemin3 merged 1 commit into
mainfrom
SPM-506
Nov 12, 2025
Merged

[FIX] jwt 토큰 인증 추가#6
taemin3 merged 1 commit into
mainfrom
SPM-506

Conversation

@taemin3
Copy link
Copy Markdown
Contributor

@taemin3 taemin3 commented Nov 12, 2025

📝 Summary

  • [FIX] jwt 토큰 인증 추가

🙏 Question & PR point

📬 Reference

Summary by CodeRabbit

릴리스 노트

  • 새로운 기능
    • 구매 관련 API 엔드포인트에 공개 접근 권한을 추가했습니다. 인증 없이 해당 경로에 접근할 수 있습니다.

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Nov 12, 2025

Caution

Review failed

The pull request is closed.

🚀 변경 사항 검토

Walkthrough

SecurityConfig에서 /api/purchase/** 경로에 대한 공개 접근 규칙(permitAll)을 추가했습니다. 기존 인증 요구사항이나 제어 흐름은 변경되지 않았으며, 해당 경로만 공개적으로 노출됩니다.

Changes

응집도 / 파일 변경 사항 요약
보안 설정 구성
src/main/java/com/sampoom/purchase/common/config/security/SecurityConfig.java
/api/purchase/** 경로에 대한 공개 접근 규칙(permitAll) 추가. HTTP 인증 설정 내에서 특정 경로에 대한 접근 제어 수정

예상 코드 리뷰 난도

🎯 2 (Simple) | ⏱️ ~10분

추가 검토 필요 영역:

  • /api/purchase/** 경로가 의도적으로 완전히 공개되어야 하는지 확인
  • 기존 JWT 기반 보안 체인과의 충돌 여부 검토 (PR #5와의 관계성)
  • 다른 인증 규칙과의 우선순위 및 상호작용 확인

관련 PR

  • PR #5: 동일한 SecurityConfig 인증 규칙을 수정하며, JWT 기반 보안 체인 추가와 현재 PR의 permitAll 변경이 직접적으로 연관됨

제안 리뷰어

  • Sangyoon98
  • CHOOSLA
  • yangjiseonn
  • vivivim
  • Lee-Jong-Jin

🐰 축하 시

🐇 보안 문을 활짝 열었네요,
/purchase로 누구나 들어오도록!
JWT도 좋지만 때론 자유로움이,
더 큰 가치를 일깨워주니까요.
경로의 선택, 현명히 하길 바라며 ✨

✨ 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 SPM-506

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 578ffee and 484e75d.

📒 Files selected for processing (1)
  • src/main/java/com/sampoom/purchase/common/config/security/SecurityConfig.java (1 hunks)

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.

@taemin3 taemin3 merged commit 71e7149 into main Nov 12, 2025
4 of 5 checks passed
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