bitprophet/ssh — Issue #6: Make SSHConfig correctly interpret IdentityFile
Author: bitprophet
Created: 2012-02-03 20:07:41+00:00
State: open

The SSH spec says that multiple `IdentityFile` entries may be given and will be checked in order.

However, `ssh.SSHConfig` naively rolls everything into single-value dicts, and so what happens is the "last" `IdentityFile` it finds becomes the value of the resulting `identityfile` key (from `lookup()`).

It should be amended to make that value either always a list, or at least a list when >1 value is found. The former feels cleaner to me as the latter requires clients to test (like Fabric does for its own `env.key_filename` setting.)