Skip to content

Conversation

@sg-gs
Copy link
Member

@sg-gs sg-gs commented Feb 10, 2026

This pull request adds a comprehensive health check feature to the application, enabling monitoring of both database and S3 avatar bucket connectivity. The health check is exposed via a new endpoint and includes robust error handling and test coverage. The changes are grouped below by theme:

Health Check Feature Implementation

  • Added a new HealthModule with HealthService and HealthController to provide a /healthz endpoint for health checks, reporting status, uptime, and connectivity to DB and S3. (src/modules/health/health.module.ts, src/modules/health/health.controller.ts, src/modules/health/health.service.ts) [1] [2] [3]
  • Integrated HealthModule into the main application module, registering it as an import. (src/app.module.ts) [1] [2]

Avatar Service Enhancements

  • Added a checkBucket method to AvatarService to verify S3 bucket accessibility with timeout support, used by the health check. (src/externals/avatar/avatar.service.ts) [1] [2]

Testing

  • Added unit tests for HealthService to verify health check logic under normal, degraded, and error conditions. (src/modules/health/health.service.spec.ts)

@sg-gs sg-gs self-assigned this Feb 10, 2026
@sg-gs sg-gs requested a review from jzunigax2 as a code owner February 10, 2026 17:14
@sg-gs sg-gs added the enhancement New feature or request label Feb 10, 2026
@sonarqubecloud
Copy link

Quality Gate Failed Quality Gate failed

Failed conditions
45.8% Coverage on New Code (required ≥ 80%)

See analysis details on SonarQube Cloud

@jzunigax2 jzunigax2 merged commit d1dac89 into master Feb 11, 2026
6 of 7 checks passed
@jzunigax2 jzunigax2 deleted the feat/add-health-check branch February 11, 2026 05:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants