fix Driver.String()

CHANGELOG.md

@@ -3,8 +3,6 @@
 * Extended metrics (fill database.sql callbacks, recognize TLI error)
 * Refactored config prefix in metrics
 * Removed excess status labels from metrics
-
-## v3.54.3
 * Implement `String` interface for `Driver` struct
 
 ## v3.54.2

config/defaults.go

@@ -3,6 +3,7 @@ package config
 import (
 	"crypto/tls"
 	"crypto/x509"
+	"github.com/ydb-platform/ydb-go-sdk/v3/internal/stack"
 	"time"
 
 	"google.golang.org/grpc"
@@ -84,7 +85,7 @@ func defaultTLSConfig() *tls.Config {
 func defaultConfig() (c *Config) {
 	return &Config{
 		credentials: credentials.NewAnonymousCredentials(
-			credentials.WithSourceInfo("default"),
+			credentials.WithSourceInfo(stack.Record(0)),
 		),
 		balancerConfig: balancers.Default(),
 		tlsConfig:      defaultTLSConfig(),

driver.go

@@ -3,7 +3,6 @@ package ydb
 import (
 	"context"
 	"errors"
-	"fmt"
 	"os"
 	"sync"
 
@@ -211,17 +210,6 @@ func (d *Driver) Topic() topic.Client {
 	return d.topic
 }
 
-// String returns string representation of Driver
-func (d *Driver) String() string {
-	return fmt.Sprintf(
-		"Driver{User: %s, Endpoint: %s, Database: %s, IsSecure %t}",
-		d.userInfo.User,
-		d.config.Endpoint(),
-		d.config.Database(),
-		d.config.Secure(),
-	)
-}
-
 // Open connects to database by DSN and return driver runtime holder
 //
 // DSN accept Driver string like

driver_string.go

@@ -0,0 +1,22 @@
+package ydb
+
+import (
+	"fmt"
+
+	"github.com/ydb-platform/ydb-go-sdk/v3/internal/xstring"
+)
+
+// String returns string representation of Driver
+func (d *Driver) String() string {
+	buffer := xstring.Buffer()
+	defer buffer.Free()
+	buffer.WriteString("Driver{")
+	fmt.Fprintf(buffer, "Endpoint:%q,", d.config.Endpoint())
+	fmt.Fprintf(buffer, "Database:%q,", d.config.Database())
+	fmt.Fprintf(buffer, "Secure:%v", d.config.Secure())
+	if c, has := d.config.Credentials().(fmt.Stringer); has {
+		fmt.Fprintf(buffer, ",Credentials:%v", c.String())
+	}
+	buffer.WriteByte('}')
+	return buffer.String()
+}

driver_string_test.go

@@ -0,0 +1,72 @@
+package ydb
+
+import (
+	"testing"
+
+	"github.com/stretchr/testify/require"
+
+	"github.com/ydb-platform/ydb-go-sdk/v3/config"
+	"github.com/ydb-platform/ydb-go-sdk/v3/credentials"
+	"github.com/ydb-platform/ydb-go-sdk/v3/internal/xtest"
+)
+
+func TestDriver_String(t *testing.T) {
+	for _, tt := range []struct {
+		name string
+		d    *Driver
+		s    string
+	}{
+		{
+			name: xtest.CurrentFileLine(),
+			d: &Driver{config: config.New(
+				config.WithEndpoint("localhost"),
+				config.WithDatabase("local"),
+				config.WithSecure(false),
+			)},
+			s: `Driver{Endpoint:"localhost",Database:"local",Secure:false,Credentials:Anonymous{From:"github.com/ydb-platform/ydb-go-sdk/v3/config.defaultConfig(defaults.go:88)"}}`,
+		},
+		{
+			name: xtest.CurrentFileLine(),
+			d: &Driver{config: config.New(
+				config.WithEndpoint("localhost"),
+				config.WithDatabase("local"),
+				config.WithSecure(true),
+			)},
+			s: `Driver{Endpoint:"localhost",Database:"local",Secure:true,Credentials:Anonymous{From:"github.com/ydb-platform/ydb-go-sdk/v3/config.defaultConfig(defaults.go:88)"}}`,
+		},
+		{
+			name: xtest.CurrentFileLine(),
+			d: &Driver{config: config.New(
+				config.WithEndpoint("localhost"),
+				config.WithDatabase("local"),
+				config.WithSecure(false),
+				config.WithCredentials(credentials.NewAnonymousCredentials(credentials.WithSourceInfo(t.Name()))),
+			)},
+			s: `Driver{Endpoint:"localhost",Database:"local",Secure:false,Credentials:Anonymous{From:"TestDriver_String"}}`,
+		},
+		{
+			name: xtest.CurrentFileLine(),
+			d: &Driver{config: config.New(
+				config.WithEndpoint("localhost"),
+				config.WithDatabase("local"),
+				config.WithSecure(true),
+				config.WithCredentials(credentials.NewStaticCredentials("user", "password", "")),
+			)},
+			s: `Driver{Endpoint:"localhost",Database:"local",Secure:true,Credentials:Static{User:"user",Password:"pas***rd",Token:"****(CRC-32c: 00000000)",From:"github.com/ydb-platform/ydb-go-sdk/v3/credentials.NewStaticCredentials(credentials.go:35)"}}`,
+		},
+		{
+			name: xtest.CurrentFileLine(),
+			d: &Driver{config: config.New(
+				config.WithEndpoint("localhost"),
+				config.WithDatabase("local"),
+				config.WithSecure(true),
+				config.WithCredentials(credentials.NewAccessTokenCredentials("AUTH_TOKEN")),
+			)},
+			s: `Driver{Endpoint:"localhost",Database:"local",Secure:true,Credentials:AccessToken{Token:"****(CRC-32c: 9F26E847)",From:"github.com/ydb-platform/ydb-go-sdk/v3/credentials.NewAccessTokenCredentials(credentials.go:20)"}}`,
+		},
+	} {
+		t.Run(tt.name, func(t *testing.T) {
+			require.Equal(t, tt.s, tt.d.String())
+		})
+	}
+}

internal/credentials/access_token.go

@@ -46,12 +46,12 @@ func (c AccessToken) Token(_ context.Context) (string, error) {
 func (c AccessToken) String() string {
 	buffer := xstring.Buffer()
 	defer buffer.Free()
-	buffer.WriteString("AccessToken(token:")
+	buffer.WriteString("AccessToken{Token:")
 	fmt.Fprintf(buffer, "%q", secret.Token(c.token))
 	if c.sourceInfo != "" {
-		buffer.WriteString(",from:")
+		buffer.WriteString(",From:")
 		fmt.Fprintf(buffer, "%q", c.sourceInfo)
 	}
-	buffer.WriteByte(')')
+	buffer.WriteByte('}')
 	return buffer.String()
 }

internal/credentials/anonymous.go

@@ -42,11 +42,11 @@ func (c Anonymous) Token(_ context.Context) (string, error) {
 func (c Anonymous) String() string {
 	buffer := xstring.Buffer()
 	defer buffer.Free()
-	buffer.WriteString("Anonymous(")
+	buffer.WriteString("Anonymous{")
 	if c.sourceInfo != "" {
-		buffer.WriteString("from:")
+		buffer.WriteString("From:")
 		fmt.Fprintf(buffer, "%q", c.sourceInfo)
 	}
-	buffer.WriteByte(')')
+	buffer.WriteByte('}')
 	return buffer.String()
 }

internal/credentials/static.go

@@ -147,16 +147,16 @@ func parseExpiresAt(raw string) (expiresAt time.Time, err error) {
 func (c *Static) String() string {
 	buffer := xstring.Buffer()
 	defer buffer.Free()
-	buffer.WriteString("Static(user:")
+	buffer.WriteString("Static{User:")
 	fmt.Fprintf(buffer, "%q", c.user)
-	buffer.WriteString(",password:")
+	buffer.WriteString(",Password:")
 	fmt.Fprintf(buffer, "%q", secret.Password(c.password))
-	buffer.WriteString(",token:")
+	buffer.WriteString(",Token:")
 	fmt.Fprintf(buffer, "%q", secret.Token(c.token))
 	if c.sourceInfo != "" {
-		buffer.WriteString(",from:")
+		buffer.WriteString(",From:")
 		fmt.Fprintf(buffer, "%q", c.sourceInfo)
 	}
-	buffer.WriteByte(')')
+	buffer.WriteByte('}')
 	return buffer.String()
 }