Fix lints

lib/tbot/cli/start_database.go

@@ -34,10 +34,10 @@ type DatabaseCommand struct {
 	*sharedDestinationArgs
 	*genericMutatorHandler
 
-	Format      string
-	Service     string
-	Username    string
-	Database    string
+	Format   string
+	Service  string
+	Username string
+	Database string
 }
 
 // NewDatabaseCommand initializes a command and flags for database outputs and

lib/tbot/cli/start_identity.go

@@ -34,7 +34,7 @@ type IdentityCommand struct {
 	*sharedDestinationArgs
 	*genericMutatorHandler
 
-	Cluster     string
+	Cluster string
 }
 
 // NewIdentityCommand initializes the command and flags for identity outputs

lib/tbot/cli/start_shared.go

@@ -238,7 +238,7 @@ func (s *sharedStartArgs) ApplyConfig(cfg *config.BotConfig, l *slog.Logger) err
 
 // sharedDestinationArgs are arguments common to all commands that accept a
 // --destination flag and any related flags. Downstream commands will need to
-// call `BuildDestination()` to retreive the value.
+// call `BuildDestination()` to retrieve the value.
 type sharedDestinationArgs struct {
 	Destination  string
 	ReaderUsers  []string

lib/tbot/config/destination_directory.go

@@ -168,12 +168,13 @@ func (dd *DestinationDirectory) Init(ctx context.Context, subdirs []string) erro
 					"underlying issue.")
 			}
 
+			const msg = "ACLs were requested but could not be configured, they " +
+				"will be disabled. To resolve this warning, ensure ACLs are " +
+				"supported and the directory is owned by the bot user, or " +
+				"remove any configured readers to disable ACLs"
 			log.WarnContext(
 				ctx,
-				"ACLs were requested but could not be configured, they will be "+
-					"disabled. To resolve this warning, ensure ACLs are supported "+
-					"and the directory is owned by the bot user, or remove any "+
-					"configured readers to disable ACLs",
+				msg,
 				"path", dd.Path,
 				"error", err,
 				"readers", dd.Readers,
@@ -322,11 +323,12 @@ func (dd *DestinationDirectory) Verify(keys []string) error {
 	if len(dd.Readers) > 0 {
 		// TODO: consider moving ownership check into Init()?
 		if !ownedByBot {
+			const msg = "This destination is not owned by the bot user so ACLs " +
+				"will not be enforced. To silence this warning, fix destination " +
+				"file ownership or remove configured `readers`"
 			log.WarnContext(
 				context.TODO(),
-				"This destination is not owned by the bot user so ACLs will "+
-					"not be enforced. To silence this warning, fix destination "+
-					"file ownership or remove configured `readers`",
+				msg,
 				"path", dd.Path,
 			)
 

lib/tbot/config/destination_directory_test.go

@@ -24,11 +24,11 @@ import (
 	"path/filepath"
 	"testing"
 
+	"github.com/gravitational/trace"
 	"github.com/stretchr/testify/require"
 
 	"github.com/gravitational/teleport/lib/tbot/botfs"
 	"github.com/gravitational/teleport/lib/utils"
-	"github.com/gravitational/trace"
 )
 
 func TestDestinationDirectory_Lock(t *testing.T) {