Skip to content

GRPC health endpoint with List, Check and Watch methods#114

Merged
TxCorpi0x merged 3 commits into
masterfrom
mehdi/grpc-health-server
Apr 2, 2026
Merged

GRPC health endpoint with List, Check and Watch methods#114
TxCorpi0x merged 3 commits into
masterfrom
mehdi/grpc-health-server

Conversation

@TxCorpi0x

@TxCorpi0x TxCorpi0x commented Apr 1, 2026

Copy link
Copy Markdown
Contributor

Description

This pull request introduces gRPC health checking support to the application by registering a health server with the gRPC query router. This enables external systems to check the health status of the gRPC service.

gRPC Health Checking:

  • Added imports for google.golang.org/grpc/health and google.golang.org/grpc/health/grpc_health_v1 to support gRPC health checking functionality in app.go.
  • Created a new gRPC health server, set its serving status, and registered it with the gRPC query router to enable health check endpoints.

Reviewers checklist:

  • Try to write more meaningful comments with clear actions to be taken.
  • Nit-picking should be unblocking. Focus on core issues.

Authors checklist

  • Provide a concise and meaningful description
  • Review the code yourself first, before making the PR.
  • Annotate your PR in places that require explanation.
  • Think and try to split the PR to smaller PR if it is big.

This change is Reviewable

@TxCorpi0x TxCorpi0x requested a review from a team as a code owner April 1, 2026 14:04
@TxCorpi0x TxCorpi0x requested review from akhlopiachyi, masihyeganeh, metalarm10, miladz68, tx-development and ysv and removed request for a team April 1, 2026 14:04

@metalarm10 metalarm10 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@metalarm10 reviewed 1 file and all commit messages.
Reviewable status: :shipit: complete! all files reviewed, all discussions resolved (waiting on akhlopiachyi, masihyeganeh, miladz68, tx-development, and ysv).

@TxCorpi0x TxCorpi0x merged commit 60f985e into master Apr 2, 2026
14 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.

3 participants