Skip to content

chore: cors 오리진 추가#177

Merged
SeongHo5356 merged 1 commit into
developfrom
chore/176-cors
Feb 28, 2026
Merged

chore: cors 오리진 추가#177
SeongHo5356 merged 1 commit into
developfrom
chore/176-cors

Conversation

@SeongHo5356
Copy link
Copy Markdown
Member

@SeongHo5356 SeongHo5356 commented Feb 28, 2026

✨ Related Issue


📌 Task Details

  • Task 1
  • Task 2

💬 Review Requirements (Optional)

Summary by CodeRabbit

릴리스 노트

  • 기타 (Chores)
    • 새로운 프로덕션 및 스테이징 환경(prd.ilhaeng.cloud, stg.ilhaeng.cloud)에 대한 서버 구성 추가
    • API 문서의 서버 정보 업데이트

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Feb 28, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between d0e4630 and 1774796.

📒 Files selected for processing (2)
  • src/main/java/KUSITMS/WITHUS/global/config/CorsConfig.java
  • src/main/java/KUSITMS/WITHUS/global/config/SwaggerConfig.java

Walkthrough

CORS 설정에 두 개의 새로운 도메인(prd.ilhaeng.cloud, stg.ilhaeng.cloud)이 추가되었고, Swagger 설정에서 프로덕션 서버 엔트리가 Oracle 프로덕션, Oracle 스테이징, NCP 프로덕션 서버로 확장되었습니다. 제어 흐름과 메서드 서명은 변경되지 않았습니다.

Changes

Cohort / File(s) Summary
인프라 설정 업데이트
src/main/java/KUSITMS/WITHUS/global/config/CorsConfig.java, src/main/java/KUSITMS/WITHUS/global/config/SwaggerConfig.java
CORS 허용 원점에 Oracle 관련 도메인 2개 추가 및 Swagger 프로덕션 서버 엔트리를 3개 서버로 확장(Oracle 프로덕션, Oracle 스테이징, NCP 프로덕션)

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10분

Poem

🐰 구름 너머 새로운 길을 열고,
서버들이 춤을 추기 시작했나요?
설정의 마법으로 도메인이 늘어나고,
Swagger는 더욱 정확한 지도를 그립니다.
CORS와 함께, 더 넓은 세상으로! ✨

🚥 Pre-merge checks | ✅ 1 | ❌ 2

❌ Failed checks (2 warnings)

Check name Status Explanation Resolution
Description check ⚠️ Warning 설명이 템플릿 구조를 따르고 있지만, 관련 이슈, 작업 세부 사항, 검토 요구 사항이 모두 비어있거나 완성되지 않았습니다. Related Issue와 Task Details를 구체적으로 작성하고, 변경 사항의 목적과 영향 범위에 대한 설명을 추가하세요.
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.
✅ Passed checks (1 passed)
Check name Status Explanation
Title check ✅ Passed 제목이 주요 변경 사항을 명확하게 요약하고 있습니다. CORS 오리진 추가라는 핵심 변경을 정확하게 반영합니다.

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

✨ Finishing Touches
  • 📝 Generate docstrings (stacked PR)
  • 📝 Generate docstrings (commit on current branch)
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch chore/176-cors

Tip

Try Coding Plans. Let us write the prompt for your AI agent so you can ship faster (with fewer bugs).
Share your feedback on Discord.


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.

@SeongHo5356 SeongHo5356 merged commit 0d726f5 into develop Feb 28, 2026
3 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