Yes that looks like a bug. Can you create a bugzilla issue for this please?
-Jaikiran On 17/06/20 10:52 am, Dougal Graham wrote: > OK, after digging a little bit, it turns out that I didn't have a > default identity file set in my ssh config. > > After rectifying that issue, I thought things were fixed, but > actually, they are still broken! > > [sshexec] Loading SSH configuration file /Users/dougal/.ssh/config > [sshexec] Using SSH key file ~/.ssh/id_rsa for host null > [sshexec] Caught exception: host must not be null. > > So I looked at the code again, and it seems that after loading the SSH > config file, it overrides the current host with the host from the SSH > config. So, if you just specify a default identity file in your > config, then the host gets blown away: > host = config.getHostname(); > > Assuming I am interpreting this correctly... > > On Wed, Jun 17, 2020 at 11:59 AM Dougal Graham <doug...@gmail.com > <mailto:doug...@gmail.com>> wrote: > > You are, of course, correct! Thank you. > > OK, running with debug I see: > > [sshexec] Loading SSH configuration file /Users/dougal/.ssh/config > [sshexec] Using SSH key file null for host null > > So maybe it isn't picking up my host/default ssh key properly. ie: > maybe I've configured my host wrong, or my ssh config incorrectly. > > Thanks again for your help! > > On Wed, Jun 17, 2020 at 11:40 AM Jaikiran Pai > <jai.forums2...@gmail.com <mailto:jai.forums2...@gmail.com>> wrote: > > Hello Dougal, > > On 17/06/20 10:07 am, Dougal Graham wrote: >> Hi Jaikaran, >> >> Thanks for getting back to me. I just got the code downloaded >> for Ant, and I think I see the problem: >> >> protected void loadSshConfig() throws BuildException { >> if (sshConfig != null && (userInfo.getName() == null || >> userInfo.getKeyfile() == null)) { >> It would seem that the sshConfig file is only used if both >> name and keyfile are empty. > > Looking at that code, the sshConfig file will be used if the > username *or* (not and) keyfile is not specified. i.e. if > either of them is not specified. So from what I saw in the > original task definition that you pasted, I think the > sshConfig file should have been used. > > -Jaikiran > > > > -- > Dougal Graham > > > > -- > Dougal Graham
