From e40aa515f24708a71fd25f97c8aea59c93b5a798 Mon Sep 17 00:00:00 2001 From: Stephen Fox Date: Tue, 5 Feb 2019 12:00:01 -0500 Subject: [PATCH] Added TODO about public key removal logic. --- common/step_cleanup_temp_keys.go | 1 + 1 file changed, 1 insertion(+) diff --git a/common/step_cleanup_temp_keys.go b/common/step_cleanup_temp_keys.go index c44a5c723..74b528a80 100644 --- a/common/step_cleanup_temp_keys.go +++ b/common/step_cleanup_temp_keys.go @@ -38,6 +38,7 @@ func (s *StepCleanupTempKeys) Run(_ context.Context, state multistep.StateBag) m ui.Say("Trying to remove ephemeral keys from authorized_keys files") + // TODO: Why create a backup file if you are going to remove it? cmd.Command = fmt.Sprintf("sed -i.bak '/ %s$/d' ~/.ssh/authorized_keys; rm ~/.ssh/authorized_keys.bak", s.Comm.SSHTemporaryKeyPairName) if err := cmd.StartWithUi(comm, ui); err != nil { log.Printf("Error cleaning up ~/.ssh/authorized_keys; please clean up keys manually: %s", err)