Skip to content

fix: use SELECT 1 for health checks #518#574

Closed
edehvictor wants to merge 3 commits into
emdevelopa:mainfrom
edehvictor:fix/health-check-select1-18
Closed

fix: use SELECT 1 for health checks #518#574
edehvictor wants to merge 3 commits into
emdevelopa:mainfrom
edehvictor:fix/health-check-select1-18

Conversation

@edehvictor

@edehvictor edehvictor commented Apr 22, 2026

Copy link
Copy Markdown
Contributor

Closes #518


Refs #18

Changes

  • replace the health-check database probe with SELECT 1 through the shared pg pool
  • reuse the same database and Horizon probe logic in both �ackend/src/app.js and �ackend/src/index.js
  • add focused unit coverage for the new health probe helper

Testing

  • npm test -- src/lib/health.test.js
  • npm test (currently fails in this branch because of existing missing dependencies/env such as pino, jsonwebtoken, supertest, prom-client, and SUPABASE_URL/SUPABASE_SERVICE_ROLE_KEY)

@vercel

vercel Bot commented Apr 22, 2026

Copy link
Copy Markdown

@edehvictor is attempting to deploy a commit to the Emmanuel's projects Team on Vercel.

A member of the Team first needs to authorize it.

@edehvictor edehvictor changed the title fix: use SELECT 1 for health checks fix: use SELECT 1 for health checks #518 Apr 22, 2026
@edehvictor edehvictor closed this Apr 22, 2026
@edehvictor edehvictor reopened this Apr 22, 2026
@edehvictor edehvictor closed this Apr 25, 2026
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.

[DX] Enhance Command Palette with Global Navigation and Actions

1 participant