Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Changes logic when no ssh_key_path has been set #216

Merged
merged 1 commit into from
Nov 24, 2015
Merged

Conversation

petems
Copy link
Owner

@petems petems commented Nov 22, 2015

 IdentitiesOnly
             Specifies that ssh(1) should only use the authentication identity
             files configured in the ssh_config files, even if ssh-agent(1) or
             a PKCS11Provider offers more identities.  The argument to this
             keyword must be “yes” or “no”.  This option is intended for situ‐
             ations where ssh-agent offers many different identities.  The
             default is “no”.

So when we have an ssh that is not in a path (such as using an ssh key located on a smart card), you can still ssh to it! 👍

Also updates specs for this change

Closes #160

```
 IdentitiesOnly
             Specifies that ssh(1) should only use the authentication identity
             files configured in the ssh_config files, even if ssh-agent(1) or
             a PKCS11Provider offers more identities.  The argument to this
             keyword must be “yes” or “no”.  This option is intended for situ‐
             ations where ssh-agent offers many different identities.  The
             default is “no”.
```

So when we have an ssh that is not in a path (such as using an ssh key located on a smart card), you can still ssh to it! 👍

Also updates specs for this change

Closes #160
petems added a commit that referenced this pull request Nov 24, 2015
Changes logic when no ssh_key_path has been set
@petems petems merged commit f80006c into master Nov 24, 2015
@rauchrob
Copy link

@petems Great, thank you!

@petems petems deleted the fix_ssh_agent_config branch November 25, 2015 10:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants