Merge pull request docker#5345 from thaJeztah/connhelper_cleanups

cli/connhelper: getConnectionHelper: move ssh-option funcs out of closure
thaJeztah · Aug 13, 2024 · 35666cf · 35666cf
2 parents 1a7b7dd + 0fd3fb0
commit 35666cf
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/cli/connhelper/connhelper.go b/cli/connhelper/connhelper.go
@@ -45,14 +45,14 @@ func getConnectionHelper(daemonURL string, sshFlags []string) (*ConnectionHelper
 		if err != nil {
 			return nil, errors.Wrap(err, "ssh host connection is not valid")
 		}
+		sshFlags = addSSHTimeout(sshFlags)
+		sshFlags = disablePseudoTerminalAllocation(sshFlags)
 		return &ConnectionHelper{
 			Dialer: func(ctx context.Context, network, addr string) (net.Conn, error) {
 				args := []string{"docker"}
 				if sp.Path != "" {
 					args = append(args, "--host", "unix://"+sp.Path)
 				}
-				sshFlags = addSSHTimeout(sshFlags)
-				sshFlags = disablePseudoTerminalAllocation(sshFlags)
 				args = append(args, "system", "dial-stdio")
 				return commandconn.New(ctx, "ssh", append(sshFlags, sp.Args(args...)...)...)
 			},