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

Add informative BadHostKeyException #404

Closed
wants to merge 1 commit into from
Closed

Add informative BadHostKeyException #404

wants to merge 1 commit into from

Conversation

@ghost
Copy link

@ghost ghost commented Sep 25, 2014

Displaying the keys being compared makes it easy to diagnose the
problem. Otherwise there is more guessing involved.

Signed-off-by: Loic Dachary loic@dachary.org

@ghost ghost changed the title Add informative BadHostKeyException str() Add informative BadHostKeyException Sep 25, 2014
Displaying the keys being compared makes it easy to diagnose the
problem. Otherwise there is more guessing involved.

Signed-off-by: Loic Dachary <loic@dachary.org>
@coveralls
Copy link

@coveralls coveralls commented Sep 25, 2014

Coverage Status

Coverage decreased (-0.01%) when pulling ad9a259 on dachary:master into 1af8db2 on paramiko:master.

@coveralls
Copy link

@coveralls coveralls commented Sep 25, 2014

Coverage Status

Coverage increased (+0.06%) when pulling ad9a259 on dachary:master into 1af8db2 on paramiko:master.

@coveralls
Copy link

@coveralls coveralls commented Sep 25, 2014

Coverage Status

Coverage increased (+0.07%) when pulling ad9a259 on dachary:master into 1af8db2 on paramiko:master.

@coveralls
Copy link

@coveralls coveralls commented Sep 25, 2014

Coverage Status

Coverage increased (+0.03%) when pulling ad9a259 on dachary:master into 1af8db2 on paramiko:master.

@lndbrg
Copy link
Contributor

@lndbrg lndbrg commented Oct 3, 2014

LGTM!

@bitprophet bitprophet added this to the 1.13.4 / 1.14.3 / 1.15.3 milestone Sep 30, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

3 participants