Skip to content

fix: 배포환경 수정#30

Merged
kim-subsub merged 1 commit into
developfrom
feat/27
Apr 24, 2026
Merged

fix: 배포환경 수정#30
kim-subsub merged 1 commit into
developfrom
feat/27

Conversation

@kim-subsub

@kim-subsub kim-subsub commented Apr 24, 2026

Copy link
Copy Markdown
Contributor

Summary by CodeRabbit

릴리스 노트

  • 새 기능

    • 프록시가 HTTP 및 HTTPS 백엔드 대상을 모두 지원하도록 확대
    • 단일 또는 다중 경로 접두사를 지원하여 더 유연한 요청 라우팅 구현
    • 새로운 대상 URL 빌드 기능을 모듈 내보내기에 추가
  • 테스트

    • 경로 접두사 제거 및 매칭 동작을 검증하는 새로운 테스트 스위트 추가

@vercel

vercel Bot commented Apr 24, 2026

Copy link
Copy Markdown

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

Project Deployment Actions Updated (UTC)
veri-q Ready Ready Preview, Comment Apr 24, 2026 10:34am

@coderabbitai

coderabbitai Bot commented Apr 24, 2026

Copy link
Copy Markdown

Caution

Review failed

Pull request was closed or merged during review

개요

프록시 모듈이 HTTP/HTTPS 프로토콜을 모두 지원하도록 업데이트되었으며, 단일 접두어와 접두어 배열을 모두 처리할 수 있도록 확장되었습니다. API 엔드포인트는 다중 경로 접두어를 지원하도록 구성되었고, buildTargetUrl 함수에 대한 테스트가 추가되었습니다.

변경 사항

제목 / 파일 요약
프록시 모듈 핵심 변경
api/_proxy.js
프로토콜 검증을 완화하여 HTTP와 HTTPS 백엔드 대상을 모두 허용하도록 변경. proxyRequestbuildTargetUrl이 단일 prefix 또는 prefixes 배열을 모두 지원하도록 확장. 들어오는 경로명에서 일치하는 첫 번째 프록시 접두어를 제거하는 방식으로 처리. buildTargetUrl을 모듈 내보내기에 추가.
API 엔드포인트 설정
api/be1/[...path].js, api/be3/[...path].js
proxyRequest 설정에서 단일 접두어(/api/be1, /api/be3)에서 다중 접두어(/api/be1/be1, /api/be3/be3)로 변경하여 여러 경로에서의 요청 수신 지원.
테스트 추가
api/_proxy.test.js
buildTargetUrl 함수에 대한 새로운 Vitest 테스트 스위트 추가. 접두어 완전 일치 여부 검증 및 부분 일치 거짓 양성 방지를 확인하는 테스트 케이스 포함.
🚥 Pre-merge checks | ✅ 3 | ❌ 2

❌ Failed checks (1 warning, 1 inconclusive)

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.
Title check ❓ Inconclusive 제목이 한국어로 '배포환경 수정'이라고 되어 있어 구체적인 변경 사항을 명확히 설명하지 못합니다. 제목을 보다 구체적으로 수정하세요. 예: 'fix: 프록시 설정에서 HTTP/HTTPS 지원 및 다중 경로 접두사 처리' 또는 'fix: API 프록시 요청에서 다중 접두사 지원 추가'
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

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


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.

@kim-subsub
kim-subsub merged commit 3c0dbf1 into develop Apr 24, 2026
7 of 8 checks passed
@kim-subsub
kim-subsub deleted the feat/27 branch April 24, 2026 10:35
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