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

Handle missing SSH key error #69

Merged
merged 3 commits into from Mar 11, 2021
Merged

Handle missing SSH key error #69

merged 3 commits into from Mar 11, 2021

Commits on Mar 4, 2021

  1. Use catch all exception

    andersy005 committed Mar 4, 2021
    Configuration menu
    Copy the full SHA
    4705e90 View commit details
    Browse the repository at this point in the history
  2. Configuration menu
    Copy the full SHA
    101edf5 View commit details
    Browse the repository at this point in the history

Commits on Mar 5, 2021

  1. Add failback if auth_interactive_dumb fails

    When loc_transport.auth_interactive_dumb() returns an exception (which was
    happening on the CGD machines when a user did not set up sshkeys), fallback to
    loc_transport.auth_password() instead.
    
    I don't fully understand why auth_interactive_dumb() was failing on the CGD
    machines, so I don't know if this is a general solution or if I got lucky that
    it works on the CGD machines but future machines may reject both
    auth_interactive_dumb() and auth_password()... hopefully it's the former.
    mnlevy1981 committed Mar 5, 2021
    Configuration menu
    Copy the full SHA
    14b3391 View commit details
    Browse the repository at this point in the history