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 Certificate::from_der + Certificate::from_pem #302

Closed
wants to merge 3 commits into from

Conversation

blacktea1105
Copy link

Related to #274 Add Certificate::from_der + Certificate::from_pem.

@djc
Copy link
Member

djc commented Dec 19, 2024

What is your use case? I wrote in #293:

If you still believe there is a use case that rcgen doesn't cover, I suggest you open a new issue with a clear explanation of your use case rather than commenting on this confusingly-named closed PR.

And instead you open another PR that's similar to #293 rather than doing as asked.

@blacktea1105
Copy link
Author

Sorry for the confusion earlier!

I initially thought the leaf certificate invalidity was due to the fingerprint mismatch between the root trust CA certificate in the OS (create and save PEM to file/DB) and the restore/resingned certificate from PEM (after several tests, I found this wasn't the case).

My problem has been resolved (my mistake). Perhaps I should close this PR now?

Thank you very much for your reply and assistance!

@djc djc closed this Dec 19, 2024
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.

2 participants