Skip to content

Health Checks Liveness and Readiness Verification - #385

Merged
PeterOche merged 1 commit into
degenspot:mainfrom
Adedayo-Data:main
Aug 31, 2026
Merged

Health Checks Liveness and Readiness Verification#385
PeterOche merged 1 commit into
degenspot:mainfrom
Adedayo-Data:main

Conversation

@Adedayo-Data

Copy link
Copy Markdown
Contributor

Description

Fixes the HTTP response status code behavior in the GET /health/ready endpoint when critical dependency checks fail.

  • Injected NestJS @Res({ passthrough: true }) to dynamically update the HTTP status code.
  • Ensures the endpoint correctly responds with 503 Service Unavailable on database failure or when non-critical failures exceed the configured HEALTH_DEGRADED_THRESHOLD.
  • Updated unit tests to mock the response object and assert that a 503 status code is set under error conditions.

Verification

  • Unit tests run: pnpm --filter backend test health (Passed)
  • Application build check: pnpm --filter backend build (Passed)

closes #310

@drips-wave

drips-wave Bot commented Aug 31, 2026

Copy link
Copy Markdown

@Adedayo-Data Great news! 🎉 Based on an automated assessment of this PR, the linked Wave issue(s) no longer count against your application limits.

You can now already apply to more issues while waiting for a review of this PR. Keep up the great work! 🚀

Learn more about application limits

@Adedayo-Data Adedayo-Data changed the title FIX: bug in the HTTP status code returning on critical dependency fai… Health Checks Liveness and Readiness Verification Aug 31, 2026
@PeterOche
PeterOche merged commit 1add390 into degenspot:main Aug 31, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

BE-30 — Health Checks + Terminus Readiness/Liveness

2 participants