From ee76d5a102df172d3d0068816d075d837568c427 Mon Sep 17 00:00:00 2001 From: Tanner Stirrat Date: Wed, 29 Jan 2025 16:34:28 -0700 Subject: [PATCH] Update tracing logic in authn code --- pkg/datastore/credentials.go | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/pkg/datastore/credentials.go b/pkg/datastore/credentials.go index 1c557ad818..9c4a093930 100644 --- a/pkg/datastore/credentials.go +++ b/pkg/datastore/credentials.go @@ -87,7 +87,8 @@ func (d awsIamCredentialsProvider) IsCleartextToken() bool { func (d awsIamCredentialsProvider) Get(ctx context.Context, dbEndpoint string, dbUser string) (string, string, error) { authToken, err := rdsauth.BuildAuthToken(ctx, dbEndpoint, d.awsSdkConfig.Region, dbUser, d.awsSdkConfig.Credentials) if err != nil { - log.Ctx(ctx).Trace().Str("region", d.awsSdkConfig.Region).Str("endpoint", dbEndpoint).Str("user", dbUser).Msg("successfully retrieved IAM auth token for DB") + return "", "", err } - return dbUser, authToken, err + log.Ctx(ctx).Trace().Str("region", d.awsSdkConfig.Region).Str("endpoint", dbEndpoint).Str("user", dbUser).Msg("successfully retrieved IAM auth token for DB") + return dbUser, authToken, nil }