Bug 93 - Added ability for ssh-add to parse config files to retrieve a list of valid IdentityFiles.
Summary: Added ability for ssh-add to parse config files to retrieve a list of valid I...
Status: CLOSED WONTFIX
Alias: None
Product: Portable OpenSSH
Classification: Unclassified
Component: ssh-add (show other bugs)
Version: -current
Hardware: All All
: P2 enhancement
Assignee: OpenSSH Bugzilla mailing list
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2002-02-02 08:44 AEDT by Jason Prondak
Modified: 2005-04-01 01:33 AEST (History)
1 user (show)

See Also:


Attachments
ssh-add.c patch to search ssh_config for IdentityFile(s) (4.15 KB, patch)
2002-02-02 08:45 AEDT, Jason Prondak
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Jason Prondak 2002-02-02 08:44:04 AEDT
This patches allows ssh-add to try and load all the identity files listed in ssh_config or ~/.ssh/config. In addition ssh-add will try and use passphrases 
from a previous key first before prompting for a passphrase (only if there is more than one IdentityFile listed)
Comment 1 Jason Prondak 2002-02-02 08:45:40 AEDT
Created attachment 16 [details]
ssh-add.c patch to search ssh_config for IdentityFile(s)
Comment 2 Markus Friedl 2002-02-04 03:59:30 AEDT
i don't like the idea of parsing ssh_config in ssh-add.

-current ssh-add tries to load all 3 default key.
that's enough for the default.

it's easy to add more keys on the command line.
Comment 3 Nicolas Williams 2002-02-05 02:43:30 AEDT
> i don't like the idea of parsing ssh_config in ssh-add.
> 
> -current ssh-add tries to load all 3 default key.
> that's enough for the default.
> 
> it's easy to add more keys on the command line.

Markus,

One reason we want this is so that we can specify non-standard
locations for identity files for app accounts that run daemons,
things from cron, etc...

It's a small patch and not at all far-fetched.

I just ran smack into the MaxStartups option. How did that
bug^H^H^Hfeature ever get in anyways.

Nico
Comment 4 Nicolas Williams 2002-02-05 03:51:12 AEDT
> I just ran smack into the MaxStartups option. How did that
> bug^H^H^Hfeature ever get in anyways.

I take this back, I get the point of that feature. I ran
into it stress-testing OpenSSH.

:)

Nico
Comment 5 Markus Friedl 2002-02-05 04:33:08 AEDT
/usr/bin/ssh is the only program that should read ssh_config.

passing non-default keys to ssh-add is easy with the current
version of ssh-add.

of course we could have options and defaults for all kinds of
uses, but openssh is already very fat.
Comment 6 Kevin Steves 2002-04-05 19:18:25 AEST
wontfix
Comment 7 Damien Miller 2004-04-14 12:24:17 AEST
Mass change of RESOLVED bugs to CLOSED
Comment 8 Derek Poon 2005-04-01 01:33:57 AEST
I was just going to make the same suggestion as Jason, but found that the ideas
has been rejected already.  From the user's perspective, it would be really nice
ssh-add just works according to the Principle of Least Surprise, rather than
according to arbitrary decisions of the developer not to read a config file. 
The OpenSSH clients are a suite -- why shouldn't they work together nicely?