Skip to content

fix: prod CORS 설정 목록에 어드민 페이지 추가#727

Merged
whqtker merged 1 commit into
developfrom
fix/726-add-www-admins-cors-allowed-origin
Jun 2, 2026
Merged

fix: prod CORS 설정 목록에 어드민 페이지 추가#727
whqtker merged 1 commit into
developfrom
fix/726-add-www-admins-cors-allowed-origin

Conversation

@whqtker
Copy link
Copy Markdown
Member

@whqtker whqtker commented Jun 1, 2026

관련 이슈

작업 내용

image

특이 사항

리뷰 요구사항 (선택)

@whqtker whqtker self-assigned this Jun 1, 2026
@whqtker whqtker added the 버그 Something isn't working label Jun 1, 2026
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Jun 1, 2026

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Run ID: 32835e22-349a-44f8-98fe-cab361819bae

📥 Commits

Reviewing files that changed from the base of the PR and between 67aad12 and 9dad5f4.

📒 Files selected for processing (1)
  • src/main/resources/config/application-variable.yml

🔍 Walkthrough

이 변경 사항은 프로덕션 환경의 CORS 설정을 간단하게 확장하는 작은 수정입니다.

  1. 프로덕션 CORS 허용 오리진 추가
    • application-variable.yml 파일의 prod 프로파일에서 CORS 허용 오리진 목록에 https://www.admins.solid-connection.com이 추가되었습니다.
    • 이를 통해 관리자 도메인에서 서버로의 크로스 오리진 요청이 프로덕션 환경에서 허용됩니다.

📊 Estimated Code Review Effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

👥 Suggested Reviewers

  • wibaek
  • Hexeong
  • Gyuhyeok99
  • lsy1307
🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 inconclusive)

Check name Status Explanation Resolution
Description check ❓ Inconclusive PR 설명이 필수 섹션 중 '관련 이슈' 항목은 포함하고 있으나, '작업 내용' 섹션에서 구체적인 텍스트 설명이 부족하고 이미지만 첨부되어 있습니다. 작업 내용 섹션에 변경사항에 대한 구체적인 텍스트 설명을 추가하여, 이미지가 로드되지 않을 경우에도 변경 내용을 이해할 수 있도록 명확히 해주세요.
✅ Passed checks (4 passed)
Check name Status Explanation
Title check ✅ Passed PR 제목이 주요 변경사항을 명확하게 설명하고 있으며, prod 환경의 CORS 설정에 어드민 페이지 URL을 추가하는 내용과 정확히 일치합니다.
Linked Issues check ✅ Passed PR의 코드 변경사항이 #726 이슈의 요구사항을 충족하고 있습니다. prod 환경의 CORS 설정에 https://www.admins.solid-connection.com을 추가하여 어드민 페이지의 CORS 오류를 해결합니다.
Out of Scope Changes check ✅ Passed 모든 변경사항이 #726 이슈의 범위 내에 있으며, prod 환경의 CORS 설정에만 어드민 페이지 origin을 추가하는 정확한 수정사항입니다.
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 unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/726-add-www-admins-cors-allowed-origin

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.

@whqtker whqtker added the 최종 리뷰 최소 1명 필수 label Jun 1, 2026
@whqtker whqtker merged commit c7407b9 into develop Jun 2, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

버그 Something isn't working 최종 리뷰 최소 1명 필수

Projects

None yet

Development

Successfully merging this pull request may close these issues.

fix: 어드민 페이지 CORS 설정 목록에 추가

2 participants