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

feat: add user keys #123

Closed
wants to merge 16 commits into from
Closed

feat: add user keys #123

wants to merge 16 commits into from

Conversation

caarlos0
Copy link
Member

@caarlos0 caarlos0 commented Apr 21, 2022

This PR allows the user to add its own keys to their charm account.

This includes yubikeys or anything else the x/crypto libs support.

This is still not using the actual user keys for anything... for that we would probably want to support ssh agents et al as well

Likely depends on #121

This PR allows the user to add its own keys to their charm account.

This includes yubikeys or anything else the x/crypto libs support.

Signed-off-by: Carlos A Becker <caarlos0@gmail.com>
@caarlos0 caarlos0 self-assigned this Apr 21, 2022
@caarlos0 caarlos0 added the enhancement New feature or request label Apr 21, 2022
@caarlos0 caarlos0 marked this pull request as ready for review April 21, 2022 19:37
@bashbunni
Copy link
Member

yubikeys >:o

Signed-off-by: Carlos A Becker <caarlos0@gmail.com>
Signed-off-by: Carlos A Becker <caarlos0@gmail.com>
Signed-off-by: Carlos A Becker <caarlos0@gmail.com>
Signed-off-by: Carlos A Becker <caarlos0@gmail.com>
Signed-off-by: Carlos A Becker <caarlos0@gmail.com>
Signed-off-by: Carlos A Becker <caarlos0@gmail.com>
Signed-off-by: Carlos A Becker <caarlos0@gmail.com>
client/client.go Outdated Show resolved Hide resolved
Signed-off-by: Carlos A Becker <caarlos0@gmail.com>
Signed-off-by: Carlos A Becker <caarlos0@gmail.com>
Signed-off-by: Carlos A Becker <caarlos0@gmail.com>
Signed-off-by: Carlos A Becker <caarlos0@gmail.com>
Signed-off-by: Carlos A Becker <caarlos0@gmail.com>
Signed-off-by: Carlos A Becker <caarlos0@gmail.com>
@caarlos0 caarlos0 requested a review from toby April 27, 2022 18:08
@caarlos0
Copy link
Member Author

@toby would love your feedback in this one as well

Signed-off-by: Carlos A Becker <caarlos0@gmail.com>
@caarlos0 caarlos0 marked this pull request as draft April 29, 2022 00:43
@caarlos0
Copy link
Member Author

changed it to a draft because a lot of things don't work, and some can't really work the way we're doing things today

so... back to the drawing board :D

@caarlos0 caarlos0 closed this Jun 13, 2023
@caarlos0 caarlos0 deleted the link-keys branch June 13, 2023 18:29
@bashbunni
Copy link
Member

oof

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants