[BUG] - SSH config files with Include sections are not supported.

gvoysey commented 1 month ago

Description

Reading Host specifications from a ssh config file does not support files that use the Include directive. termscp host-name fails to connect.

Steps to reproduce

I have my ~/.ssh/config file as follows:

Include config-work
Include config-personal

and inside config-work i have a properly credentialed host:

Host foo
  HostName foo.work.com
  IdentityFile ~/.ssh/foo-key
  User work-user

If i termscp foo, i am prompted to enter the (nonexistent) password for foo (the Host).

If i move the host declaration into the ~/.ssh/config file instead of an Include-d one, I connect to the host as expected.

Expected behaviour

I expect termscp foo to work with a ssh config file that partitions Hosts by Include-d files.

Environment

OS: Arch
Architecture x86_64
Rust version cargo 1.82.0 (8f40fc59f 2024-08-21)
termscp version termscp v0.16.0 (VERGEN_IDEMPOTENT_OUTPUT, VERGEN_I, 2024-10-15T21:08:53.000000000Z) - Developed by Christian Visintin christian.visintin@veeso.dev (via https://archlinux.org/packages/extra/x86_64/termscp/)
Protocol used scp/sftp
Remote server version and name

github-actions[bot] commented 1 day ago

This issue is stale because it has been open for 30 days with no activity.

gvoysey commented 1 day ago

bump

On Sun, Dec 1, 2024 at 21:10 github-actions[bot] @.***> wrote:

This issue is stale because it has been open for 30 days with no activity.

— Reply to this email directly, view it on GitHub https://github.com/veeso/termscp/issues/309#issuecomment-2510415332, or unsubscribe https://github.com/notifications/unsubscribe-auth/AA3ZD325OVCTWL32PHQXCMT2DO6P5AVCNFSM6AAAAABRASQSXKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKMJQGQYTKMZTGI . You are receiving this because you authored the thread.Message ID: @.***>

veeso / termscp