RFR: 343: Use `ssh -G` for parsing ssh configuration files
Robin Westberg
rwestberg at openjdk.java.net
Tue Mar 31 07:24:39 UTC 2020
On Tue, 31 Mar 2020 07:17:15 GMT, Erik Helin <ehelin 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!
-------------
Marked as reviewed by rwestberg (Reviewer).
PR: https://git.openjdk.java.net/skara/pull/553
More information about the skara-dev
mailing list