Merge pull request #473 from aws/NihalM-patch

Fix for issue #144
aws · Oct 25, 2023 · 53460c0 · 53460c0
2 parents 9662e36 + 27af63a
commit 53460c0
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 5 deletions.
diff --git a/ebcli/operations/sshops.py b/ebcli/operations/sshops.py
@@ -132,7 +132,7 @@ def ssh_into_instance(instance_id, keep_open=False, force_open=False, custom_ssh
             custom_ssh = custom_ssh.split()
         else:
             ident_file = _get_ssh_file(keypair_name)
-            custom_ssh = ['ssh', '-i', ident_file]
+            custom_ssh = ['ssh', '-i', ident_file, '-o', 'IdentitiesOnly yes']
 
         custom_ssh.extend([user + '@' + ip])
 

diff --git a/tests/unit/operations/test_sshops.py b/tests/unit/operations/test_sshops.py
@@ -55,7 +55,7 @@ def test_generate_and_upload_keypair__exit_code_0(
                 '-f',
                 os.path.expanduser('~') + '{0}.ssh{0}aws-eb-us-west-2'.format(os.path.sep),
                 '-C',
-                'aws-eb-us-west-2'
+                'aws-eb-us-west-2'                
             ]
         )
 
@@ -320,7 +320,7 @@ def test_ssh_into_instance__uses_private_address(
         call_mock.return_value = 0
 
         sshops.ssh_into_instance('instance-id')
-        call_mock.assert_called_once_with(['ssh', '-i', 'aws-eb-us-west-2', 'ec2-user@172.31.35.210'])
+        call_mock.assert_called_once_with(['ssh', '-i', 'aws-eb-us-west-2', '-o', 'IdentitiesOnly yes', 'ec2-user@172.31.35.210'])
 
     @mock.patch('ebcli.operations.sshops.ec2.describe_instance')
     @mock.patch('ebcli.operations.sshops.ec2.describe_security_group')
@@ -346,7 +346,7 @@ def test_ssh_into_instance__ssh_rule_exists(
         sshops.ssh_into_instance('instance-id')
         authorize_ssh_mock.assert_not_called()
         revoke_ssh_mock.assert_not_called()
-        call_mock.assert_called_once_with(['ssh', '-i', 'aws-eb-us-west-2', 'ec2-user@54.218.96.238'])
+        call_mock.assert_called_once_with(['ssh', '-i', 'aws-eb-us-west-2', '-o', 'IdentitiesOnly yes', 'ec2-user@54.218.96.238'])
 
     @mock.patch('ebcli.operations.sshops.ec2.describe_instance')
     @mock.patch('ebcli.operations.sshops.ec2.describe_security_group')
@@ -372,7 +372,7 @@ def test_ssh_into_instance__no_ssh_rule_exists(
         sshops.ssh_into_instance('instance-id')
         authorize_ssh_mock.assert_called_once_with('sg-12312313')
         revoke_ssh_mock.assert_called_once_with('sg-12312313')
-        call_mock.assert_called_once_with(['ssh', '-i', 'aws-eb-us-west-2', 'ec2-user@54.218.96.238'])
+        call_mock.assert_called_once_with(['ssh', '-i', 'aws-eb-us-west-2', '-o', 'IdentitiesOnly yes', 'ec2-user@54.218.96.238'])
 
     @mock.patch('ebcli.operations.sshops.prompt_for_ec2_keyname')
     @mock.patch('ebcli.operations.sshops.commonops.update_environment')