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

1633 port genkey command #98

Merged
merged 5 commits into from
Nov 12, 2024
Merged

1633 port genkey command #98

merged 5 commits into from
Nov 12, 2024

Conversation

jimmykarily
Copy link
Contributor

@jimmykarily jimmykarily force-pushed the 1633-port-genkey-command branch from 45ed607 to be4fe1a Compare November 12, 2024 05:34
@jimmykarily jimmykarily marked this pull request as ready for review November 12, 2024 05:34
@jimmykarily jimmykarily requested a review from a team November 12, 2024 05:35
Copy link
Member

@Itxaka Itxaka left a comment

Choose a reason for hiding this comment

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

Im not reading all this, this is just the genkey command dumped into here right?

@jimmykarily
Copy link
Contributor Author

Im not reading all this, this is just the genkey command dumped into here right?

yes with some minor changes to read the args from the command ctx instead of viper.

as part of kairos-io/kairos#1633

Signed-off-by: Dimitris Karakasilis <dimitris@karakasilis.me>
Signed-off-by: Dimitris Karakasilis <dimitris@karakasilis.me>
Signed-off-by: Dimitris Karakasilis <dimitris@karakasilis.me>
Not sure why the asus certs have a negative serial number but they do.
If the asus box had that others might too, so we should better support
it. The alternative would be to generate certs with positive serial
number for the tests.

https://github.com/golang/go/blob/master/src/crypto/x509/parser.go#L1014-L1018
microsoft/mssql-docker#895 (comment)

Signed-off-by: Dimitris Karakasilis <dimitris@karakasilis.me>
Signed-off-by: Dimitris Karakasilis <dimitris@karakasilis.me>
@jimmykarily jimmykarily force-pushed the 1633-port-genkey-command branch from be4fe1a to 7694cf1 Compare November 12, 2024 10:21
@jimmykarily jimmykarily merged commit cb118b3 into main Nov 12, 2024
8 checks passed
@jimmykarily jimmykarily deleted the 1633-port-genkey-command branch November 12, 2024 10:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

2 participants