Skip to content

fix: admin 웹 도메인 변경에 따른 cors 허용 도메인 수정#722

Merged
Hexeong merged 1 commit into
developfrom
fix/721-change-admin-web-domain
May 25, 2026
Merged

fix: admin 웹 도메인 변경에 따른 cors 허용 도메인 수정#722
Hexeong merged 1 commit into
developfrom
fix/721-change-admin-web-domain

Conversation

@Hexeong
Copy link
Copy Markdown
Contributor

@Hexeong Hexeong commented May 25, 2026

관련 이슈

작업 내용

fix: admin 웹 도메인 변경에 따른 cors 허용 도메인 수정

특이 사항

리뷰 요구사항 (선택)

@Hexeong Hexeong self-assigned this May 25, 2026
@Hexeong Hexeong added the 버그 Something isn't working label May 25, 2026
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented May 25, 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: 9cce62aa-18cf-4158-8b1f-a76ab50a888a

📥 Commits

Reviewing files that changed from the base of the PR and between dea5689 and 67e2087.

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

전체 변경 요약

이 PR은 프로덕션 환경의 CORS(Cross-Origin Resource Sharing) 설정을 정밀하게 업데이트합니다. application-variable.yml 파일의 prod 프로파일에서 허용된 오리진 호스트명을 한 글자 변경하여, 기존의 admin.solid-connection.com에서 새로운 admins.solid-connection.com으로 전환합니다.

변경사항 상세

  1. CORS 허용 오리진 도메인 수정
    • 파일: src/main/resources/config/application-variable.yml
    • 프로파일: prod
    • 변경: https://admin.solid-connection.comhttps://admins.solid-connection.com
    • 영향: 프로덕션 환경에서 브라우저 기반 요청을 허용하는 도메인이 변경됨

예상 코드 리뷰 난이도

🎯 1 (Trivial) | ⏱️ ~2 분

추천 리뷰어

  • wibaek
  • whqtker
  • JAEHEE25
  • lsy1307
  • sukangpunch
  • Gyuhyeok99
🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed PR 제목은 CORS 허용 도메인 수정이라는 주요 변경사항을 명확하게 설명하고 있습니다.
Description check ✅ Passed PR 설명은 템플릿 구조를 따르고 있으며, 관련 이슈(#721) 참조 및 작업 내용이 포함되어 있습니다.
Linked Issues check ✅ Passed PR의 코드 변경사항이 연결된 이슈 #721의 목표를 완벽하게 충족합니다. admin 도메인에서 admins으로의 변경에 따른 CORS 설정 업데이트가 이루어졌습니다.
Out of Scope Changes check ✅ Passed 모든 변경사항이 이슈 #721의 범위 내에 있으며, CORS 허용 도메인 설정 변경만 포함되어 있습니다.
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/721-change-admin-web-domain

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.

@Hexeong Hexeong merged commit 67aad12 into develop May 25, 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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

fix: cors 도메인 설정 변경

1 participant