Skip to content

[#44] CORS 허용 Origin 추가#46

Closed
1117mg wants to merge 1 commit intodevfrom
feat/44-cors-origin-expand
Closed

[#44] CORS 허용 Origin 추가#46
1117mg wants to merge 1 commit intodevfrom
feat/44-cors-origin-expand

Conversation

@1117mg
Copy link
Member

@1117mg 1117mg commented Feb 5, 2026

작업 내용

  • Swagger 및 프론트 도메인 호출을 위한 CORS 허용 Origin 추가

관련 이슈

@1117mg 1117mg requested review from earlgrey02 and mjk25 February 5, 2026 06:14
@1117mg 1117mg self-assigned this Feb 5, 2026
@1117mg 1117mg added the fix 버그 수정 label Feb 5, 2026
@1117mg 1117mg linked an issue Feb 5, 2026 that may be closed by this pull request
@github-actions
Copy link

github-actions bot commented Feb 5, 2026

🤖 Gemini Code Review

✅ 변경 사항 요약

  • 쉼표로 구분된 다중 CORS 오리진을 스프링 설정에서 올바르게 처리하도록 로직을 수정했습니다.
  • WEB_URI 환경 변수를 애플리케이션 프로퍼티에 바인딩했습니다.

✏️ 파일별 상세 리뷰

WebConfiguration.kt

  • 📍 Issue: 이전 구현은 webUri 변수에 쉼표로 구분된 다중 오리진이 포함될 경우 이를 단일 문자열 오리진으로 인식하여 CORS 설정이 올바르게 동작하지 않는 기능적 결함이 있었습니다.
  • Improvement: allowedOrigins에 문자열 배열을 전달하도록 수정하여 다중 오리진을 정확하게 허용하도록 기능을 개선했습니다. 이 수정은 필수적입니다.

@1117mg 1117mg closed this Feb 5, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

fix 버그 수정

Projects

None yet

Development

Successfully merging this pull request may close these issues.

CORS 허용 Origin 추가

1 participant