Skip to content

feat: add /health endpoint#326

Open
hetdagli234 wants to merge 2 commits into
mainfrom
feat/add-health-endpoint
Open

feat: add /health endpoint#326
hetdagli234 wants to merge 2 commits into
mainfrom
feat/add-health-endpoint

Conversation

@hetdagli234
Copy link
Copy Markdown
Contributor

@hetdagli234 hetdagli234 commented Feb 6, 2026

Summary

  • Add /health HTTP GET endpoint that mirrors the existing /liveness endpoint behavior

Changes

  • Add health() method to PhotonApi in src/api/api.rs
  • Register /health route via ProxyGetRequestLayer in src/api/rpc_server.rs
  • Register health RPC method

Notes

  • The /liveness and /readiness endpoints remain unchanged
  • This is a non-breaking additive change

🤖 Generated with Claude Code

Add a /health HTTP GET endpoint that mirrors the existing /liveness
endpoint behavior. This provides consistency with other Helius services
(DAS, RPC, AccountIndex, etc.) that expose health checks at /health.

- Add health() method to PhotonApi
- Register /health route via ProxyGetRequestLayer
- Register "health" RPC method

The /liveness and /readiness endpoints remain unchanged.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@hetdagli234 hetdagli234 changed the title feat: add /health endpoint for consistency with other services feat: add /health endpoint Feb 6, 2026
Health checks should verify the service can actually serve traffic.
If the database is inaccessible, the service cannot handle requests,
so /health should fail rather than return a false positive.

This matches DAS behavior where /health also performs a DB check.
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.

2 participants