Skip to content

[#27] CORS 설정#29

Merged
earlgrey02 merged 7 commits intodevfrom
feature/27-cors
Feb 2, 2026
Merged

[#27] CORS 설정#29
earlgrey02 merged 7 commits intodevfrom
feature/27-cors

Conversation

@earlgrey02
Copy link
Member

작업 내용

  • CORS(Cross-Origin Resource Sharing) 설정

관련 이슈

@earlgrey02 earlgrey02 requested review from 1117mg and mjk25 January 31, 2026 13:04
@earlgrey02 earlgrey02 self-assigned this Jan 31, 2026
@earlgrey02 earlgrey02 added the feature 기능 구현 label Jan 31, 2026
@github-actions
Copy link

🤖 Gemini Code Review

✅ 변경 사항 요약

  • Spring Actuator 및 CORS 관련 설정 추가.
  • Actuator Health 엔드포인트를 노출하도록 보안 및 애플리케이션 설정 변경.
  • Docker Compose 환경에 Redis 서비스를 추가하고 DB 서비스명을 mysql로 변경.

✏️ 파일별 상세 리뷰

api/src/main/kotlin/com/retoday/api/global/config/SecurityConfiguration.kt

  • 📍 Issue: /actuator/** 경로를 permitAll()로 설정했다.
  • Improvement: application.yaml에서 health 엔드포인트만 노출하도록 제한했지만, 관리 엔드포인트를 인증 없이 전체 허용하는 것은 보안상 위험 부담이 크므로, 운영 환경 배포 시 설정 파일의 엔드포인트 제한이 정확히 동작하는지 철저히 확인해야 한다.

api/src/main/kotlin/com/retoday/api/global/config/WebConfiguration.kt

  • 📍 Issue: 설정 값을 주입받는 방식이 @Value로 되어 있다.
  • Improvement: 설정 필드가 추가될 가능성 및 테스트 용이성을 고려하여 @ConfigurationProperties를 사용하는 것이 더 구조적이다.

docker-compose.yml

  • 📍 Issue: 데이터베이스 서비스명이 db에서 mysql로 변경되었다.
  • Improvement: 애플리케이션의 JDBC 연결 URL (spring.datasource.url)이 변경된 서비스명(mysql)을 사용하도록 애플리케이션 설정 파일(.env 또는 application.yaml)에서 호스트 이름을 즉시 수정해야 한다. 그렇지 않으면 API 및 배치 서비스가 데이터베이스 연결에 실패한다.

Copy link
Member

@1117mg 1117mg left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

DB관련 환경변수 업데이트했습니다. 고생하셨습니다.

@earlgrey02 earlgrey02 merged commit 15cf006 into dev Feb 2, 2026
@earlgrey02 earlgrey02 deleted the feature/27-cors branch February 2, 2026 14:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

feature 기능 구현

Projects

None yet

Development

Successfully merging this pull request may close these issues.

CORS 설정

2 participants