From 553c61def61d7bf7decd326c7d7ef8dd0db9246b Mon Sep 17 00:00:00 2001 From: Dmitry Moskowski Date: Sun, 21 Aug 2022 06:02:44 +0000 Subject: [PATCH] fix bastion ssh config extension --- provider/provider.go | 32 +++++++++++++++----------------- 1 file changed, 15 insertions(+), 17 deletions(-) diff --git a/provider/provider.go b/provider/provider.go index b05ede3..1c16152 100644 --- a/provider/provider.go +++ b/provider/provider.go @@ -304,23 +304,21 @@ func (p *Provider) NewSsh(resource ResourceBox) *Ssh { bastionHost, _ := bastionSettings[KeySshHost].(string) if bastionHost != "" { bastionConfigMap := p.SshConfigMap(bastionSettings) - if bastionConfigMap.Len() > 0 { - // NOTE: base ssh configuration (ssh {}) extended with bastion ssh configuration (ssh { bastion {} }) - extendedBastionConfigMap := configMap.Copy() - extendedBastionConfigMap.Extend(bastionConfigMap) - - bastion := NewSsh( - SshOptionConfigMap(extendedBastionConfigMap), - SshOptionNonInteractive(), - SshOptionIORedirection("%h", "%p"), - SshOptionHost(bastionHost), - ) - command, arguments, _ := bastion.Command() - configMap.Set( - SshConfigKeyProxyCommand, - strings.Join(append([]string{command}, arguments...), " "), - ) - } + // NOTE: base ssh configuration (ssh {}) extended with bastion ssh configuration (ssh { bastion {} }) + extendedBastionConfigMap := configMap.Copy() + extendedBastionConfigMap.Extend(bastionConfigMap) + + bastion := NewSsh( + SshOptionConfigMap(extendedBastionConfigMap), + SshOptionNonInteractive(), + SshOptionIORedirection("%h", "%p"), + SshOptionHost(bastionHost), + ) + command, arguments, _ := bastion.Command() + configMap.Set( + SshConfigKeyProxyCommand, + strings.Join(append([]string{command}, arguments...), " "), + ) } if configMap.Len() > 0 {