Skip to content

[#26] Spring Actuator 설정#28

Merged
earlgrey02 merged 4 commits intodevfrom
feature/26-actuator
Feb 2, 2026
Merged

[#26] Spring Actuator 설정#28
earlgrey02 merged 4 commits intodevfrom
feature/26-actuator

Conversation

@earlgrey02
Copy link
Member

작업 내용

  • Spring Actuator 설정
  • 헬스 체크 설정

참고

  • 헬스 체크를 위해 누락됐었던 Redis 컨테이너도 추가했습니다.

관련 이슈

@earlgrey02 earlgrey02 requested review from 1117mg and mjk25 January 31, 2026 12:52
@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 Boot Actuator 모듈 및 Redis 환경을 구성에 추가함.
  • Actuator Health Check 엔드포인트를 포함하여 보안 설정을 업데이트함.

🚨 치명적인 이슈 (Critical Issues)

  • 🛑 docker-compose.yml
    • Problem: api 서비스 정의에서 redis 컨테이너를 환경에 추가했음에도 불구하고 depends_on 목록에 redis가 누락되었다. 이는 컨테이너 시작 순서를 보장하지 않아, API 서비스 구동 시 Redis 연결 실패를 유발할 수 있다.
    • Fix: api 서비스의 depends_on 항목에 redis를 추가하여 의존성을 명시해야 한다.

✏️ 파일별 상세 리뷰

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

  • 📍 Issue: /actuator/** 전체 경로를 permitAll()로 열어두었다. 현재 YAML 설정이 health 엔드포인트만 노출하도록 제한하고 있지만, 이 패턴은 향후 구성 변경 실수 시 민감한 모든 Actuator 엔드포인트가 외부에 노출될 수 있는 보안 위험을 내포한다.
  • Improvement: permitAll() 범위를 최소한의 필수 엔드포인트(예: /actuator/health)로 명시적으로 제한하거나, 접근 권한을 관리자 역할 또는 내부 네트워크 IP로 제한하도록 수정해야 한다.

api/src/main/resources/application.yaml

  • 📍 Issue: management 설정 섹션에서 endpoint.health.access: read_only는 기본 동작(default access is read_only)이므로 명시할 필요가 없다. (불필요한 중복 설정이다.)

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.

LGTM

@earlgrey02 earlgrey02 merged commit a10ad5f into dev Feb 2, 2026
@earlgrey02 earlgrey02 deleted the feature/26-actuator branch February 2, 2026 06:41
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.

Spring Actuator 설정

2 participants