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