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

clib: ssh-agent storage #766

Merged
merged 3 commits into from
Aug 25, 2021
Merged

clib: ssh-agent storage #766

merged 3 commits into from
Aug 25, 2021

Conversation

FintanH
Copy link
Contributor

@FintanH FintanH commented Aug 20, 2021

@FintanH FintanH requested a review from a team as a code owner August 20, 2021 14:23
@FintanH FintanH marked this pull request as draft August 20, 2021 14:23
alexjg
alexjg previously approved these changes Aug 20, 2021
@FintanH FintanH mentioned this pull request Aug 20, 2021
@FintanH FintanH requested review from kim and alexjg August 25, 2021 08:24
kim
kim previously approved these changes Aug 25, 2021
Copy link
Contributor

@kim kim left a comment

Choose a reason for hiding this comment

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

lgtm

To allow us to easily add the PublicKey to the ssh-agent in keystore, we
need to able to convert to the keystore's version of a PublicKey.

Signed-off-by: Fintan Halpenny <[email protected]>
We add the functionality for initialising the storage via the ssh-agent.

Signed-off-by: Fintan Halpenny <[email protected]>
There are code paths that require having a Signer, for example needing
to set up `transport::Settings`. Since we are already fetching the
signer to create the storage we also return it.

Signed-off-by: Fintan Halpenny <[email protected]>
@FintanH FintanH merged commit fdc6db8 into master Aug 25, 2021
@FintanH FintanH deleted the clib-ssh branch August 25, 2021 09:57
@github-pages github-pages bot temporarily deployed to github-pages August 25, 2021 09:57 Inactive
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.

3 participants