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

fixed add in paramiko/hostkeys.py #2356

Closed
wants to merge 1 commit into from

Conversation

uchihaMadara-7
Copy link

This change resolve #2355 by comparing hostname literally and with hashed hostname
Instead of adding key blindly if hostname is not found
I added a check of _hostname_matches, which will match the hashed hostname also

@uchihaMadara-7 uchihaMadara-7 deleted the add_host_keys branch February 9, 2024 11:53
@uchihaMadara-7 uchihaMadara-7 restored the add_host_keys branch February 9, 2024 11:54
@uchihaMadara-7 uchihaMadara-7 reopened this Feb 9, 2024
@uchihaMadara-7 uchihaMadara-7 deleted the add_host_keys branch February 9, 2024 11:57
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.

[BUG] - <title> HostKeys.add() duplicates entries instead of overriding existing ones
1 participant