refactor(lint/unconvert): remove base64 call returns a string (#717)

* refactor(lint/unconvert): remove base64 call returns a string

* chore(nats): fix comment on exported function

* fix(addsvc): dsl.Request requires an StringMatcher as path

tracing/opentracing/grpc.go

@@ -53,7 +53,7 @@ type metadataReaderWriter struct {
 func (w metadataReaderWriter) Set(key, val string) {
 	key = strings.ToLower(key)
 	if strings.HasSuffix(key, "-bin") {
-		val = string(base64.StdEncoding.EncodeToString([]byte(val)))
+		val = base64.StdEncoding.EncodeToString([]byte(val))
 	}
 	(*w.MD)[key] = append((*w.MD)[key], val)
 }

transport/grpc/request_response_funcs.go

@@ -69,8 +69,7 @@ func SetResponseTrailer(key, val string) ServerResponseFunc {
 func EncodeKeyValue(key, val string) (string, string) {
 	key = strings.ToLower(key)
 	if strings.HasSuffix(key, binHdrSuffix) {
-		v := base64.StdEncoding.EncodeToString([]byte(val))
-		val = string(v)
+		val = base64.StdEncoding.EncodeToString([]byte(val))
 	}
 	return key, val
 }

transport/nats/publisher.go

@@ -19,7 +19,7 @@ type Publisher struct {
 	timeout   time.Duration
 }
 
-// NewClient constructs a usable Publisher for a single remote method.
+// NewPublisher constructs a usable Publisher for a single remote method.
 func NewPublisher(
 	publisher *nats.Conn,
 	subject string,