Minimize allocations writing User-Agent header (connectrpc#446)

The fmt.Sprintf stood out again in my profiling as a relatively hot
path, and through the lifetime of an application, this value will not
change. So we can generate this header value at init() time and avoid
the extra memory allocation as well on each request.

**Before submitting your PR:** Please read through the contribution
guide at
https://github.com/bufbuild/connect-go/blob/main/.github/CONTRIBUTING.md

Author: mattrobenolt

protocol_connect.go

@@ -49,6 +49,11 @@ const (
 	connectStreamingContentTypePrefix = "application/connect+"
 )
 
+// defaultConnectUserAgent returns a User-Agent string similar to those used in gRPC.
+//
+//nolint:gochecknoglobals
+var defaultConnectUserAgent = fmt.Sprintf("connect-go/%s (%s)", Version, runtime.Version())
+
 type protocolConnect struct{}
 
 // NewHandler implements protocol, so it must return an interface.
@@ -243,7 +248,7 @@ func (c *connectClient) WriteRequestHeader(streamType StreamType, header http.He
 	// We know these header keys are in canonical form, so we can bypass all the
 	// checks in Header.Set.
 	if header.Get(headerUserAgent) == "" {
-		header[headerUserAgent] = []string{connectUserAgent()}
+		header[headerUserAgent] = []string{defaultConnectUserAgent}
 	}
 	header[connectHeaderProtocolVersion] = []string{connectProtocolVersion}
 	header[headerContentType] = []string{
@@ -1028,11 +1033,6 @@ func connectHTTPToCode(httpCode int) Code {
 	}
 }
 
-// connectUserAgent returns a User-Agent string similar to those used in gRPC.
-func connectUserAgent() string {
-	return fmt.Sprintf("connect-go/%s (%s)", Version, runtime.Version())
-}
-
 func connectCodecFromContentType(streamType StreamType, contentType string) string {
 	if streamType == StreamTypeUnary {
 		return strings.TrimPrefix(contentType, connectUnaryContentTypePrefix)

protocol_grpc.go

@@ -65,6 +65,17 @@ var (
 	}
 	grpcTimeoutUnitLookup        = make(map[byte]time.Duration)
 	errTrailersWithoutGRPCStatus = fmt.Errorf("gRPC protocol error: no %s trailer", grpcHeaderStatus)
+
+	// defaultGrpcUserAgent follows
+	// https://github.com/grpc/grpc/blob/master/doc/PROTOCOL-HTTP2.md#user-agents:
+	//
+	//	While the protocol does not require a user-agent to function it is recommended
+	//	that clients provide a structured user-agent string that provides a basic
+	//	description of the calling library, version & platform to facilitate issue diagnosis
+	//	in heterogeneous environments. The following structure is recommended to library developers:
+	//
+	//	User-Agent → "grpc-" Language ?("-" Variant) "/" Version ?( " ("  *(AdditionalProperty ";") ")" )
+	defaultGrpcUserAgent = fmt.Sprintf("grpc-go-connect/%s (%s)", Version, runtime.Version())
 )
 
 func init() {
@@ -227,7 +238,7 @@ func (g *grpcClient) WriteRequestHeader(_ StreamType, header http.Header) {
 	// We know these header keys are in canonical form, so we can bypass all the
 	// checks in Header.Set.
 	if header.Get(headerUserAgent) == "" {
-		header[headerUserAgent] = []string{grpcUserAgent()}
+		header[headerUserAgent] = []string{defaultGrpcUserAgent}
 	}
 	header[headerContentType] = []string{grpcContentTypeFromCodecName(g.web, g.Codec.Name())}
 	// gRPC handles compression on a per-message basis, so we don't want to
@@ -762,19 +773,6 @@ func grpcEncodeTimeout(timeout time.Duration) (string, error) {
 	return "", errNoTimeout
 }
 
-// grpcUserAgent follows
-// https://github.com/grpc/grpc/blob/master/doc/PROTOCOL-HTTP2.md#user-agents:
-//
-//	While the protocol does not require a user-agent to function it is recommended
-//	that clients provide a structured user-agent string that provides a basic
-//	description of the calling library, version & platform to facilitate issue diagnosis
-//	in heterogeneous environments. The following structure is recommended to library developers:
-//
-//	User-Agent → "grpc-" Language ?("-" Variant) "/" Version ?( " ("  *(AdditionalProperty ";") ")" )
-func grpcUserAgent() string {
-	return fmt.Sprintf("grpc-go-connect/%s (%s)", Version, runtime.Version())
-}
-
 func grpcCodecFromContentType(web bool, contentType string) string {
 	if (!web && contentType == grpcContentTypeDefault) || (web && contentType == grpcWebContentTypeDefault) {
 		// implicitly protobuf