fix: /notice/keywords 엔드포인트를 USER 권한으로 제한 - #335
Conversation
- permitAll에서 /notice/** 제거 - JWT 필터 ignore 목록에서 /notice/** 제거 - /notice/keywords, /notice/keywords/** 를 user 권한 경로로 이동 (토큰 없이 호출 시 SecurityContext 미설정으로 인증 실패하던 문제 수정) Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
|
Warning Review limit reached
More reviews will be available in 57 minutes and 30 seconds. Learn how PR review limits work. Your organization has used up its prepaid credits, and credit purchases are no longer available. Enable the review add-on in the billing tab to keep reviews running — you're only billed for reviews past your plan's rate limits ($0.25/file). ⌛ How to resolve this issue?After more reviews become available, a review can be triggered using the To avoid repeated limits, reduce automatic review volume by pausing incremental auto-reviews earlier, using label-based review opt-in, excluding WIP or generated PR titles, or requesting reviews manually when the PR is ready. If your team needs uninterrupted high-volume reviews, an organization admin can enable usage-based credits. 🚦 How do rate limits work?CodeRabbit enforces per-developer PR review limits for each organization. Most developers receive the normal plan refill rate. For paid Pro and Pro+ PR reviews, CodeRabbit uses adaptive limits for sustained high-volume activity. When a developer's recent PR review activity reaches the 95th percentile or higher among CodeRabbit users, the refill rate gradually slows as usage increases. The highest same-day bursts are limited more strictly. Please see our Fair Usage Limits Policy for further information. ℹ️ Review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
📝 WalkthroughWalkthrough
Changes공지 경로 인증 정책 변경
추정 코드 리뷰 노력🎯 1 (Trivial) | ⏱️ ~3 minutes Poem
🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 inconclusive)
✅ Passed checks (4 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
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. Comment |
- /notice/{departmentType} (공지 목록) 은 비인증 접근이 필요하므로 /notice/** 를 permitAll에 유지
- Security 매칭 순서상 /notice/keywords 가 userAllowedPaths에서 먼저 잡히므로 의도대로 동작
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Summary
permitAll에 등록된/notice/**가 JWT 필터 ignore 목록에도 포함되어, Bearer 토큰을 전송해도 SecurityContext에 인증 정보가 설정되지 않는 문제 수정/notice/keywords(GET/POST/DELETE) 를 USER 권한 경로로 이동하여 토큰 기반 인증이 정상 동작하도록 수정Changes
authentication.path.all에서/notice/**제거authentication.filter.ignore.paths에서/notice/**제거authentication.path.user에/notice/keywords,/notice/keywords/**추가Test plan
GET /notice/keywords호출 → 401 응답 확인GET /notice/keywords호출 → 200 + 본인 키워드 목록 반환 확인POST /notice/keywords호출 → 201 응답 확인DELETE /notice/keywords/{id}호출 → 204 응답 확인🤖 Generated with Claude Code
Summary by CodeRabbit
릴리스 노트