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

fix: verify CSR signature before issuing a certificate #11

Merged
merged 1 commit into from
Apr 27, 2021

Conversation

smira
Copy link
Member

@smira smira commented Apr 26, 2021

This is required to make sure that the CSR was generated by the owner of
the private key.

Signed-off-by: Andrey Smirnov smirnov.andrey@gmail.com

This is required to make sure that the CSR was generated by the owner of
the private key.

Signed-off-by: Andrey Smirnov <smirnov.andrey@gmail.com>
@smira
Copy link
Member Author

smira commented Apr 26, 2021

/approve

@smira
Copy link
Member Author

smira commented Apr 27, 2021

/lgtm

@talos-bot talos-bot merged commit 4f80b97 into siderolabs:master Apr 27, 2021
smira added a commit to smira/talos that referenced this pull request Apr 27, 2021
See siderolabs/crypto#11

Signed-off-by: Andrey Smirnov <smirnov.andrey@gmail.com>
talos-bot pushed a commit to siderolabs/talos that referenced this pull request Apr 27, 2021
See siderolabs/crypto#11

Signed-off-by: Andrey Smirnov <smirnov.andrey@gmail.com>
smira added a commit to smira/talos that referenced this pull request May 5, 2021
See siderolabs/crypto#11

Signed-off-by: Andrey Smirnov <smirnov.andrey@gmail.com>
(cherry picked from commit d540a4a)
smira added a commit to smira/talos that referenced this pull request May 5, 2021
See siderolabs/crypto#11

Signed-off-by: Andrey Smirnov <smirnov.andrey@gmail.com>
(cherry picked from commit d540a4a)
smira added a commit to siderolabs/talos that referenced this pull request May 5, 2021
See siderolabs/crypto#11

Signed-off-by: Andrey Smirnov <smirnov.andrey@gmail.com>
(cherry picked from commit d540a4a)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants