RFR: 343: Use `ssh -G` for parsing ssh configuration files

Per Lidén pliden at openjdk.java.net
Tue Mar 31 07:47:59 UTC 2020


On Tue, 31 Mar 2020 07:22:31 GMT, Robin Westberg <rwestberg at openjdk.org> wrote:

>> Hi all,
>> 
>> please review this patch that uses `ssh -G` instead of relying on my (somewhat
>> buggy) `ssh_config` parser. Resolving hosts defined in a `ssh_config` file is
>> actually fairly complex and my parser only handled a subset of scenarios. One
>> drawback is that we now rely on `ssh` being present on `PATH`, but that must be
>> the case if a user has a SSH path for a remote.
>> 
>> Testing:
>> - Manual testing using `git-webrev` with various paths
>> 
>> Thanks,
>> Erik
>
> Looks good!

Thanks for fixing! :)

-------------

PR: https://git.openjdk.java.net/skara/pull/553


More information about the skara-dev mailing list