Thanks for this!@SmsS4

I tried to useNewWithConfigFile() function withgoph.NewConn() but gotopen ~/.ssh/identity: no such file or directory error. Any ideas? 🤔

hmmmm this PRis for 1.5 years ago
I will check it tomorrow :)

Thanks for prompt reply! Also it'd be nice to add another function to return onlyAuth:

funcGetAuth(hoststring) (goph.Auth,error) {auth,err:=goph.Key(ssh_config.Get(host,"IdentityFile"),"")iferr!=nil {returnnil,fmt.Errorf("get key: %w",err)}returnauth,nil}

I fixed your comments

Thanks for prompt reply! Also it'd be nice to add another function to return onlyAuth:

funcGetAuth(hoststring) (goph.Auth,error) {auth,err:=goph.Key(ssh_config.Get(host,"IdentityFile"),"")iferr!=nil {returnnil,fmt.Errorf("get key: %w",err)}returnauth,nil}

I used your code in new commit
thanks

By defaultssh_config.Get(host, "IdentityFile") returns~/.ssh/identity so you have to setIdentityFile in ssh config.
for example:

Host vpnHostName 127.0.0.1User testIdentityFile /home/demol/.ssh/id_rsa

Hey thanks for the update,

one more thing if you can useNewConn instead ofNew in case of custom port with fallback to 22.

v2 of this package I should add more configuration options suport

By default ssh_config.Get(host, "IdentityFile") returns ~/.ssh/identity so you have to set IdentityFile in ssh config.

I was wondering if it does make sense toreturn an error in case ifIdentityFile is empty or equals tossh_config.Default("IdentityFile")

Also you'd consider to remove leading and trailing"s, I just filed an issue for that:kevinburke/ssh_config#61

strings.Trim(ssh_config.Get(host,key),"\"")

I will update my fork and useNewConn instead ofNew (probably tomorrow or day after tomorrow
Thanks@melbahja

Hey thanks for the update,

one more thing if you can useNewConn instead ofNew in case of custom port with fallback to 22.

v2 of this package I should add more configuration options suport

By default ssh_config.Get(host, "IdentityFile") returns ~/.ssh/identity so you have to set IdentityFile in ssh config.

I was wondering if it does make sense toreturn an error in case ifIdentityFile is empty or equals tossh_config.Default("IdentityFile")

Also you'd consider to remove leading and trailing"s, I just filed an issue for that:kevinburke/ssh_config#61

strings.Trim(ssh_config.Get(host,key),"\"")

It's probably good idea to use~/.ssh/id_rsa by default and not~/.ssh/identity but I only return error if~/.ssh/identity doesn't exists.
I will update this soon@Dentrax

@melbahja
I used NewConn instead of New with a little bit of refactoring

I'll need a bit of time to test the changes. Thanks.

Can you review these changes ? this pr could really help us.

Thessh_config lib has many issues like:

• can't find the default IdentityFile if not set in the config file.
• if your~/.ssh/config or '/etc/ssh/ssh_config' file containsInclude the libGet will always returns empty values
• @Dentrax issue will still exists even with Trim bcz the fallbacks to not exists file~/.ssh/identity, see:https://github.com/kevinburke/ssh_config/blob/974a52c2c04b3e24e189311ef9776b87153dd961/validators.go#L120

I think you should solve the ssh config problem outside of goph and just pass the hostname, port, and identity to it, and I don't think this is a good think to add it to v1 maybe in v2.