refactor(grpc.go, interceptors.go, utils.go): change function name fr…

…om setNewGRPCTraceHeaderFromContext to SetNewGRPCTraceHeaderFromContext to follow Go naming conventions and improve semantics
shumkovdenis · May 11, 2023 · 88a5ba4 · 88a5ba4
1 parent 00579c8
commit 88a5ba4
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 2 deletions.
diff --git a/services/integration/grpc.go b/services/integration/grpc.go
@@ -66,6 +66,7 @@ func (s *GRPCServer) GetBalance(
 	res := connect.NewResponse(&integration.GetBalanceResponse{
 		Balance: resBalance.Msg.Balance,
 	})
+	helpers.SetNewGRPCTraceHeaderFromContext(res.Header(), ctx)
 
 	return res, nil
 }
diff --git a/services/integration/helpers/interceptors.go b/services/integration/helpers/interceptors.go
@@ -39,7 +39,7 @@ func NewTraceInterceptor() connect.UnaryInterceptorFunc {
 		) (connect.AnyResponse, error) {
 			if req.Spec().IsClient {
 				// setTraceHeaderFromContext(grpcTraceBinContextKey, req.Header(), ctx)
-				setNewGRPCTraceHeaderFromContext(req.Header(), ctx)
+				SetNewGRPCTraceHeaderFromContext(req.Header(), ctx)
 			} else {
 				ctx = WithTrace(ctx,
 					req.Header().Get(traceParentHeader),

diff --git a/services/integration/helpers/utils.go b/services/integration/helpers/utils.go
@@ -139,7 +139,7 @@ func BinaryFromSpanContext(sc trace.SpanContext) []byte {
 	return b[:]
 }
 
-func setNewGRPCTraceHeaderFromContext(header HeaderSetter, ctx context.Context) {
+func SetNewGRPCTraceHeaderFromContext(header HeaderSetter, ctx context.Context) {
 	value := ExtractGRPCTraceBin(ctx)
 	log.Println("ExtractGRPCTraceBin", value)
 	if value != "" {