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

libgit2: Add support for hashed known_hosts #720

Merged
merged 2 commits into from
May 16, 2022
Merged

Conversation

pjbgf
Copy link
Member

@pjbgf pjbgf commented May 13, 2022

Hashed known_hosts was previously only supported when using go-git.
Now both Git implementations benefit from this feature, and the code coverage across them can ensure no future regression.

Fixes: fluxcd/flux2#1593
Depends on: fluxcd/pkg#274

Both libgit2 and go-git now have tests that confirms
support for the supported authentication algorithms.
This test was created previously and is no longer
needed.

Signed-off-by: Paulo Gomes <[email protected]>
@pjbgf pjbgf added the area/git Git related issues and pull requests label May 13, 2022
@pjbgf pjbgf added this to the GA milestone May 13, 2022
Hashed known_hosts was previously only supported when using
go-git. Now both Git implementations benefit from this
features, and the code coverage across them can ensure no
future regression.

Signed-off-by: Paulo Gomes <[email protected]>
@pjbgf pjbgf marked this pull request as ready for review May 16, 2022 15:59
@pjbgf pjbgf requested a review from hiddeco May 16, 2022 16:45
Copy link
Member

@hiddeco hiddeco left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Thanks @pjbgf

@pjbgf pjbgf added the enhancement New feature or request label May 16, 2022
@pjbgf pjbgf merged commit b31c98f into fluxcd:main May 16, 2022
@pjbgf pjbgf deleted the git-tests branch May 16, 2022 17:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/git Git related issues and pull requests enhancement New feature or request
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

Libgit2 doesn't support hashed hosts in known_hosts file
2 participants