vbatts/microservices-demo
1
0
Fork 0
Code Issues Pull requests Releases Wiki Activity

cartservice: fix health Check() method signature (#57)

Browse source 
- Use the method provided in HealthBase
- Use the same version of GrpcHealth as Grpc core.

Fixes #50.
This commit is contained in:
Ahmet Alp Balkan 2018-09-28 12:52:06 -07:00 committed by GitHub
parent c61a8aea4c
commit be65dd1fcd
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
2 changed files with 7 additions and 5 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

8
src/cartservice/HealthImpl.cs
View file

@ -1,5 +1,7 @@
using System;
using System.Threading.Tasks;
using cartservice.interfaces;
using Grpc.Core;
using Grpc.Health.V1;
using StackExchange.Redis;
using static Grpc.Health.V1.Health;
@ -11,12 +13,12 @@ namespace cartservice {
this.dependency = dependency;
}
public HealthCheckResponse Check (HealthCheckRequest request) {
public override Task<HealthCheckResponse> Check(HealthCheckRequest request, ServerCallContext context){
Console.WriteLine ("Checking CartService Health");
return new HealthCheckResponse {
return Task.FromResult(new HealthCheckResponse {
Status = dependency.Ping() ? HealthCheckResponse.Types.ServingStatus.Serving : HealthCheckResponse.Types.ServingStatus.NotServing
};
});
}
}
}

2
src/cartservice/cartservice.csproj
View file

@ -10,7 +10,7 @@
<PackageReference Include="Google.Protobuf" Version="3.6.1" />
<PackageReference Include="Google.Protobuf.Tools" Version="3.5.1" />
<PackageReference Include="grpc" Version="1.12.0" />
<PackageReference Include="Grpc.HealthCheck" Version="1.15.0" />
<PackageReference Include="Grpc.HealthCheck" Version="1.12.0" />
<PackageReference Include="grpc.tools" Version="1.12.0" />
<PackageReference Include="Microsoft.Extensions.Configuration" Version="2.1.1" />
<PackageReference Include="Microsoft.Extensions.Configuration.Json" Version="2.1.1" />