Skip to content
New issue

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

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

Already on GitHub? # to your account

ssh_userauth_publickey_auto: should accept empty passphrase #1361

Merged
merged 2 commits into from
Nov 4, 2017
Merged

ssh_userauth_publickey_auto: should accept empty passphrase #1361

merged 2 commits into from
Nov 4, 2017

Conversation

rayrapetyan
Copy link
Contributor

Empty passphrase should be a valid input for ssh_userauth_publickey_auto. Otherwise there is an extra passphrase dialog. It should appear only when ssh_userauth_publickey_auto really fails with empty passphrase.

Empty passphrase should be a valid input for ssh_userauth_publickey_auto. Otherwise there is an extra passphrase dialog.
@antenore
Copy link
Member

antenore commented Nov 4, 2017

Well done, thank you!
Can you just remove entirely the commented if?

I'll do some tests later and then I will merge it.

- removed commented "if" statement completely
@antenore
Copy link
Member

antenore commented Nov 4, 2017

Thanks.

@antenore
Copy link
Member

antenore commented Nov 4, 2017

Neat, remember me to add your name in the AUTHORS and in the remmina_about.glade file before the next release.

@antenore antenore merged commit 66049f6 into FreeRDP:next Nov 4, 2017
@rayrapetyan
Copy link
Contributor Author

Thanks.
Seems this patch is still not ideal. E.g. there are situations when first call to ssh_userauth_publickey_auto ends up with a timeout (e.g. server is temporary busy), it will raise a passphrase prompt window (which is unrelated in this given situation).
Looks like there is no nice way to determine if passphrase is set or not yet. Here are some more thoughts on this issue: https://security.stackexchange.com/questions/129724/how-to-check-if-an-ssh-private-key-has-passphrase-or-not.

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

Successfully merging this pull request may close these issues.

2 participants