cjihrig/grpc-server-js

Implement health checking protocol

SProst opened this issue · 3 comments

https://github.com/grpc/grpc/blob/master/doc/health-checking.md

There appears to be support for this protocol in the various language bindings to GRPC. It might make sense to include it with this implementation.

So I haven't looked at the existing package in a ton of detail, but are you basically looking for https://github.com/grpc/grpc-node/tree/master/packages/grpc-health-check, but using the pure JS implementation instead of the grpc module? If so, I think we should try to work with the gRPC team to get support added there since it's just a service, and not functionality built into the server itself. If the gRPC team doesn't want to support pure JS in that module, we can fork it.

Note: As of grpc-tools@1.8.0, that module can generate code that works with grpc or grpc-js. I'm hoping the gRPC team will generate a new version of the health-check package, and that the grpc-js compatibility will work with this server as well. I'll have to investigate more.

OK, it looks like there is a plan in grpc/grpc-node#926. Since this shouldn't require any action in this repo, I'm going to close this.