Skip to content

Conversation

@stoneTiger0912
Copy link
Member

@stoneTiger0912 stoneTiger0912 commented Feb 12, 2026

Summary by CodeRabbit

릴리스 노트

  • 새로운 기능
    • 이미지 비공개 경로에 대한 인증 기능이 추가되었습니다.
    • /v1/images/** 경로(업로드 제외)에 인증 필터가 적용되었습니다.

@stoneTiger0912 stoneTiger0912 self-assigned this Feb 12, 2026
@coderabbitai
Copy link

coderabbitai bot commented Feb 12, 2026

Warning

Rate limit exceeded

@stoneTiger0912 has exceeded the limit for the number of commits that can be reviewed per hour. Please wait 9 minutes and 59 seconds before requesting another review.

⌛ How to resolve this issue?

After the wait time has elapsed, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout.

Please see our FAQ for further information.

Walkthrough

스프링 클라우드 게이트웨이 설정에서 /v1/images/** 경로(업로드 제외)를 타겟으로 하고 인증 필터를 적용하는 새로운 라우트를 추가했습니다.

Changes

Cohort / File(s) Summary
Gateway Route Configuration
src/main/resources/application.yml
인증 필터가 적용된 새로운 게이트웨이 라우트 image-private 추가: http://image-service:8082로의 /v1/images/** 경로 라우팅 (단, /v1/images/upload 제외)

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

Suggested labels

enhancement

Poem

🐰 이미지 서비스의 새로운 길을 만들어,
보안 필터와 함께 춤을 춘다네!
업로드 빼곤 모두 잠기고,
게이트웨이가 똑똑해졌어! 🎭✨

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed PR 제목이 변경 사항의 주요 내용을 명확하게 반영하고 있습니다. 이미지 서비스 추가라는 제목은 application.yml에 image-private 게이트웨이 라우트를 추가한 실제 변경 사항과 일치합니다.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

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

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch feat/add-image-service

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.

Copy link

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 1

🤖 Fix all issues with AI agents
In `@src/main/resources/application.yml`:
- Around line 51-60: 현재 Path predicate에서 "!/v1/images/upload" 부정 패턴이 동작하지 않으므로
업로드를 제외하려면 라우트를 두 개로 분리하세요: 먼저 공개 라우트(id: image-public)로 Path predicate에
"/v1/images/upload"만 매칭되게 하고 filters에서 AuthenticationFilter를 제거하여 인증 없이 통과시키고, 그
다음에 기존의 보호된 라우트(id: image-private)를 두어 Path predicate에 "/v1/images/**"를 매칭시키고
filters에 AuthenticationFilter를 적용하세요; 라우트 순서(image-public 먼저, image-private 다음)를
보장하면 "/v1/images/upload"는 공개 라우트가 처리되고 나머지 경로는 인증 라우트로 처리됩니다.

@stoneTiger0912 stoneTiger0912 merged commit 34c3209 into main Feb 12, 2026
3 checks passed
@stoneTiger0912 stoneTiger0912 deleted the feat/add-image-service branch February 12, 2026 10:43
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