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

open() should support /dev/tty #762

Closed
bagajjal opened this issue Jun 7, 2017 · 1 comment
Closed

open() should support /dev/tty #762

bagajjal opened this issue Jun 7, 2017 · 1 comment
Labels
Issue-Enhancement Feature request
Milestone

Comments

@bagajjal
Copy link
Collaborator

bagajjal commented Jun 7, 2017

open() should support /dev/tty.

This is fixed but reverted the changes because password authentication testcases are failing.
With this fix, when we have a terminal SSH will always reads the password from the terminal. This makes our password authentication testcases to fail as there is no way to pass the password through SSH_ASKPASS.

As of now, /dev/tty is only used in the readpassphrase so for now we are not implementing the /dev/tty support.

@bagajjal
Copy link
Collaborator Author

bagajjal commented May 4, 2021

fixed - PowerShell/openssh-portable#506

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

Successfully merging a pull request may close this issue.

2 participants