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

Print user@host when connecting with sftp #473

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

neverpanic
Copy link

Prior to d989217110932490ba8ce92127a9a6838878928b, using sftp user@host would print "Connected to user@host", because the 'host' variable would also contain the username.

Parsing switched in the above mentioned commit for good reasons, but some users prefer to see the username if it was specified on the command line and never noticed that it did not show up when they specified a path on connection.

Restore the behavior of showing the username if one was specified and make it consistent regardless of whether a path was passed on the command line or not.

Prior to d989217110932490ba8ce92127a9a6838878928b, using sftp user@host
would print "Connected to user@host", because the 'host' variable would
also contain the username.

Parsing switched in the above mentioned commit for good reasons, but
some users prefer to see the username if it was specified on the command
line and never noticed that it did not show up when they specified
a path on connection.

Restore the behavior of showing the username if one was specified and
make it consistent regardless of whether a path was passed on the
command line or not.
@neverpanic
Copy link
Author

The two CI failures seem to be due to network issues and unrelated to the PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant