Merge pull request #49 from crosbymichael/status

estesp · web-flow · commit 92c8520ef9f8 · 2019-08-28T13:29:38.000-04:00
Handle ok status
diff --git a/client.go b/client.go
@@ -29,6 +29,7 @@ import (
 	"github.com/gogo/protobuf/proto"
 	"github.com/pkg/errors"
 	"github.com/sirupsen/logrus"
+	"google.golang.org/grpc/codes"
 	"google.golang.org/grpc/status"
 )
 
@@ -134,10 +135,9 @@ func (c *Client) Call(ctx context.Context, service, method string, req, resp int
 		return err
 	}
 
-	if cresp.Status != nil {
+	if cresp.Status != nil && cresp.Status.Code != int32(codes.OK) {
 		return status.ErrorProto(cresp.Status)
 	}
-
 	return nil
 }
 

Original file line number	Diff line number	Diff line change
`@@ -29,6 +29,7 @@ import (`
`29`	`29`	`"github.com/gogo/protobuf/proto"`
`30`	`30`	`"github.com/pkg/errors"`
`31`	`31`	`"github.com/sirupsen/logrus"`
	`32`	`+ "google.golang.org/grpc/codes"`
`32`	`33`	`"google.golang.org/grpc/status"`
`33`	`34`	`)`
`34`	`35`
`@@ -134,10 +135,9 @@ func (c *Client) Call(ctx context.Context, service, method string, req, resp int`
`134`	`135`	`return err`
`135`	`136`	`}`
`136`	`137`
`137`		`- if cresp.Status != nil {`
	`138`	`+ if cresp.Status != nil && cresp.Status.Code != int32(codes.OK) {`
`138`	`139`	`return status.ErrorProto(cresp.Status)`
`139`	`140`	`}`
`140`		`-`
`141`	`141`	`return nil`
`142`	`142`	`}`
`143`	`143`