Apply suggestions from code review
Co-authored-by: Jose Garcia <josebalius@github.com>
This commit is contained in:
parent
049a3676f4
commit
045bb54157
1 changed files with 2 additions and 2 deletions
|
|
@ -277,7 +277,7 @@ func selectSSHKeys(
|
||||||
|
|
||||||
configKeyPair, err := firstConfiguredKeyPair(ctx, customConfigPath, opts.profile)
|
configKeyPair, err := firstConfiguredKeyPair(ctx, customConfigPath, opts.profile)
|
||||||
if err != nil {
|
if err != nil {
|
||||||
return nil, false, fmt.Errorf("checking configured keys: %v", err)
|
return nil, false, fmt.Errorf("checking configured keys: %w", err)
|
||||||
}
|
}
|
||||||
|
|
||||||
// Even if there was no error, there may simply be no configured keys, so still check for nil here
|
// Even if there was no error, there may simply be no configured keys, so still check for nil here
|
||||||
|
|
@ -288,7 +288,7 @@ func selectSSHKeys(
|
||||||
// Finally, fall back to generating new automatic keys
|
// Finally, fall back to generating new automatic keys
|
||||||
autoKeyPair, err := setupAutomaticSSHKeys(sshContext)
|
autoKeyPair, err := setupAutomaticSSHKeys(sshContext)
|
||||||
if err != nil {
|
if err != nil {
|
||||||
return nil, false, fmt.Errorf("generating automatic keypair: %v", err)
|
return nil, false, fmt.Errorf("generating automatic keypair: %w", err)
|
||||||
}
|
}
|
||||||
|
|
||||||
return autoKeyPair, true, nil
|
return autoKeyPair, true, nil
|
||||||
|
|
|
||||||
Loading…
Add table
Add a link
Reference in a new issue