Merge pull request #24506 from timstclair/ssh

Automatic merge from submit-queue Don't log private SSH key Log files may have more inclusive permissions than private SSH keys, and as such we should not log the key, even if it looks invalid. I accidentally leaked my key this way when posting e2e test logs.

Merge pull request #24506 from timstclair/ssh
4c9bbea0 · k8s-merge-robot · 28f968aa · df42b984 · 4c9bbea0
Commit 4c9bbea0 authored Apr 19, 2016 by k8s-merge-robot
Hide whitespace changes
Inline Side-by-side

Showing with 1 addition and 1 deletion

ssh.go pkg/ssh/ssh.go +1 -1

No files found.
--- a/pkg/ssh/ssh.go
+++ b/pkg/ssh/ssh.go
@@ -265,7 +265,7 @@ func MakePrivateKeySignerFromFile(key string) (ssh.Signer, error) {
 func MakePrivateKeySignerFromBytes(buffer []byte) (ssh.Signer, error) {
 	signer, err := ssh.ParsePrivateKey(buffer)
 	if err != nil {
-		return nil, fmt.Errorf("error parsing SSH key %s: '%v'", buffer, err)
+		return nil, fmt.Errorf("error parsing SSH key: '%v'", err)
 	}
 	return signer, nil
 }