Skip to content

Conversation

iamjpotts
Copy link
Contributor

@iamjpotts iamjpotts commented Sep 21, 2025

Adds an example based on 0.14 that reads an existing certificate authority (root or intermediate) from its PEM files into an Issuer and generates and signs a new leaf/server certificate using that certificate authority.

Alternative to #378.

@iamjpotts iamjpotts marked this pull request as ready for review September 21, 2025 16:37
@djc
Copy link
Member

djc commented Sep 22, 2025

I think this is definitely better than #378.

Copy link
Member

@djc djc left a comment

Choose a reason for hiding this comment

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

Please squash your changes into a single commit.

@iamjpotts iamjpotts force-pushed the jp/sign-server-with-intermediate-pems branch from e33ee51 to 135b0e2 Compare September 23, 2025 23:39
@iamjpotts iamjpotts mentioned this pull request Sep 26, 2025
@iamjpotts iamjpotts force-pushed the jp/sign-server-with-intermediate-pems branch from 135b0e2 to 6b85378 Compare September 26, 2025 01:15
@iamjpotts
Copy link
Contributor Author

Rebased; no changes.

@iamjpotts iamjpotts force-pushed the jp/sign-server-with-intermediate-pems branch from 6b85378 to 9e4fa69 Compare September 26, 2025 01:31
@djc djc requested a review from cpu September 26, 2025 08:34
@djc
Copy link
Member

djc commented Sep 26, 2025

@cpu or @est31 any thoughts?

@iamjpotts iamjpotts force-pushed the jp/sign-server-with-intermediate-pems branch from 9e4fa69 to 5f19737 Compare September 28, 2025 17:07
@iamjpotts iamjpotts force-pushed the jp/sign-server-with-intermediate-pems branch from 5f19737 to ab22e1b Compare October 1, 2025 01:05
@iamjpotts
Copy link
Contributor Author

If #381 is merged first, this will need a rebase and a cargo fmt run applied to it.

@djc djc requested a review from est31 October 1, 2025 08:35
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.

4 participants