feat(storage): suppress AWS SDK logging with Nop logger (#15)

Xe · web-flow · commit d3f9338d9529 · 2026-01-21T14:14:07.000-05:00
Add config.WithLogger(logging.Nop{}) to suppress default AWS SDK
logging output when creating S3 clients.

Closes: TIG-7391

Assisted-by: GLM 4.7 via Claude Code

Signed-off-by: Xe Iaso &lt;xe@tigrisdata.com&gt;
diff --git a/storage.go b/storage.go
@@ -12,6 +12,7 @@ import (
 	awsConfig "github.com/aws/aws-sdk-go-v2/config"
 	"github.com/aws/aws-sdk-go-v2/credentials"
 	"github.com/aws/aws-sdk-go-v2/service/s3"
+	"github.com/aws/smithy-go/logging"
 )
 
 const (
@@ -125,7 +126,9 @@ func New(ctx context.Context, options ...Option) (*Client, error) {
 		creds = credentials.NewStaticCredentialsProvider(o.AccessKeyID, o.SecretAccessKey, "")
 	}
 
-	cfg, err := awsConfig.LoadDefaultConfig(ctx)
+	cfg, err := awsConfig.LoadDefaultConfig(ctx,
+		awsConfig.WithLogger(logging.Nop{}),
+	)
 	if err != nil {
 		return nil, fmt.Errorf("failed to load Tigris config: %w", err)
 	}

Original file line number	Diff line number	Diff line change
`@@ -12,6 +12,7 @@ import (`
`12`	`12`	`awsConfig "github.com/aws/aws-sdk-go-v2/config"`
`13`	`13`	`"github.com/aws/aws-sdk-go-v2/credentials"`
`14`	`14`	`"github.com/aws/aws-sdk-go-v2/service/s3"`
	`15`	`+ "github.com/aws/smithy-go/logging"`
`15`	`16`	`)`
`16`	`17`
`17`	`18`	`const (`
`@@ -125,7 +126,9 @@ func New(ctx context.Context, options ...Option) (*Client, error) {`
`125`	`126`	`creds = credentials.NewStaticCredentialsProvider(o.AccessKeyID, o.SecretAccessKey, "")`
`126`	`127`	`}`
`127`	`128`
`128`		`- cfg, err := awsConfig.LoadDefaultConfig(ctx)`
	`129`	`+ cfg, err := awsConfig.LoadDefaultConfig(ctx,`
	`130`	`+ awsConfig.WithLogger(logging.Nop{}),`
	`131`	`+ )`
`129`	`132`	`if err != nil {`
`130`	`133`	`return nil, fmt.Errorf("failed to load Tigris config: %w", err)`
`131`	`134`	`}`