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

Provide user feedback for common SSH configuration issues #170

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

qguv
Copy link

@qguv qguv commented Dec 2, 2019

This PR introduces error messages to clarify some regular issues I encounter:

  • Sup always tries to connect to a SSH_AUTH_SOCK even if one is not defined
  • Sup silently continues if a defined SSH_AUTH_SOCK is unreachable
  • Sup doesn't report any permissions issues preventing keys from being read
  • Sup can't currently handle passphrase-encrypted keys but produces no warning about this

Dependencies needed to be updated to accomplish this last task, and a small change was required to maintain the existing (lack of) SSH hostname check behavior.

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