-
-
Notifications
You must be signed in to change notification settings - Fork 2.5k
Closed
Description
Was just looking through the grpc metadata docs the other day and noticed metadata.NewContext
and metadata.FromContext
have been deprecated. From grpc-metadata looks seems the new prefer way is to use FromIncomingContext
and NewOutgoingContext
respectively (granted there does look to be a bug in the doc where it still states NewContext
in one place, the examples had it change).
I know that FromContext
is used in grpc/service.go and NewContext
in grpc/client.go.
This is definitely not a big thing at all, but wasn't sure if it had be brought to your attention yet or if there were any plans to replace those with FromIncomingContext
and NewOutgoingContext
. If there aren't and it's alright, can push up a quick change for that.