fullstorydev / grpchan

Channels for gRPC: custom transports
MIT License
205 stars 23 forks source link

Call options and server transport streams: Unary response metadata now works (FTW!) #9

Closed jhump closed 6 years ago

jhump commented 6 years ago

This closes #6.

This uses recently landed changes in the grpc package to support call options and unary response metadata, in both the client and server.

This adds some new stuff to the channel test cases to verify it all works with both httpgrpc and inprocgrpc implementations. It also adds a test for the channel test cases (which just runs them all against a standard *grpc.Server and *grpc.ClientConn to ensure that the test cases themselves are asserting that channel behavior matches the "reference implementation").

Most of the seven commits are reasonably well-contained if you want to review them one-by-one. There is a little bit of re-work that appears in the third and sixth commits (that cleans up stuff from the prior commits).