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.

Sign up for GitHub

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

Jump to bottom

MT-55 Catch and handle Paramiko SSH errors #48

Merged
merged 4 commits into from
Aug 23, 2023
Merged

MT-55 Catch and handle Paramiko SSH errors #48

merged 4 commits into from
Aug 23, 2023

Commits on Jul 17, 2023

  1. MT-55 Catch and handle Paramiko SSH errors 

    Added a custom Exception class SSHConnectionError.
Wraps Remote class with try except raise condition.
Prints more explicit error message with possible workarounds instead of basic Exception error message which recommends to file a bug.
    Ganda Tchabana committed Jul 17, 2023
    Configuration menu
    Copy the full SHA
    10b63e4 View commit details
    Browse the repository at this point in the history

  2. Tweak Ganda's code a bit 

    Signed-off-by: Fabrice Normandin <normandf@mila.quebec>
    @lebrice
    lebrice committed Jul 17, 2023
    Configuration menu
    Copy the full SHA
    d395e73 View commit details
    Browse the repository at this point in the history

Commits on Aug 18, 2023

  1. MT-55 

    Reformat Paramiko SSH Exception error message and made it more specific
    Ganda Tchabana committed Aug 18, 2023
    Configuration menu
    Copy the full SHA
    51d4bfc View commit details
    Browse the repository at this point in the history

Commits on Aug 23, 2023

  1. Clean code from comments and unnecessary exception handling 

    Commit for code review suggestions

Co-authored-by: Olivier Breuleux <breuleux@gmail.com>
    @gandatchabana @breuleux
    gandatchabana and breuleux authored Aug 23, 2023
    Configuration menu
    Copy the full SHA
    4115ae6 View commit details
    Browse the repository at this point in the history