Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Feature Request] - Add support for Windows Terminal #150

Closed
fcolledani-solari opened this issue Feb 27, 2023 · 9 comments · Fixed by #151
Closed

[Feature Request] - Add support for Windows Terminal #150

fcolledani-solari opened this issue Feb 27, 2023 · 9 comments · Fixed by #151
Assignees
Labels
bug Something isn't working
Milestone

Comments

@fcolledani-solari
Copy link

Description

Hi Veeso! TermSCP installs on Windows Terminal, but it does not connect to anything.
Ather filling the connection form, this is the response I get:

immagine

Look at the image and the destination URL, you will appreciate them

Changes

Support WSL2

Implementation

No idea, most likely some internal file descriptors are not supported by WSL.

@fcolledani-solari fcolledani-solari added the new feature New feature or request label Feb 27, 2023
@veeso
Copy link
Owner

veeso commented Feb 27, 2023

Hi there!

Does the config path exist on your machine?

Please try with

mkdir -p $HOME/.config/termscp

@veeso veeso added bug Something isn't working and removed new feature New feature or request labels Feb 27, 2023
@veeso veeso added this to the 0.11.1 milestone Feb 27, 2023
@veeso veeso linked a pull request Feb 27, 2023 that will close this issue
17 tasks
@fcolledani-solari
Copy link
Author

nano-bug

Yes, this gif shows the step to reproduce the issue.

@veeso
Copy link
Owner

veeso commented Feb 27, 2023

WSL2...

Does the file $HOME/.config/termscp/config.toml exist?
Is the env variable $XDG_CONFIG_HOME unset?

@fcolledani-solari
Copy link
Author

immagine

Yes, the contents of config.toml are in the image. The env var is not set

@veeso
Copy link
Owner

veeso commented Feb 27, 2023

Please try running termscp with -D option and send me the log file located at $HOME/.config/termscp.log

@fcolledani-solari
Copy link
Author

termscp_log.zip

You will have no issues guessing the password. It is not the first you will think of, but the second one.

@veeso
Copy link
Owner

veeso commented Feb 27, 2023

It's failing the parse of your ssh config. Probably contains some non standard keywords.

2023-02-27T13:35:49.233729543Z [TRACE] (1) remotefs_ssh::ssh::config: [/root/.cargo/registry/src/github.com-1ecc6299db9ec823/remotefs-ssh-0.1.3/src/ssh/config.rs:68] Parsing configuration at /home/.../.ssh/config .

Try to remove the configuration path from your configuration.

@fcolledani-solari
Copy link
Author

Replied in private.

@veeso
Copy link
Owner

veeso commented Feb 28, 2023

Fixed: the problem was that ssh_config_path was set by default to ~/.ssh/config but without checking whether this file exists. So from 0.11.1 this value will be set only if the file exists.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants