fix: Health check Redis 상태 실제 PING 체크로 변경 (#156) #157

seonkyu.kim · 2026-02-11T01:50:14Z

seonkyu.kim commented

2026-02-11 01:50:14 +00:00

📋 작업 요약

Health check에서 Redis 상태가 not_configured로 하드코딩되어 있던 문제 수정
실제 Redis PING 체크로 변경, 응답에 latency 포함

Closes #156

🛠️ 작업 내용 (Changes)

SPMS.API/Controllers/PublicController.cs — RedisConnection 주입, PingAsync() 체크

📢 리뷰어 참고 사항 (To Reviewers)

Redis 연결 실패 시 unhealthy + error 메시지 반환
정상 시 healthy + latency(ms) 반환

✅ 체크리스트 (Self Checklist)

빌드(Build)가 성공적으로 수행되었는가?
모든 단위 테스트(Unit Test)를 통과하였는가?
불필요한 로그나 주석을 제거하였는가?
컨벤션(Clean Architecture, Naming)을 준수하였는가?
기밀 정보(비밀번호, 키 등)가 하드코딩 되어있지 않은가?

📸 스크린샷 / 테스트 로그 (Screenshots/Logs)

Build succeeded. 0 Warning(s) 0 Error(s)

## 📋 작업 요약 - Health check에서 Redis 상태가 not_configured로 하드코딩되어 있던 문제 수정 - 실제 Redis PING 체크로 변경, 응답에 latency 포함 ## 🔗 관련 이슈 (Related Issues) Closes #156 ## 🛠️ 작업 내용 (Changes) - [x] SPMS.API/Controllers/PublicController.cs — RedisConnection 주입, PingAsync() 체크 ## 📢 리뷰어 참고 사항 (To Reviewers) - Redis 연결 실패 시 unhealthy + error 메시지 반환 - 정상 시 healthy + latency(ms) 반환 ## ✅ 체크리스트 (Self Checklist) - [x] 빌드(Build)가 성공적으로 수행되었는가? - [x] 모든 단위 테스트(Unit Test)를 통과하였는가? - [x] 불필요한 로그나 주석을 제거하였는가? - [x] 컨벤션(Clean Architecture, Naming)을 준수하였는가? - [x] 기밀 정보(비밀번호, 키 등)가 하드코딩 되어있지 않은가? ## 📸 스크린샷 / 테스트 로그 (Screenshots/Logs) ``` Build succeeded. 0 Warning(s) 0 Error(s) ```

seonkyu.kim added 1 commit 2026-02-11 01:50:16 +00:00

fix: Health check Redis 상태를 실제 PING 체크로 변경 (#156 ) c63a61bf6a

- not_configured 하드코딩 제거
- RedisConnection 주입 후 PingAsync()로 실제 연결 상태 확인
- 응답에 latency 포함

Closes #156

seonkyu.kim self-assigned this 2026-02-11 01:50:33 +00:00

seonkyu.kim requested review from Owners 2026-02-11 01:50:42 +00:00

seonkyu.kim added the

labels 2026-02-11 01:50:54 +00:00

seonkyu.kim added this to the Phase 3-2: 통계 & Webhook & 배치 milestone 2026-02-11 01:51:01 +00:00

seonkyu.kim merged commit f972982b85 into develop

2026-02-11 01:52:00 +00:00

seonkyu.kim deleted branch fix/#156-redis-health-check

2026-02-11 02:09:58 +00:00

seonkyu.kim added

and removed

labels 2026-02-11 02:23:26 +00:00

Sign in to join this conversation.

No reviewers

SPMS/Owners

No Label

No Milestone

No project

No Assignees

1 Participants

Notifications

Due Date

The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: SPMS/SPMS_API#157