Skip to content

fix: SecurityConfig에서 불필요한 h2-console permitAll 설정 제거#117

Merged
neibler merged 1 commit intodevelopfrom
feature/115
Feb 10, 2026
Merged

fix: SecurityConfig에서 불필요한 h2-console permitAll 설정 제거#117
neibler merged 1 commit intodevelopfrom
feature/115

Conversation

@neibler
Copy link
Copy Markdown
Contributor

@neibler neibler commented Feb 10, 2026

🔍️ 작업 내용

  • Closes #
    fix: SecurityConfig에서 불필요한 h2-console permitAll 설정 제거

✨ 상세 설명

🛠️ 추후 리팩토링 및 고도화 계획

📸 스크린샷 (선택)

💬 리뷰 요구사항

Summary by CodeRabbit

  • 작업
    • H2 콘솔에 대한 공개 접근이 제거되었습니다.

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai bot commented Feb 10, 2026

개요

SecurityConfig에서 "/h2-console/**" 엔드포인트를 공개 접근 허용 목록에서 제거했습니다. Swagger 관련 공개 엔드포인트와 기타 인가 규칙은 변경되지 않았습니다.

변경사항

Cohort / File(s) Summary
Security Configuration
src/main/java/com/ongil/backend/global/config/SecurityConfig.java
"/h2-console/**" 엔드포인트를 permitAll 목록에서 제거. Swagger UI, API 문서, 핑 등 다른 공개 엔드포인트는 유지되며, 관리자·WebSocket·사용자·상품 등 기존 인가 규칙은 그대로 적용됨.

예상 코드 리뷰 소요시간

🎯 1 (Trivial) | ⏱️ ~3 minutes

관련 이슈

🚥 Pre-merge checks | ✅ 2 | ❌ 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 (2 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed PR 제목이 변경사항의 핵심을 명확하게 설명하고 있습니다. SecurityConfig에서 불필요한 h2-console permitAll 설정을 제거했다는 주요 변경사항을 정확히 반영하고 있습니다.

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

✨ 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 feature/115

No actionable comments were generated in the recent review. 🎉

🧹 Recent nitpick comments
src/main/java/com/ongil/backend/global/config/SecurityConfig.java (1)

41-42: 기존 코드 참고: /api/admin/** permitAll 설정은 보안 위험 요소입니다.

이번 PR 범위는 아니지만, h2-console을 제거하면서 같은 맥락으로 언급합니다. "Swagger 테스트용"으로 관리자 API 전체를 permitAll로 열어두는 것은 프로덕션에서 심각한 보안 취약점이 될 수 있습니다. 향후 적절한 인증/인가 처리로 전환하는 것을 권장합니다.


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.

@neibler neibler merged commit 8df95f3 into develop Feb 10, 2026
1 check 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