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

Signing certificates based on CSR #28

Open
kimmokantojarvi opened this issue Aug 3, 2020 · 5 comments
Open

Signing certificates based on CSR #28

kimmokantojarvi opened this issue Aug 3, 2020 · 5 comments
Labels
enhancement New feature or request

Comments

@kimmokantojarvi
Copy link

Hi,
I like the package and therefore would like to propose some small enhancements for signing the certificates.
Basically functions in CreateCertificates.cs always create a new key pair instead of using existing keys from a CSR and therefore would like to propose adding possibility to sign certificates based on CSRs.
Instead of creating CertificateRequest inside the function allow bringing it as a parameter to the functions.
Functions affected:

  • NewECDsaSelfSignedCertificate
  • NewRsaSelfSignedCertificate
  • NewRsaChainedCertificate
  • NewECDsaChainedCertificate
    Best regards,
    Kimmo
@kimmokantojarvi
Copy link
Author

Just added PR for this one if it helps.

@damienbod
Copy link
Owner

damienbod commented Aug 4, 2020

@kimmokantojarvi thanks! merged

I'll release with version 1.0.5, probably at the weekend

@damienbod
Copy link
Owner

damienbod commented Aug 5, 2020

@kimmokantojarvi just wondering about docs here, maybe a simple example in the repo would be cool showing this feature.

Would you be interested?

@kimmokantojarvi
Copy link
Author

Sure, I can add maybe early next week. Where would you like to have those examples?

@damienbod
Copy link
Owner

damienbod commented Aug 5, 2020

@kimmokantojarvi Thanks! In this repo would be cool.

Examples using:

https://github.com/damienbod/AspNetCoreCertificates/tree/master/examplesUsingCertificateAuthentication

(I'll rename this to /examplesUsingCertificates)

Or creating:

https://github.com/damienbod/AspNetCoreCertificates/tree/master/src

Greetings Damien

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

No branches or pull requests

2 participants