node-triton icon indicating copy to clipboard operation
node-triton copied to clipboard

Triton SSH w/ Git Bash - ssh version change breaks

Open qdzlug opened this issue 7 years ago • 3 comments

Hi,

Under Git Bash (installed using the Git tooling), using triton ssh works as expected under:

$ ssh -V
OpenSSH_7.3p1, OpenSSL 1.0.2j  26 Sep 2016

However, if you are using:

$ ssh -V
OpenSSH_7.6p1, OpenSSL 1.0.2m  2 Nov 2017

You receive this error:

$ triton ssh 3a1f3346
command-line line 0: garbage at end of line; "Cobine\\AppData\\Roaming\\Joyent\\Triton\\tmp\\nullSshControlPath".

Looks to be something perhaps with a default change with the ssh client?

Jay

qdzlug avatar Jan 05 '18 15:01 qdzlug

Robby restested this using OpenSSH for Windows version 7. This is still an issue but would be good to correct for our customers.

ccobine avatar Nov 05 '19 19:11 ccobine

One of our users just ran into this as well. They are the only user with a space in their Windows username. Seems like this line should quote the path:

https://github.com/TritonDataCenter/node-triton/blob/master/lib/do_instance/do_ssh.js#L316

Will be testing this shortly and if a change to that line fixes the issue I'll create a PR.

blackwood821 avatar May 31 '23 12:05 blackwood821

I created a pull request for a change that addresses this issue: https://github.com/TritonDataCenter/node-triton/pull/334

blackwood821 avatar Jun 13 '23 20:06 blackwood821