Skip to content

Comments

!HOTFIX: 프록시 변경#249

Merged
jjangminii merged 3 commits intodevelopfrom
fix/#247/thumbnail-title-not-loading
Feb 1, 2026
Merged

!HOTFIX: 프록시 변경#249
jjangminii merged 3 commits intodevelopfrom
fix/#247/thumbnail-title-not-loading

Conversation

@jjangminii
Copy link
Collaborator

@jjangminii jjangminii commented Feb 1, 2026

📌 Related Issues

관련된 Issue를 태그해주세요. (e.g. - close #25)

  • close #

📄 Tasks

⭐ PR Point (To Reviewer)

📷 Screenshot

Summary by CodeRabbit

  • Chores
    • 메타데이터 조회 서비스의 프록시 구성을 업데이트하여 안정성을 개선했습니다.

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

@vercel
Copy link

vercel bot commented Feb 1, 2026

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

Project Deployment Actions Updated (UTC)
pinback-client-client Ready Ready Preview, Comment Feb 1, 2026 5:28am
pinback-client-landing Ready Ready Preview, Comment Feb 1, 2026 5:28am

@coderabbitai
Copy link

coderabbitai bot commented Feb 1, 2026

Walkthrough

fetchOgData.ts의 CORS 프록시 설정을 교체했습니다. CORSProxy.io 항목을 주석 처리하고 Codetabs 프록시를 활성화하여 프록시 URL 생성 및 HTML 추출에 사용합니다. 기존 AllOrigins 항목은 계속 주석 상태로 유지되며, 기능 흐름은 변경 없습니다.

Changes

Cohort / File(s) Summary
CORS 프록시 설정 교체
apps/client/src/shared/utils/fetchOgData.ts
CORSProxy.io 프록시를 Codetabs 프록시로 교체. 매개변수명 조정(targetUrl → url, response → res). 병렬 쿼리, HTML 유효성 검사, parseOgData 파싱, 폴백 로직은 유지.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

  • !HOTFIX: 프록시 수정 #248: fetchOgData.ts의 proxyServices 설정을 동일하게 수정하며, CORSProxy.io를 다른 프록시로 교체하는 동일한 목적의 변경사항입니다.

Suggested labels

fix

Suggested reviewers

  • jllee000
  • constantly-dev

Poem

🐰 프록시 교체의 춤을 춘다네,
Codetabs로 갈아타며,
CORS의 벽을 우아하게 넘고,
데이터는 흐르고 흐르네.
🌐✨

🚥 Pre-merge checks | ✅ 2 | ❌ 3
❌ Failed checks (3 warnings)
Check name Status Explanation Resolution
Description check ⚠️ Warning PR 설명에 필수 섹션 헤더가 있지만 실제 내용이 비어있습니다. 관련 Issue 링크와 작업 내용이 작성되지 않았습니다. PR 설명을 완성하세요. Related Issues에 Issue 번호를 추가하고 Tasks 섹션에 프록시 변경 내용(CORSProxy.io→Codetabs)을 구체적으로 작성해주세요.
Linked Issues check ⚠️ Warning PR의 코드 변경(프록시 설정 변경)은 연결된 Issue #25(Progress 컴포넌트 구현)와 무관합니다. 코드 변경 사항이 Progress 컴포넌트 구현 요구사항을 충족하지 않습니다. PR 설명에서 Related Issues 섹션을 올바르게 채우거나, 실제 해당하는 Issue를 찾아 링크하세요. 프록시 변경과 관련된 새로운 Issue가 필요할 수 있습니다.
Out of Scope Changes check ⚠️ Warning PR 제목과 코드 변경은 프록시 설정 수정과 관련되어 있으나, 연결된 Issue #25는 Progress 컴포넌트 구현입니다. 이는 범위 불일치입니다. PR에 올바른 Issue를 연결하거나 새로운 Issue를 생성하세요. 프록시 변경이 Progress 컴포넌트 작업과 어떻게 관련되는지 명확히 하세요.
✅ Passed checks (2 passed)
Check name Status Explanation
Title check ✅ Passed PR 제목이 코드 변경 사항과 일치합니다. CORS 프록시를 Codetabs로 변경하는 핫픽스 작업을 명확히 표현했습니다.
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 docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch fix/#247/thumbnail-title-not-loading

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.

@jjangminii jjangminii changed the title Fix/#247/thumbnail title not loading !HOTFIX: 프록시 변경 Feb 1, 2026
@github-actions
Copy link

github-actions bot commented Feb 1, 2026

✅ Storybook chromatic 배포 확인:
🐿️ storybook

@jjangminii jjangminii merged commit 9084f02 into develop Feb 1, 2026
9 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