Skip to content

feat: ip 기능 추가#14

Merged
choihooo merged 1 commit into
mainfrom
develop
Jan 25, 2026
Merged

feat: ip 기능 추가#14
choihooo merged 1 commit into
mainfrom
develop

Conversation

@choihooo

@choihooo choihooo commented Jan 25, 2026

Copy link
Copy Markdown
Member

📝 변경 사항

🎯 작업 목적

🔍 변경 내용 상세

  • 변경 사항 1
  • 변경 사항 2
  • 변경 사항 3

📸 스크린샷 (선택사항)

✅ 체크리스트

  • 코드 스타일이 프로젝트 가이드라인을 따릅니다
  • 자체 검토를 완료했습니다
  • 주석을 추가했으며, 특히 이해하기 어려운 부분에 주석을 달았습니다
  • 문서를 업데이트했습니다 (필요한 경우)
  • 변경 사항이 새로운 경고를 생성하지 않습니다
  • 테스트를 추가했으며, 기존 테스트를 통과합니다 (필요한 경우)
  • 의존성 변경이 있다면 package.json을 업데이트했습니다

🔗 관련 이슈

Closes #

💬 리뷰어에게 전달할 사항

Summary by CodeRabbit

  • Chores
    • Optimized API request routing to improve server and client-side request handling.
    • Configured API request proxying to enhance performance and ensure proper request distribution.

✏️ Tip: You can customize this high-level summary in your review settings.

@vercel

vercel Bot commented Jan 25, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Review Updated (UTC)
119-web-client Ready Ready Preview, Comment Jan 25, 2026 2:16pm

@coderabbitai

coderabbitai Bot commented Jan 25, 2026

Copy link
Copy Markdown

Caution

Review failed

The pull request is closed.

📝 Walkthrough

Walkthrough

The changes implement client-side API request rewriting by modifying the base URL logic and adding a Next.js rewrites configuration. The API client now returns a relative path on the client while preserving absolute URLs server-side, enabling requests to be proxied through Next.js to the external API endpoint.

Changes

Cohort / File(s) Summary
API Proxy Configuration
lib/api/client.ts, next.config.ts
Modified getBaseUrl() to conditionally return relative path /api/v1 on client (when window is undefined) and absolute URL on server; added async rewrites() configuration to proxy requests from /api/:path* to https://api.daepiro.site/api/:path*

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Poem

🐰 bounce bounce
API routes now dance with grace,
Client whispers, server relays,
Rewrites blend both near and far,
Proxied paths shine like a star! ✨

✨ Finishing touches
  • 📝 Generate docstrings

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.

@choihooo choihooo merged commit 06a8a20 into main Jan 25, 2026
6 of 7 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