Skip to content

fix: filter Component 제외#120

Merged
wlgns12370 merged 1 commit intomainfrom
feat/#118
Mar 17, 2026
Merged

fix: filter Component 제외#120
wlgns12370 merged 1 commit intomainfrom
feat/#118

Conversation

@wlgns12370
Copy link
Contributor

@wlgns12370 wlgns12370 commented Mar 17, 2026

✨ 구현한 기능

  • filter 등록 수정

Summary by CodeRabbit

변경 사항

  • New Features

    • API v1 엔드포인트(/api/v1/*)에 클라이언트 IP 기반 필터링 기능이 추가되었습니다.
  • Chores

    • 요청 처리 파이프라인의 필터 체인 구성이 최적화되었습니다.

@wlgns12370 wlgns12370 merged commit 44c6d97 into main Mar 17, 2026
1 of 2 checks passed
@coderabbitai
Copy link

coderabbitai bot commented Mar 17, 2026

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Run ID: 8a8e8ebe-508b-4d3f-9b04-d90e2beaf8d8

📥 Commits

Reviewing files that changed from the base of the PR and between e0b6243 and 3d7db1b.

📒 Files selected for processing (2)
  • src/main/java/kr/co/knuserver/global/config/FilterConfig.java
  • src/main/java/kr/co/knuserver/global/filter/ClientIpFilter.java

📝 Walkthrough

Walkthrough

ClientIpFilter의 Spring 빈 등록 방식을 리팩토링합니다. @Component 어노테이션을 제거하고, FilterConfig 클래스에서 명시적 clientIpFilter() 메서드를 통해 빈을 등록하도록 변경합니다. 필터는 /api/v1/* URL 패턴에 순서 1로 할당됩니다.

Changes

Cohort / File(s) Summary
필터 빈 등록
src/main/java/kr/co/knuserver/global/config/FilterConfig.java
ClientIpFilter를 등록하는 새로운 FilterRegistrationBean 메서드(clientIpFilter()) 추가. /api/v1/* 패턴에 순서 1로 설정.
필터 구현
src/main/java/kr/co/knuserver/global/filter/ClientIpFilter.java
@component 어노테이션 및 해당 임포트 제거. 필터 로직은 변경 없음.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Poem

토끼가 설정을 정돈하네 🐰
어노테이션 지우고 빈을 명시적으로 등록하니
필터의 여행이 더욱 선명해졌어 ✨
순서 1번으로 API 요청을 맞이하고
더욱 깔끔한 구조로 도약하는 모습이 멋져! 🏃‍♂️

✨ Finishing Touches
  • 📝 Generate docstrings (stacked PR)
  • 📝 Generate docstrings (commit on current branch)
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch feat/#118
📝 Coding Plan
  • Generate coding plan for human review comments

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.

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